refactor: don't pass too many args to a function

Author: azzamsa

src/main.rs

@@ -1,5 +1,5 @@
 use clap::Parser;
-use std::process;
+use std::{process, sync::Arc};
 
 use anyhow::Result;
 use zman::{
@@ -16,9 +16,9 @@ fn main() {
 }
 
 fn run() -> Result<()> {
-    let opts = Opts::parse();
+    let opts = Arc::new(Opts::parse());
 
-    let mut printer = Printer::new(opts.width, &opts.full_bar, &opts.rest_bar, opts.json);
+    let mut printer = Printer::new(Arc::clone(&opts));
     match opts.period {
         Period::Year => {
             let ratio = progress::year()?;

src/output.rs

@@ -1,25 +1,24 @@
-use std::io::{self, Write};
+use std::{
+    io::{self, Write},
+    sync::Arc,
+};
 
 use owo_colors::OwoColorize;
 
+use crate::cli::Opts;
+
 pub struct Printer {
+    opts: Arc<Opts>,
     ratio: f64,
-    width: i32,
-    full_bar: String,
-    rest_bar: String,
-    json_format: bool,
     /// only used in status bar
     ratio_char: String,
 }
 
 impl Printer {
-    pub fn new(width: i32, full_bar: &str, rest_bar: &str, json_format: bool) -> Self {
+    pub fn new(opts: Arc<Opts>) -> Self {
         Self {
+            opts,
             ratio: 0.0,
-            width,
-            full_bar: full_bar.to_string(),
-            rest_bar: rest_bar.to_string(),
-            json_format,
             ratio_char: "y".to_string(),
         }
     }
@@ -34,13 +33,13 @@ impl Printer {
     /// Show progress-bar
     pub fn print(&self) {
         let ratio_int = (self.ratio * 100.0) as i32;
-        let progress_int = (self.ratio * f64::from(self.width)).round() as i32;
-        let rest_int = self.width - progress_int;
+        let progress_int = (self.ratio * f64::from(self.opts.width)).round() as i32;
+        let rest_int = self.opts.width - progress_int;
 
         let mut progress_fmt = format!(
             "{}{} {}%",
-            self.full_bar.repeat(progress_int as usize),
-            self.rest_bar.repeat(rest_int as usize),
+            self.opts.full_bar.repeat(progress_int as usize),
+            self.opts.rest_bar.repeat(rest_int as usize),
             ratio_int
         );
         let state = {
@@ -51,11 +50,11 @@ impl Printer {
             }
         };
         // color
-        if state == "Critical" && !self.json_format {
+        if state == "Critical" && !self.opts.json {
             progress_fmt = format!("{}", progress_fmt.red());
         }
         // JSON
-        if self.json_format {
+        if self.opts.json {
             progress_fmt = format!(
                 r#"{{"icon": "{}", "state": "{}", "text": "{}: {}"}}"#,
                 "zman", state, self.ratio_char, progress_fmt