Code cleanup

Author: celarye

src/days/_1.rs

@@ -13,7 +13,7 @@ pub fn init() -> Result<_1, ()> {
         collection_2: vec![],
     };
 
-    match utils::http::request(1, None) {
+    match utils::http::request(1) {
         Ok(input) => {
             for input_line in input.split("\n") {
                 let inputs: Vec<&str> = input_line.split("   ").collect();

src/days/_2.rs

@@ -7,7 +7,7 @@ pub struct _2 {
 pub fn init() -> Result<_2, ()> {
     let mut _2 = _2 { collection: vec![] };
 
-    match utils::http::request(2, None) {
+    match utils::http::request(2) {
         Ok(input) => {
             for input_line in input.split("\n") {
                 if input_line.len() == 0 {
@@ -27,8 +27,6 @@ pub fn init() -> Result<_2, ()> {
 
                 _2.collection.push(input_line_collection);
             }
-            // TODO: Fix edge case:
-            // _2.collection.push(vec![18, 21, 19, 17, 16]);
         }
         Err(()) => {
             return Err(());
@@ -43,40 +41,6 @@ impl _2 {
 
         for line_collection in self.collection.iter() {
             result += self.is_valid_line_collection(line_collection, None, 0);
-            // let mut is_valid_line_collection = true;
-
-            // let mut p_previous_number: Option<&i32> = None;
-            // let mut p_increase: Option<bool> = None;
-
-            // for number in line_collection.iter() {
-            //     match p_previous_number {
-            //         Some(previous_number) => {
-            //             let difference = previous_number - number;
-
-            //             if !(0 < difference.abs() && difference.abs() < 4) {
-            //                 is_valid_line_collection = false;
-            //                 break;
-            //             }
-
-            //             p_previous_number = Some(number);
-
-            //             match p_increase {
-            //                 Some(increase) => {
-            //                     if increase != difference.is_positive() {
-            //                         is_valid_line_collection = false;
-            //                         break;
-            //                     }
-            //                 }
-            //                 None => p_increase = Some(difference.is_positive()),
-            //             }
-            //         }
-            //         None => p_previous_number = Some(number),
-            //     }
-            // }
-
-            // if is_valid_line_collection {
-            //     result += 1;
-            // }
         }
 
         result.to_string()
@@ -105,7 +69,7 @@ impl _2 {
 
         let mut i_line_collection = line_collection.clone();
         if let Some(invalid_number_index) = p_invalid_number_index {
-            if invalid_number_index as i8 - allowed_mut_count as i8 >= 0 {
+            if !((invalid_number_index as i8 - allowed_mut_count as i8) < 0) {
                 i_line_collection.remove(invalid_number_index - allowed_mut_count);
             }
         }

src/main.rs

@@ -1,5 +1,3 @@
-// TODO: Review day 1 and 2 code
-
 use std::process::ExitCode;
 
 mod days;

src/utils/env.rs

@@ -1,7 +1,7 @@
 use std::{env, fs};
 
 pub fn load_session_token() -> Result<(), u8> {
-    println!("Loading all environment variables from the dotenv file");
+    println!("Loading all environment variables from the dotenv file...");
 
     let p_dotenv_vec = fs::read(".env");
 

src/utils/http.rs

@@ -1,20 +1,11 @@
 // TODO: Replace the Curl subprocess with a basic HTTP/2 request using std::io::net
-// TODO: Cache the response
 // TODO: Prevent early fetching
-// TODO: Improve input parsing
+// TODO: Cache the response
 
 use std::{env, process::Command};
 
-pub fn request(day: u8, part: Option<u8>) -> Result<String, ()> {
-    match part {
-        Some(part_number) => {
-            println!(
-                "Requesting the input of day {}, part {}",
-                &day, &part_number
-            )
-        }
-        None => println!("Requesting the input of day {}", &day),
-    }
+pub fn request(day: u8) -> Result<String, ()> {
+    println!("Requesting the input of day {}...", &day);
 
     let p_request = Command::new("/usr/bin/curl")
         .arg("-H")