X-Git-Url: http://git.scottworley.com/tablify/blobdiff_plain/a638dfe78611c069402096f0c702c3f51648962c..b59ad4335c434a858a991137445fd34aaa4c6b21:/src/lib.rs?ds=sidebyside diff --git a/src/lib.rs b/src/lib.rs index b4b8bbc..d85de02 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -4,11 +4,18 @@ use std::fmt::Write; use std::io::BufRead; use std::iter::Iterator; +fn tally_marks(n: usize) -> String { + let fives = { 0..n / 5 }.map(|_| '𝍸'); + let ones = { 0..n % 5 }.map(|_| '𝍷'); + fives.chain(ones).collect() +} + #[derive(PartialEq, Eq, Debug)] struct Config { column_threshold: usize, static_columns: Vec>, hidden_columns: HashSet, + substitute_labels: HashMap, } impl Config { fn apply_command(&mut self, line_num: usize, cmd: &str) -> Result<(), std::io::Error> { @@ -25,6 +32,18 @@ impl Config { self.static_columns.push(Some(col.to_owned())); } else if cmd == "colsep" { self.static_columns.push(None); + } else if let Some(directive) = cmd.strip_prefix("label ") { + match directive.split_once(':') { + None => { + return Err(std::io::Error::new( + std::io::ErrorKind::InvalidInput, + format!("line {line_num}: Annotation missing ':'"), + )) + } + Some((col, label)) => self + .substitute_labels + .insert(col.to_owned(), label.to_owned()), + }; } else { return Err(std::io::Error::new( std::io::ErrorKind::InvalidInput, @@ -40,6 +59,7 @@ impl Default for Config { column_threshold: 2, static_columns: vec![], hidden_columns: HashSet::new(), + substitute_labels: HashMap::new(), } } } @@ -265,52 +285,60 @@ fn column_order(config: &Config, rows: &[Rowlike]) -> Vec { .collect() } -fn render_one_instance(instance: &Option) -> HTML { - match instance { - None => HTML::from("✓"), - Some(instance) => HTML::escape(instance.as_ref()), - } -} - fn render_instances(instances: &[Option]) -> HTML { - let all_empty = instances.iter().all(Option::is_none); - if all_empty && instances.len() == 1 { - HTML::from("") - } else if all_empty { - HTML(format!("{}", instances.len())) - } else { - HTML( - instances - .iter() - .map(render_one_instance) - .map(|html| html.0) // Waiting for slice_concat_trait to stabilize - .collect::>() - .join(" "), - ) + let mut tally = 0; + let mut out = vec![]; + for ins in instances { + match ins { + None => tally += 1, + Some(content) => { + if tally > 0 { + out.push(HTML(tally_marks(tally))); + tally = 0; + } + out.push(HTML::escape(content)); + } + } + } + if tally > 0 { + out.push(HTML(tally_marks(tally))); } + HTML( + out.into_iter() + .map(|html| html.0) // Waiting for slice_concat_trait to stabilize + .collect::>() + .join(" "), + ) } fn render_cell(col: &str, row: &mut Row) -> HTML { let row_label = HTML::escape(row.label.as_ref()); let col_label = HTML::escape(col); let instances: Option<&Vec>> = row.entries.get(col); - let class = HTML::from(if instances.is_none() { "" } else { "yes" }); + let is_empty = match instances { + None => true, + Some(is) => is.iter().all(|ins| match ins { + None => false, + Some(content) => content == "×", + }), + }; + let class = HTML::from(if is_empty { "" } else { r#" class="yes""# }); let contents = match instances { None => HTML::from(""), Some(is) => render_instances(is), }; row.entries.remove(col); HTML(format!( - r#"{contents}"# + r#"{contents}"# )) } fn render_leftover(notcol: &str, instances: &[Option]) -> HTML { let label = HTML::escape(notcol); - let rest = render_instances(instances); - if rest == HTML::from("") { + if instances.len() == 1 && instances[0].is_none() { HTML(format!("{label}")) } else { + let rest = render_instances(instances); HTML(format!("{label}: {rest}")) } } @@ -359,21 +387,27 @@ fn render_row(config: &Config, columns: &[String], rowlike: &mut Rowlike) -> HTM } } -fn column_header_order<'a>( +fn column_header_labels<'a>( config: &'a Config, columns: &'a [String], ) -> impl Iterator> { let static_columns = config.static_columns.iter().map(|oc| oc.as_ref()); let dynamic_columns = columns.iter().map(Some); - static_columns.chain(dynamic_columns).filter(|ocol| { - ocol.map_or(true, |col| !config.hidden_columns.contains(col)) - }) + static_columns + .chain(dynamic_columns) + .filter(|ocol| ocol.map_or(true, |col| !config.hidden_columns.contains(col))) + .map(|ocol| { + ocol.map(|col| match config.substitute_labels.get(col) { + None => col, + Some(substitute) => substitute, + }) + }) } fn render_column_headers(config: &Config, columns: &[String]) -> HTML { HTML( String::from(r#""#) - + &column_header_order(config, columns).fold(String::new(), |mut acc, ocol| { + + &column_header_labels(config, columns).fold(String::new(), |mut acc, ocol| { match ocol { Some(col) => { let col_header = HTML::escape(col); @@ -442,6 +476,21 @@ mod tests { ); } + #[test] + fn test_tally_marks() { + assert_eq!(tally_marks(1), "𝍷"); + assert_eq!(tally_marks(2), "𝍷𝍷"); + assert_eq!(tally_marks(3), "𝍷𝍷𝍷"); + assert_eq!(tally_marks(4), "𝍷𝍷𝍷𝍷"); + assert_eq!(tally_marks(5), "𝍸"); + assert_eq!(tally_marks(6), "𝍸𝍷"); + assert_eq!(tally_marks(7), "𝍸𝍷𝍷"); + assert_eq!(tally_marks(8), "𝍸𝍷𝍷𝍷"); + assert_eq!(tally_marks(9), "𝍸𝍷𝍷𝍷𝍷"); + assert_eq!(tally_marks(10), "𝍸𝍸"); + assert_eq!(tally_marks(11), "𝍸𝍸𝍷"); + } + fn read_rows(input: impl std::io::Read) -> Result, std::io::Error> { read_input(input).map(|(rows, _)| rows) } @@ -563,6 +612,12 @@ mod tests { read_config(&b"!col foo"[..]).unwrap().static_columns, vec![Some("foo".to_owned())] ); + assert_eq!( + read_config(&b"!label foo:bar"[..]) + .unwrap() + .substitute_labels["foo"], + "bar" + ); let bad_command = read_config(&b"!no such command"[..]); assert!(bad_command.is_err()); @@ -571,6 +626,10 @@ mod tests { let bad_num = read_config(&b"!col_threshold foo"[..]); assert!(bad_num.is_err()); assert!(format!("{bad_num:?}").contains("line 1: col_threshold must be numeric")); + + let bad_sub = read_config(&b"!label foo"[..]); + assert!(bad_sub.is_err()); + assert!(format!("{bad_sub:?}").contains("line 1: Annotation missing ':'")); } #[test] @@ -596,27 +655,35 @@ mod tests { } #[test] - fn test_column_header_order() { + fn test_column_header_labels() { let mut cfg = Config::default(); - assert!(column_header_order(&cfg, &["foo".to_owned()]).eq([Some(&"foo".to_owned())])); + assert!(column_header_labels(&cfg, &["foo".to_owned()]).eq([Some(&"foo".to_owned())])); cfg.static_columns.push(Some("bar".to_owned())); - assert!(column_header_order(&cfg, &["foo".to_owned()]) + assert!(column_header_labels(&cfg, &["foo".to_owned()]) .eq([Some(&"bar".to_owned()), Some(&"foo".to_owned())])); cfg.static_columns.push(None); - assert!(column_header_order(&cfg, &["foo".to_owned()]).eq([ + assert!(column_header_labels(&cfg, &["foo".to_owned()]).eq([ Some(&"bar".to_owned()), None, Some(&"foo".to_owned()) ])); + cfg.substitute_labels + .insert("foo".to_owned(), "foo (bits)".to_owned()); + assert!(column_header_labels(&cfg, &["foo".to_owned()]).eq([ + Some(&"bar".to_owned()), + None, + Some(&"foo (bits)".to_owned()) + ])); + cfg.hidden_columns.insert("foo".to_owned()); - assert!(column_header_order(&cfg, &["foo".to_owned()]).eq([Some(&"bar".to_owned()), None])); + assert!(column_header_labels(&cfg, &["foo".to_owned()]).eq([Some(&"bar".to_owned()), None])); cfg.hidden_columns.insert("bar".to_owned()); - assert!(column_header_order(&cfg, &["foo".to_owned()]).eq([None])); + assert!(column_header_labels(&cfg, &["foo".to_owned()]).eq([None])); } #[test] @@ -630,7 +697,7 @@ mod tests { } ), HTML::from( - r#""# + r#""# ) ); assert_eq!( @@ -642,7 +709,7 @@ mod tests { } ), HTML::from( - r#""# + r#""# ) ); assert_eq!( @@ -654,7 +721,7 @@ mod tests { } ), HTML::from( - r#""# + r#"𝍷"# ) ); assert_eq!( @@ -666,7 +733,7 @@ mod tests { } ), HTML::from( - r#"2"# + r#"𝍷𝍷"# ) ); assert_eq!( @@ -693,7 +760,19 @@ mod tests { } ), HTML::from( - r#"5 ✓"# + r#"5 𝍷"# + ) + ); + assert_eq!( + render_cell( + "foo", + &mut Row { + label: "nope".to_owned(), + entries: HashMap::from([("foo".to_owned(), vec![Some("×".to_owned())])]), + } + ), + HTML::from( + r#"×"# ) ); assert_eq!( @@ -717,7 +796,7 @@ mod tests { } ), HTML::from( - r#""# + r#"𝍷"# ) ); let mut r = Row { @@ -772,7 +851,7 @@ mod tests { ]), } ), - HTML::from("bar: 2, foo") + HTML::from("bar: 𝍷𝍷, foo") ); assert_eq!( render_all_leftovers( @@ -780,6 +859,7 @@ mod tests { column_threshold: 2, static_columns: vec![], hidden_columns: HashSet::from(["private".to_owned()]), + substitute_labels: HashMap::new(), }, &Row { label: "nope".to_owned(), @@ -802,7 +882,7 @@ mod tests { }) ), HTML::from( - r#"nopebar + r#"nopebar "# ) ); @@ -812,6 +892,7 @@ mod tests { column_threshold: 0, static_columns: vec![Some("foo".to_owned()), Some("bar".to_owned())], hidden_columns: HashSet::new(), + substitute_labels: HashMap::new(), }, &["baz".to_owned()], &mut Rowlike::Row(Row { @@ -834,6 +915,7 @@ mod tests { column_threshold: 0, static_columns: vec![Some("foo".to_owned()), None, Some("bar".to_owned())], hidden_columns: HashSet::new(), + substitute_labels: HashMap::new(), }, &[], &mut Rowlike::Row(Row { @@ -855,6 +937,7 @@ mod tests { column_threshold: 0, static_columns: vec![], hidden_columns: HashSet::from(["foo".to_owned()]), + substitute_labels: HashMap::new(), }, &[], &mut Rowlike::Row(Row { @@ -873,6 +956,7 @@ mod tests { column_threshold: 0, static_columns: vec![Some("foo".to_owned())], hidden_columns: HashSet::from(["foo".to_owned()]), + substitute_labels: HashMap::new(), }, &[], &mut Rowlike::Row(Row {