refactor: change from match to if for NO_EMOJI

Author: matt-lebl

src/exercise.rs

@@ -127,9 +127,10 @@ name = "{}"
 path = "{}.rs""#,
                     self.name, self.name, self.name
                 );
-                let cargo_toml_error_msg = match env::var("NO_EMOJI").is_ok() {
-                    true => "Failed to write Clippy Cargo.toml file.",
-                    false => "Failed to write 📎 Clippy 📎 Cargo.toml file."
+                let cargo_toml_error_msg = if env::var("NO_EMOJI").is_ok() {
+                    "Failed to write Clippy Cargo.toml file."
+                } else {
+                    "Failed to write 📎 Clippy 📎 Cargo.toml file."
                 };
                 fs::write(CLIPPY_CARGO_TOML_PATH, cargo_toml)
                     .expect(cargo_toml_error_msg);

src/ui.rs

@@ -3,21 +3,18 @@ macro_rules! warn {
         use std::env;
         use console::{style, Emoji};
         let formatstr = format!($fmt, $ex);
-        match env::var("NO_EMOJI").is_ok() {
-            true => {
-                println!(
-                    "{} {}",
-                    style("!").red(),
-                    style(formatstr).red()
-                );
-            },
-            false => {
-                println!(
-                    "{} {}",
-                    style(Emoji("⚠️ ", "!")).red(),
-                    style(formatstr).red()
-                );
-            }
+        if env::var("NO_EMOJI").is_ok() {
+            println!(
+                "{} {}",
+                style("!").red(),
+                style(formatstr).red()
+            );
+        } else {
+            println!(
+                "{} {}",
+                style(Emoji("⚠️ ", "!")).red(),
+                style(formatstr).red()
+            );
         }
     }};
 }
@@ -27,21 +24,18 @@ macro_rules! success {
         use std::env;
         use console::{style, Emoji};
         let formatstr = format!($fmt, $ex);
-        match env::var("NO_EMOJI").is_ok() {
-            true => {
-                println!(
-                    "{} {}",
-                    style("✓").green(),
-                    style(formatstr).green()
-                );
-            },
-            false => {
-                println!(
-                    "{} {}",
-                    style(Emoji("✅", "✓")).green(),
-                    style(formatstr).green()
-                );
-            }
+        if env::var("NO_EMOJI").is_ok() {
+            println!(
+                "{} {}",
+                style("✓").green(),
+                style(formatstr).green()
+            );
+        } else {
+            println!(
+                "{} {}",
+                style(Emoji("✅", "✓")).green(),
+                style(formatstr).green()
+            );
         }
     }};
 }

src/verify.rs

@@ -140,9 +140,10 @@ fn prompt_for_completion(exercise: &Exercise, prompt_output: Option<String>) ->
 
     let no_emoji = env::var("NO_EMOJI").is_ok();
 
-    let clippy_success_msg = match no_emoji {
-        true => "The code is compiling, and Clippy is happy!",
-        false => "The code is compiling, and 📎 Clippy 📎 is happy!"
+    let clippy_success_msg = if no_emoji {
+        "The code is compiling, and Clippy is happy!"
+    } else {
+        "The code is compiling, and 📎 Clippy 📎 is happy!"
     };
 
     let success_msg = match exercise.mode {
@@ -152,10 +153,11 @@ fn prompt_for_completion(exercise: &Exercise, prompt_output: Option<String>) ->
     };
 
     println!();
-    match no_emoji {
-        true => println!("~*~ {} ~*~", success_msg),
-        false => println!("🎉 🎉  {} 🎉 🎉", success_msg)
-    };
+    if no_emoji {
+        println!("~*~ {} ~*~", success_msg)
+    } else {
+        println!("🎉 🎉  {} 🎉 🎉", success_msg)
+    }
     println!();
 
     if let Some(output) = prompt_output {