From 66670c0ed38a4a621c192edacb61805e8a0044ea Mon Sep 17 00:00:00 2001
From: lekan <lekan.lekan@gmail.com>
Date: Thu, 11 Aug 2022 22:09:24 +0300
Subject: [PATCH 1/2] add error handling

---
 src/main.rs | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index c34444c..8f81fb7 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -1,10 +1,13 @@
-use std::{env, fs};
+use std::{env, fs, process};
 
 
 fn main() {
     let args: Vec<String> = env::args().collect();
 
-    let config = Config::new(&args);
+    let config = Config::new(&args).unwrap_or_else(|err| {
+        println!("Problem parsing arguments: {}", err);
+        process::exit(1);
+    });
     
     println!("Searching for {}", config.query);
     println!("In file {}", config.filename);
@@ -21,13 +24,13 @@ struct Config {
 }
 
 impl Config {
-    fn new(args: &[String]) -> Config {
+    fn new(args: &[String]) -> Result<Config, &'static str> {
         if args.len() < 3 {
-            panic!("not anough arguments");
+            return Err("not enough arguments");
         }
         let query = args[1].clone();
         let filename = args[2].clone();
 
-        Config {query, filename}
+        Ok(Config{query, filename})
     }
 }

From e8414833dae6caeca4d09df3f640d34c19f73d55 Mon Sep 17 00:00:00 2001
From: lekan <lekan.lekan@gmail.com>
Date: Sun, 14 Aug 2022 21:14:08 +0300
Subject: [PATCH 2/2] refactoring 2

---
 output.txt  |   2 +
 src/lib.rs  | 114 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 src/main.rs |  33 ++++-----------
 3 files changed, 123 insertions(+), 26 deletions(-)
 create mode 100644 output.txt
 create mode 100644 src/lib.rs

diff --git a/output.txt b/output.txt
new file mode 100644
index 0000000..d40e5a2
--- /dev/null
+++ b/output.txt
@@ -0,0 +1,2 @@
+Are you nobody, too?
+How dreary to be somebody!
diff --git a/src/lib.rs b/src/lib.rs
new file mode 100644
index 0000000..f23dccc
--- /dev/null
+++ b/src/lib.rs
@@ -0,0 +1,114 @@
+use std::error::Error;
+use std::fs;
+use std::env;
+
+pub struct Config {
+    pub query: String,
+    pub filename: String,
+    pub ignore_case: bool,
+}
+
+impl Config {
+    pub fn build(
+        mut args: impl Iterator<Item = String>,
+    ) -> Result<Config, &'static str> {
+        args.next();
+
+        let query = match args.next() {
+            Some(arg) => arg,
+            None => return Err("Didn't get a query string"),
+        };
+
+        let filename = match args.next() {
+            Some(arg) => arg,
+            None => return Err("Didn't get a file path"),
+        };
+
+        let ignore_case = env::var("IGNORE_CASE").is_ok();
+
+        Ok(Config {
+            query,
+            filename,
+            ignore_case,
+        })
+    }
+
+    pub fn new(args: &[String]) -> Result<Config, &'static str> {
+        if args.len() < 3 {
+            return Err("not enough arguments");
+        }
+        let query = args[1].clone();
+        let filename = args[2].clone();
+
+        let ignore_case = env::var("IGNORE_CASE").is_ok();
+
+        Ok(Config {
+            query,
+            filename,
+            ignore_case,
+        })
+    }
+}
+
+pub fn run(config: Config) -> Result<(), Box<dyn Error>> {
+    let contents = fs::read_to_string(config.filename)?;
+
+    let results = if config.ignore_case {
+        search_case_insensitive(&config.query, &contents)
+    } else {
+        search(&config.query, &contents)
+    };
+
+    for line in results {
+        println!("{line}");
+    }
+
+    Ok(())
+}
+
+pub fn search<'a>(query: &str, contents: &'a str) -> Vec<&'a str> {
+    contents
+        .lines()
+        .filter(|line| line.contains(query))
+        .collect()
+}
+
+pub fn search_case_insensitive<'a>(
+    query: &str,
+    contents: &'a str,
+) -> Vec<&'a str> {
+    contents
+        .lines()
+        .filter(|line| line.contains(&query.to_lowercase()))
+        .collect()
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn case_sensitive() {
+        let query = "duct";
+        let contents = "\
+Rust:
+safe, fast, productive.
+Pick three.
+Duct tape.";
+        assert_eq!(vec!["safe, fast, productive."], search(query, contents));
+    }
+
+    #[test]
+    fn case_insensitive() {
+        let query = "rUsT";
+        let contents = "\
+Rust:
+safe, fast, productive.
+Pick three.
+Trust me.";
+        assert_eq!(
+            vec!["Rust:", "Trust me."],
+            search_case_insensitive(query, contents)
+        );
+    }
+}
diff --git a/src/main.rs b/src/main.rs
index 8f81fb7..3a44bee 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -1,36 +1,17 @@
-use std::{env, fs, process};
+use std::{env, process};
 
+use minigrep::Config;
 
 fn main() {
     let args: Vec<String> = env::args().collect();
 
-    let config = Config::new(&args).unwrap_or_else(|err| {
-        println!("Problem parsing arguments: {}", err);
+    let config = Config::build(env::args()).unwrap_or_else(|err| {
+        eprintln!("Problem parsing arguments: {err}");
         process::exit(1);
     });
-    
-    println!("Searching for {}", config.query);
-    println!("In file {}", config.filename);
 
-    let contents = fs::read_to_string(config.filename)
-        .expect("Something went wrong reading the file");
-    
-    println!("With text:\n{}", contents);
-}
-
-struct Config {
-    query: String,
-    filename: String,
-}
-
-impl Config {
-    fn new(args: &[String]) -> Result<Config, &'static str> {
-        if args.len() < 3 {
-            return Err("not enough arguments");
-        }
-        let query = args[1].clone();
-        let filename = args[2].clone();
-
-        Ok(Config{query, filename})
+    if let Err(e) = minigrep::run(config) {
+        eprintln!("Application error: {e}");
+        process::exit(1);
     }
 }