refactor: save data size as number

Author: c-git

src/app.rs

@@ -725,7 +725,7 @@ impl LogViewerApp {
                         (false, _, total_len) => as_string_with_separators(total_len),
                     };
                 ui.label(format!("# Rows: {row_count_text}"));
-                ui.label(format!("Size: {}", data.file_size));
+                ui.label(format!("Size: {}", data.file_size_display()));
             }
         });
     }

src/app/data.rs

@@ -25,7 +25,7 @@ pub struct Data {
     rows: Vec<LogRow>,
     filtered_rows: Option<Vec<usize>>,
     applied_filter: Option<FilterConfig>,
-    pub file_size: String,
+    pub file_size_as_bytes: usize,
 }
 
 #[derive(serde::Deserialize, serde::Serialize, Default, Debug, PartialEq, Eq, Clone)]
@@ -136,6 +136,10 @@ impl LogRow {
 }
 
 impl Data {
+    pub fn file_size_display(&self) -> String {
+        SizeUnits::Auto.convert_trimmed(self.file_size_as_bytes)
+    }
+
     pub fn rows_iter(&self) -> impl Iterator<Item = &'_ LogRow> {
         DataIter::new(self)
     }
@@ -415,7 +419,7 @@ impl TryFrom<(&DataDisplayOptions, usize, &str)> for LogRow {
         if let Some(config) = data_display_options.row_size_config.as_ref() {
             result.or_insert(
                 config.field_name.clone(),
-                config.units.convert(row_size_in_bytes),
+                config.units.convert(row_size_in_bytes).into(),
             );
         }
         Ok(result)
@@ -465,15 +469,8 @@ impl TryFrom<(&DataDisplayOptions, &str)> for Data {
     fn try_from(
         (data_display_options, value): (&DataDisplayOptions, &str),
     ) -> Result<Self, Self::Error> {
-        let file_size = SizeUnits::Auto.convert(value.len());
-        let file_size = file_size
-            .as_str()
-            .map(|x| x.to_string())
-            .unwrap_or_else(|| file_size.to_string())
-            .trim_matches('0')
-            .to_string();
         let mut result = Data {
-            file_size,
+            file_size_as_bytes: value.len(),
             ..Default::default()
         };
         for (i, line) in value.lines().enumerate() {

src/app/data_display_options.rs

@@ -116,12 +116,17 @@ impl SizeUnits {
         }
     }
 
-    /// Output as a string because includes the unit
-    pub(crate) fn convert(&self, row_size_in_bytes: usize) -> serde_json::Value {
+    pub(crate) fn convert(&self, row_size_in_bytes: usize) -> String {
         let concrete_unit = self.to_concrete(row_size_in_bytes);
         let scalar = concrete_unit.scalar();
         let result = row_size_in_bytes as f64 / scalar;
-        format!("{result:0>9.4} {concrete_unit}").into()
+        format!("{result:0>9.4} {concrete_unit}")
+    }
+
+    pub fn convert_trimmed(&self, row_size_in_bytes: usize) -> String {
+        self.convert(row_size_in_bytes)
+            .trim_matches('0')
+            .to_string()
     }
 
     pub fn as_str(&self) -> &'static str {