X-Git-Url: http://git.scottworley.com/tablify/blobdiff_plain/58b5f36de045c760efa51b21a6d841f5b62558db..7067975b593ef6a8b639f61dda710b023ec26a25:/src/lib.rs diff --git a/src/lib.rs b/src/lib.rs index fcf5dc8..a3878c5 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -1,14 +1,69 @@ -#[cfg(test)] use std::collections::{HashMap, HashSet}; -#[cfg(test)] +use std::fmt::Write; use std::io::BufRead; -#[cfg(test)] use std::iter::Iterator; +const HEADER: &str = " + + + + + + + + + + +"; +const FOOTER: &str = " +
+ +"; + +#[derive(Debug, PartialEq, Eq, Hash)] +struct Entry { + col: String, + instance: Option, +} +impl From<&str> for Entry { + fn from(value: &str) -> Entry { + match value.split_once(':') { + None => Entry { + col: String::from(value), + instance: None, + }, + Some((col, instance)) => Entry { + col: String::from(col.trim()), + instance: Some(String::from(instance.trim())), + }, + } + } +} + #[derive(Debug, PartialEq, Eq)] struct RowInput { label: String, - entries: Vec, + entries: Vec, } struct Reader>> { @@ -16,7 +71,6 @@ struct Reader>> { row: Option, } impl>> Reader { - #[cfg(test)] fn new(input: Input) -> Self { Self { input: input.enumerate(), @@ -46,7 +100,7 @@ impl>> Iterator for Reader n + 1 )))) } - Some(ref mut row) => row.entries.push(String::from(line.trim())), + Some(ref mut row) => row.entries.push(Entry::from(line.trim())), }, Some((_, Ok(line))) => { let prev = std::mem::take(&mut self.row); @@ -63,18 +117,22 @@ impl>> Iterator for Reader } } -#[cfg(test)] fn read_rows(input: impl std::io::Read) -> impl Iterator> { Reader::new(std::io::BufReader::new(input).lines()) } -#[cfg(test)] fn column_counts(rows: &[RowInput]) -> Vec<(usize, String)> { let mut counts: Vec<_> = rows .iter() - .flat_map(|r| r.entries.iter().collect::>().into_iter()) - .fold(HashMap::new(), |mut cs, e| { - cs.entry(String::from(e)) + .flat_map(|r| { + r.entries + .iter() + .map(|e| &e.col) + .collect::>() + .into_iter() + }) + .fold(HashMap::new(), |mut cs, col| { + cs.entry(String::from(col)) .and_modify(|n| *n += 1) .or_insert(1); cs @@ -85,15 +143,108 @@ fn column_counts(rows: &[RowInput]) -> Vec<(usize, String)> { counts.sort(); counts } +fn column_order(rows: &[RowInput]) -> Vec { + column_counts(rows) + .into_iter() + .map(|(_, col)| col) + .collect() +} + +fn render_instance(entry: &Entry) -> String { + match &entry.instance { + None => String::from("✓ "), + Some(instance) => String::from(instance) + " ", + } +} + +fn render_cell(col: &str, row: &RowInput) -> String { + // TODO: Escape HTML special characters + let entries: Vec<&Entry> = row.entries.iter().filter(|e| e.col == col).collect(); + let class = if entries.is_empty() { "" } else { "yes" }; + let all_empty = entries.iter().all(|e| e.instance.is_none()); + let contents = if entries.is_empty() || (all_empty && entries.len() == 1) { + String::new() + } else if all_empty { + format!("{}", entries.len()) + } else { + entries + .iter() + .map(|i| render_instance(i)) + .collect::() + }; + format!("{}", contents.trim()) +} -pub fn tablify(_input: &impl std::io::Read) -> String { - String::from("Hello, world!") +fn render_row(columns: &[String], row: &RowInput) -> String { + // This is O(n^2) & doesn't need to be + // TODO: Escape HTML special characters + format!( + "{}{}\n", + row.label, + &columns + .iter() + .map(|col| render_cell(col, row)) + .collect::() + ) +} + +fn render_column_headers(columns: &[String]) -> String { + // TODO: Escape HTML special characters + String::from("") + + &columns.iter().fold(String::new(), |mut acc, c| { + write!(&mut acc, "{c}").unwrap(); + acc + }) + + "\n" +} + +/// # Errors +/// +/// Will return `Err` if +/// * there's an i/o error while reading `input` +/// * the log has invalid syntax: +/// * an indented line with no preceding non-indented line +pub fn tablify(input: impl std::io::Read) -> Result { + let rows = read_rows(input).collect::, _>>()?; + let columns = column_order(&rows); + Ok(String::from(HEADER) + + &render_column_headers(&columns) + + &rows + .into_iter() + .map(|r| render_row(&columns, &r)) + .collect::() + + FOOTER) } #[cfg(test)] mod tests { use super::*; + #[test] + fn test_parse_entry() { + assert_eq!( + Entry::from("foo"), + Entry { + col: String::from("foo"), + instance: None + } + ); + assert_eq!( + Entry::from("foo:bar"), + Entry { + col: String::from("foo"), + instance: Some(String::from("bar")) + } + ); + assert_eq!( + Entry::from("foo: bar"), + Entry { + col: String::from("foo"), + instance: Some(String::from("bar")) + } + ); + } + #[test] fn test_read_rows() { assert_eq!( @@ -127,7 +278,7 @@ mod tests { read_rows(&b"foo\n bar\n"[..]).flatten().collect::>(), vec![RowInput { label: String::from("foo"), - entries: vec![String::from("bar")] + entries: vec![Entry::from("bar")] }] ); assert_eq!( @@ -136,7 +287,7 @@ mod tests { .collect::>(), vec![RowInput { label: String::from("foo"), - entries: vec![String::from("bar"), String::from("baz")] + entries: vec![Entry::from("bar"), Entry::from("baz")] }] ); assert_eq!( @@ -175,7 +326,7 @@ mod tests { .collect::>(), vec![RowInput { label: String::from("foo"), - entries: vec![String::from("bar")] + entries: vec![Entry::from("bar")] }] ); @@ -214,5 +365,77 @@ mod tests { ), vec![(1, String::from("bar")), (2, String::from("baz"))] ); + assert_eq!( + column_counts( + &read_rows(&b"foo\n bar: 1\n bar: 2\n baz\n bar\nquux\n baz"[..]) + .collect::, _>>() + .unwrap() + ), + vec![(1, String::from("bar")), (2, String::from("baz"))] + ); + } + + #[test] + fn test_render_cell() { + assert_eq!( + render_cell( + "foo", + &RowInput { + label: String::from("nope"), + entries: vec![] + } + ), + String::from("") + ); + assert_eq!( + render_cell( + "foo", + &RowInput { + label: String::from("nope"), + entries: vec![Entry::from("bar")] + } + ), + String::from("") + ); + assert_eq!( + render_cell( + "foo", + &RowInput { + label: String::from("nope"), + entries: vec![Entry::from("foo")] + } + ), + String::from("") + ); + assert_eq!( + render_cell( + "foo", + &RowInput { + label: String::from("nope"), + entries: vec![Entry::from("foo"), Entry::from("foo")] + } + ), + String::from("2") + ); + assert_eq!( + render_cell( + "foo", + &RowInput { + label: String::from("nope"), + entries: vec![Entry::from("foo: 5"), Entry::from("foo: 10")] + } + ), + String::from("5 10") + ); + assert_eq!( + render_cell( + "foo", + &RowInput { + label: String::from("nope"), + entries: vec![Entry::from("foo: 5"), Entry::from("foo")] + } + ), + String::from("5 ✓") + ); } }