Update listing CLI

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "jumpy"
-version = "0.3.11"
+version = "0.3.12"
 edition = "2021"
 authors = ["Clément Nerma <[email protected]>"]
 license = "Apache-2.0"

src/cmd.rs

@@ -76,7 +76,13 @@ pub struct Query {
 }
 
 #[derive(Args)]
-pub struct List {}
+pub struct List {
+    #[clap(long, help = "Display the entry's ranking")]
+    pub ranking: bool,
+
+    #[clap(long, help = "Sort by score")]
+    pub sort_by_score: bool,
+}
 
 #[derive(Args)]
 pub struct Del {

src/index.rs

@@ -134,12 +134,8 @@ impl Index {
         Some(path)
     }
 
-    pub fn list(&self) -> Vec<&str> {
-        let mut entries: Vec<_> = self.scored_entries.iter().map(IndexEntry::from).collect();
-
-        entries.sort_by(|a, b| b.cmp(a));
-
-        entries.iter().map(|result| result.path).collect()
+    pub fn iter(&self) -> impl Iterator<Item = IndexEntry> {
+        self.scored_entries.iter().map(IndexEntry::from)
     }
 
     pub fn remove(&mut self, path: &str) -> Result<(), &'static str> {
@@ -217,8 +213,8 @@ impl Index {
 
 #[derive(PartialEq, Eq)]
 pub struct IndexEntry<'a> {
-    path: &'a str,
-    score: u64,
+    pub path: &'a str,
+    pub score: u64,
 }
 
 impl<'a> From<(&'a String, &'a u64)> for IndexEntry<'a> {

src/main.rs

@@ -7,6 +7,8 @@ mod index;
 
 use clap::Parser;
 
+use crate::index::IndexEntry;
+
 static INDEX_FILENAME: &str = "jumpy.db";
 
 fn fail(message: &str) -> ! {
@@ -76,9 +78,31 @@ fn main() {
             }
         }
 
-        Action::List(List {}) => {
-            for dir in index.list() {
-                println!("{dir}");
+        Action::List(List {
+            ranking,
+            sort_by_score,
+        }) => {
+            let mut entries = index.iter().collect::<Vec<_>>();
+
+            if !sort_by_score {
+                entries.sort_by_key(|entry| entry.path);
+            } else {
+                entries.sort_by(|a, b| a.score.cmp(&b.score).reverse());
+            }
+
+            let longest_score = entries
+                .iter()
+                .map(|entry| entry.score)
+                .max()
+                .map(|score| score.to_string().len())
+                .unwrap_or(0);
+
+            for IndexEntry { path, score } in entries {
+                if !ranking {
+                    println!("{path}");
+                } else {
+                    println!("{score:>longest_score$} {path}");
+                }
             }
         }