Change the definition of post_current_song()

There was redundant match statement in main.rs because post_current_song() returns Result<(), String>.

Author: Yourein

base/src/main.rs

@@ -79,14 +79,7 @@ async fn main() {
                     std::process::exit(0);
                 }
                 1 => {
-                    match post_current_song(&tapi, &mut sapi).await {
-                        Ok(()) => {
-                            println!{"Posted Successfully"};
-                        }
-                        Err(e) => {
-                            eprintln!{"Error!\nExpected reason: {}", e};
-                        }
-                    }
+                    post_current_song(&tapi, &mut sapi).await
                 },
                 _ => {
                     eprintln!{"Unknown choice selected. Please try again"};

base/src/main_logics.rs

@@ -50,10 +50,7 @@ pub async fn post_current_song_on_spotify(
     }
 }
 
-pub async fn post_current_song(
-    papi: &dyn PostAPI,
-    sapi: &mut SpotifyAPI<'_>
-) -> Result<(), String> {
+pub async fn post_current_song(papi: &dyn PostAPI, sapi: &mut SpotifyAPI<'_>) {
     match sapi.fetch_current_song().await {
         Ok(raw) => {
             let resp = sapi.parse_current_song_result(raw);
@@ -66,12 +63,12 @@ pub async fn post_current_song(
             };
 
             match post_result {
-                Ok(_) => Ok(()),
-                Err(reason) => Err(reason.to_string())
+                Ok(_) => println!{"Posted Successfully."},
+                Err(reason) => eprintln!{"Error!\nExpected reason: {}", reason.to_string()}
             }
         }
         Err(reason) => {
-            Err(reason.to_string())
+            eprintln!{"Error!\nExpected reason: {}", reason.to_string()};
         }
     }
 }