X-Git-Url: http://git.scottworley.com/tablify/blobdiff_plain/f272e502c4d1e2e33ab1e9fe460df69ff8f8ccd2..116fc215a7f859e20b60e91bcb24d314a9d911f9:/src/lib.rs?ds=sidebyside
diff --git a/src/lib.rs b/src/lib.rs
index e90f66b..199b8ab 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -1,22 +1,111 @@
-#[cfg(test)]
-use std::collections::HashMap;
-#[cfg(test)]
+use std::collections::{HashMap, HashSet};
+use std::fmt::Write;
use std::io::BufRead;
-#[cfg(test)]
use std::iter::Iterator;
+const HEADER: &str = "
+
+
+
+
+
+
+
+
+
+
+";
+const FOOTER: &str = "
+
+
+";
+
+#[derive(PartialEq, Eq, Debug)]
+pub struct HTML(String);
+impl HTML {
+ fn escape(value: &str) -> HTML {
+ let mut escaped: String = String::new();
+ for c in value.chars() {
+ match c {
+ '>' => escaped.push_str(">"),
+ '<' => escaped.push_str("<"),
+ '\'' => escaped.push_str("'"),
+ '"' => escaped.push_str("""),
+ '&' => escaped.push_str("&"),
+ ok_c => escaped.push(ok_c),
+ }
+ }
+ HTML(escaped)
+ }
+}
+impl From<&str> for HTML {
+ fn from(value: &str) -> HTML {
+ HTML(String::from(value))
+ }
+}
+impl FromIterator for HTML {
+ fn from_iter(iter: T) -> HTML
+ where
+ T: IntoIterator- ,
+ {
+ HTML(iter.into_iter().map(|html| html.0).collect::
())
+ }
+}
+impl std::fmt::Display for HTML {
+ fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+ write!(f, "{}", self.0)
+ }
+}
+
+#[derive(Debug, PartialEq, Eq, Hash)]
+struct Entry<'a> {
+ col: &'a str,
+ instance: Option<&'a str>,
+}
+impl<'a> From<&'a str> for Entry<'a> {
+ fn from(value: &'a str) -> Entry<'a> {
+ match value.split_once(':') {
+ None => Entry {
+ col: value,
+ instance: None,
+ },
+ Some((col, instance)) => Entry {
+ col: col.trim(),
+ instance: Some(instance.trim()),
+ },
+ }
+ }
+}
+
#[derive(Debug, PartialEq, Eq)]
-struct RowInput {
- label: String,
- entries: Vec,
+struct RowInput<'a> {
+ label: &'a str,
+ entries: Vec>,
}
-struct Reader>> {
+struct Reader<'a, Input: Iterator- >> {
input: std::iter::Enumerate
,
- row: Option,
+ row: Option>,
}
-impl>> Reader {
- #[cfg(test)]
+impl<'a, Input: Iterator- >> Reader<'a, Input> {
fn new(input: Input) -> Self {
Self {
input: input.enumerate(),
@@ -24,14 +113,15 @@ impl
>> Reader {
}
}
}
-impl>> Iterator for Reader {
- type Item = Result;
+impl<'a, Input: Iterator- >> Iterator for Reader<'a, Input> {
+ type Item = Result
, std::io::Error>;
fn next(&mut self) -> Option {
loop {
match self
.input
.next()
- .map(|(n, r)| (n, r.map(|line| String::from(line.trim_end()))))
+ // TODO: Don't leak
+ .map(|(n, r)| (n, r.map(|line| String::from(line).leak().trim_end())))
{
None => return Ok(std::mem::take(&mut self.row)).transpose(),
Some((_, Err(e))) => return Some(Err(e)),
@@ -46,7 +136,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,45 +153,159 @@ impl>> Iterator for Reader
}
}
-#[cfg(test)]
-fn read_rows(input: impl std::io::Read) -> impl Iterator- > {
+fn read_rows(
+ input: impl std::io::Read,
+) -> impl Iterator
- , std::io::Error>> {
Reader::new(std::io::BufReader::new(input).lines())
}
-#[cfg(test)]
-fn column_counts(rows: &[RowInput]) -> HashMap
{
- rows.iter()
- .flat_map(|r| r.entries.iter())
- .fold(HashMap::new(), |mut counts, e| {
- counts
- .entry(String::from(e))
- .and_modify(|c| *c += 1)
+fn column_counts(rows: &[RowInput]) -> Vec<(usize, String)> {
+ let mut counts: Vec<_> = rows
+ .iter()
+ .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);
- counts
+ cs
})
+ .into_iter()
+ .map(|(col, n)| (n, col))
+ .collect();
+ counts.sort_unstable_by(|(an, acol), (bn, bcol)| bn.cmp(an).then(acol.cmp(bcol)));
+ counts
+}
+fn column_order(rows: &[RowInput]) -> Vec {
+ column_counts(rows)
+ .into_iter()
+ .map(|(_, col)| col)
+ .collect()
}
-pub fn tablify(_input: &impl std::io::Read) -> String {
- String::from("Hello, world!")
+fn render_instance(entry: &Entry) -> HTML {
+ match &entry.instance {
+ None => HTML::from("â"),
+ Some(instance) => HTML::escape(instance.as_ref()),
+ }
+}
+
+fn render_cell(col: &str, row: &RowInput) -> HTML {
+ let row_label = HTML::escape(row.label.as_ref());
+ let col_label = HTML::escape(col);
+ let entries: Vec<&Entry> = row.entries.iter().filter(|e| e.col == col).collect();
+ let class = HTML::from(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) {
+ HTML::from("")
+ } else if all_empty {
+ HTML(format!("{}", entries.len()))
+ } else {
+ HTML(
+ entries
+ .iter()
+ .map(|i| render_instance(i))
+ .map(|html| html.0) // Waiting for slice_concat_trait to stabilize
+ .collect::>()
+ .join(" "),
+ )
+ };
+ HTML(format!("{contents} "))
+}
+
+fn render_row(columns: &[String], row: &RowInput) -> HTML {
+ // This is O(n^2) & doesn't need to be
+ let row_label = HTML::escape(row.label.as_ref());
+ HTML(format!(
+ "{row_label} {} \n",
+ &columns
+ .iter()
+ .map(|col| render_cell(col, row))
+ .collect::()
+ ))
+}
+
+fn render_column_headers(columns: &[String]) -> HTML {
+ HTML(
+ String::from(" ")
+ + &columns.iter().fold(String::new(), |mut acc, col| {
+ let col_header = HTML::escape(col.as_ref());
+ write!(
+ &mut acc,
+ ""
+ )
+ .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(HTML(format!(
+ "{HEADER}{}{}{FOOTER}",
+ render_column_headers(&columns),
+ rows.into_iter()
+ .map(|r| render_row(&columns, &r))
+ .collect::()
+ )))
}
#[cfg(test)]
mod tests {
use super::*;
+ #[test]
+ fn test_parse_entry() {
+ assert_eq!(
+ Entry::from("foo"),
+ Entry {
+ col: "foo",
+ instance: None
+ }
+ );
+ assert_eq!(
+ Entry::from("foo:bar"),
+ Entry {
+ col: "foo",
+ instance: Some("bar")
+ }
+ );
+ assert_eq!(
+ Entry::from("foo: bar"),
+ Entry {
+ col: "foo",
+ instance: Some("bar")
+ }
+ );
+ }
+
#[test]
fn test_read_rows() {
assert_eq!(
read_rows(&b"foo"[..]).flatten().collect::>(),
vec![RowInput {
- label: String::from("foo"),
+ label: "foo",
entries: vec![]
}]
);
assert_eq!(
read_rows(&b"bar"[..]).flatten().collect::>(),
vec![RowInput {
- label: String::from("bar"),
+ label: "bar",
entries: vec![]
}]
);
@@ -109,11 +313,11 @@ mod tests {
read_rows(&b"foo\nbar\n"[..]).flatten().collect::>(),
vec![
RowInput {
- label: String::from("foo"),
+ label: "foo",
entries: vec![]
},
RowInput {
- label: String::from("bar"),
+ label: "bar",
entries: vec![]
}
]
@@ -121,8 +325,8 @@ mod tests {
assert_eq!(
read_rows(&b"foo\n bar\n"[..]).flatten().collect::>(),
vec![RowInput {
- label: String::from("foo"),
- entries: vec![String::from("bar")]
+ label: "foo",
+ entries: vec![Entry::from("bar")]
}]
);
assert_eq!(
@@ -130,8 +334,8 @@ mod tests {
.flatten()
.collect::>(),
vec![RowInput {
- label: String::from("foo"),
- entries: vec![String::from("bar"), String::from("baz")]
+ label: "foo",
+ entries: vec![Entry::from("bar"), Entry::from("baz")]
}]
);
assert_eq!(
@@ -140,11 +344,11 @@ mod tests {
.collect::>(),
vec![
RowInput {
- label: String::from("foo"),
+ label: "foo",
entries: vec![]
},
RowInput {
- label: String::from("bar"),
+ label: "bar",
entries: vec![]
}
]
@@ -155,11 +359,11 @@ mod tests {
.collect::>(),
vec![
RowInput {
- label: String::from("foo"),
+ label: "foo",
entries: vec![]
},
RowInput {
- label: String::from("bar"),
+ label: "bar",
entries: vec![]
}
]
@@ -169,8 +373,8 @@ mod tests {
.flatten()
.collect::>(),
vec![RowInput {
- label: String::from("foo"),
- entries: vec![String::from("bar")]
+ label: "foo",
+ entries: vec![Entry::from("bar")]
}]
);
@@ -191,7 +395,7 @@ mod tests {
.collect::, _>>()
.unwrap()
),
- HashMap::from([(String::from("bar"), 1), (String::from("baz"), 1)])
+ vec![(1, String::from("bar")), (1, String::from("baz"))]
);
assert_eq!(
column_counts(
@@ -199,7 +403,107 @@ mod tests {
.collect::, _>>()
.unwrap()
),
- HashMap::from([(String::from("bar"), 1), (String::from("baz"), 2)])
+ vec![(2, String::from("baz")), (1, String::from("bar"))]
+ );
+ assert_eq!(
+ column_counts(
+ &read_rows(&b"foo\n bar\n bar\n baz\n bar\nquux\n baz"[..])
+ .collect::, _>>()
+ .unwrap()
+ ),
+ vec![(2, String::from("baz")), (1, String::from("bar"))]
+ );
+ assert_eq!(
+ column_counts(
+ &read_rows(&b"foo\n bar: 1\n bar: 2\n baz\n bar\nquux\n baz"[..])
+ .collect::, _>>()
+ .unwrap()
+ ),
+ vec![(2, String::from("baz")), (1, String::from("bar"))]
+ );
+ }
+
+ #[test]
+ fn test_render_cell() {
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "nope",
+ entries: vec![]
+ }
+ ),
+ HTML::from(" ")
+ );
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "nope",
+ entries: vec![Entry::from("bar")]
+ }
+ ),
+ HTML::from(" ")
+ );
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "nope",
+ entries: vec![Entry::from("foo")]
+ }
+ ),
+ HTML::from(" ")
+ );
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "nope",
+ entries: vec![Entry::from("foo"), Entry::from("foo")]
+ }
+ ),
+ HTML::from("2 ")
+ );
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "nope",
+ entries: vec![Entry::from("foo: 5"), Entry::from("foo: 10")]
+ }
+ ),
+ HTML::from("5 10 ")
+ );
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "nope",
+ entries: vec![Entry::from("foo: 5"), Entry::from("foo")]
+ }
+ ),
+ HTML::from("5 â ")
+ );
+ assert_eq!(
+ render_cell(
+ "heart",
+ &RowInput {
+ label: "nope",
+ entries: vec![Entry::from("heart: <3")]
+ }
+ ),
+ HTML::from("<3 ")
+ );
+ assert_eq!(
+ render_cell(
+ "foo",
+ &RowInput {
+ label: "bob's",
+ entries: vec![Entry::from("foo")]
+ }
+ ),
+ HTML::from(" ")
);
}
}