code in chapter 6.14.1

refactor using try_from()

src/routes/subscriptions.rs

@@ -1,7 +1,7 @@
 use actix_web::{HttpResponse, web};
 use sqlx::PgPool;
-use uuid::Uuid;
 use sqlx::types::chrono::Utc;
+use uuid::Uuid;
 
 use crate::domain::*;
 
@@ -11,6 +11,16 @@ pub struct FormData {
     name: String,
 }
 
+
+impl TryFrom<FormData> for NewSubscriber {
+    type Error = String;
+    fn try_from(value: FormData) -> Result<Self, Self::Error> {
+        let name = SubscriberName::parse(value.name)?;
+        let email = SubscriberEmail::parse(value.email)?;
+        Ok(Self { email, name })
+    }
+}
+
 // you can test this with:
 // curl -X POST localhost:8000/subscriptions    -H "Content-Type: application/x-www-form-urlencoded"  --data "name=noam&email=g@g.com"
 
@@ -25,16 +35,12 @@ subscriber_name = % form.name
 )]
 pub(crate) async fn subscribe(form: web::Form<FormData>,
                               pool: web::Data<PgPool>, ) -> HttpResponse {
-    let name = match SubscriberName::parse(form.0.name){
-        Ok(name) => name,
-        Err(_) => return HttpResponse::BadRequest().finish(),
-    };
-      let email = match SubscriberEmail::parse(form.0.email){
-        Ok(em) => em,
+    let new_subscriber = match form.0.try_into() {
+        Ok(form) => form,
         Err(_) => return HttpResponse::BadRequest().finish(),
     };
-    let subscriber = NewSubscriber {email, name};
-    match insert_subscriber(&pool, &subscriber).await {
+
+    match insert_subscriber(&pool, &new_subscriber).await {
         Ok(_) => HttpResponse::Ok().finish(),
         Err(_) => HttpResponse::InternalServerError().finish()
     }