[Rust 1.75.0][Linux] Comment trier de manière élégante un objet ReadDir ?

Salutations !!

J'ai un bug dans mon programme, les commentaires n'arrivent pas dans le bon ordre, et ceux-ci sont calqués sur des fichiers lus dans un répertoire.
Ces fichiers sont lus, contrairement à ce que je pensais, sur base du nom du fichier, et non de la date de création ce qui me pose soucis.

Du coup, me voilà obligé de trier les entrées DirEntry contenues dans un ReadDir.
Le soucis c'est qu'il n'y a pas de méthode sort() pour ReadDir :{

Comment je pourrais faire ?

voilà ce que je ne trouve pas élégant:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

// Pas beau je trouve   for fichieratrier in iterator_folder.unwrap() { if fichieratrier.is_ok() { vecteurtri.push(fichieratrier.unwrap()); } }   vecteurtri.sort_by(|gauche,droite|gauche.metadata().unwrap().modified().unwrap().cmp(&droite.metadata().unwrap().modified().unwrap()));

---

L'idée de créer un vecteur a trier est bonne. Pour avoir un code plus simple tu pourrais :

• utiliser directement les méthodes filter_map() et collect() de l’itérateur plutôt que de faire une boucle pour recréer un vecteur.
• utiliser sort_by_key() plutôt que sort_by() pour simplifier la comparaison
• utiliser l'opérateur ? plutôt que des unwrap() (pas possible dans la closure malheureusement)

Ca donnerait

Code:

---

1 2 3 4 5

let mut dirs = iterator_folder? .filter_map(|f| f.ok()) // ne garde que les resultats Ok et extrait leur contenu .collect::<Vec<_>>(); // convertit en Vec dirs.sort_by_key(|i| i.metadata().unwrap().modified().unwrap());

---

Oui !! Wow !!
Merci !!

Bonjour,

Citation:

---

Envoyé par Uther

utiliser l'opérateur ? plutôt que des unwrap() (pas possible dans la closure malheureusement)
[…]

Code:

---

1 2 3 4 5

let mut dirs = iterator_folder? .filter_map(|f| f.ok()) // ne garde que les resultats Ok et extrait leur contenu .collect::<Vec<_>>(); // convertit en Vec dirs.sort_by_key(|i| i.metadata().unwrap().modified().unwrap());

---

---

Pour compléter la réponse d'Uther, pour éviter les .unwrap(), on peut faire ceci :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37

use std::fs::{self, DirEntry, Metadata}; use std::io; use std::path::Path; use std::time::SystemTime;   #[allow(unused)] #[derive(Debug)] struct ExtendedDirEntry { entry: DirEntry, metadata: Metadata, modified: SystemTime, }   fn read_dir_and_sort_by_modified(path: &Path) -> io::Result<Vec<ExtendedDirEntry>> { let mut result = fs::read_dir(path)? .map(|entry| { let entry = entry?; let metadata = entry.metadata()?; let modified = metadata.modified()?; Ok(ExtendedDirEntry { entry, metadata, modified, }) }) .collect::<io::Result<Vec<ExtendedDirEntry>>>()?; result.sort_by_key(|x| x.modified); Ok(result) }   fn main() -> io::Result<()> { let extended_entries = read_dir_and_sort_by_modified(".".as_ref())?; for extended_entry in &extended_entries { println!("- {extended_entry:?}"); } Ok(()) }

---

Remarques :

• Mon code a un comportement différent : si un io::Result<DirEntry> est un Err au lieu d'un Ok, je remonte l'erreur au lieu de l'ignorer. Si on veut vraiment l'ignorer, alors on peut remplacer .map(|entry| { let entry = entry?; par .filter_map(Result::ok).map(|entry| {, mais pourquoi ignorer l'erreur ?
• Dans mon code, pour simplifier, on peut retirer le champ metadata de la structure ExtendedDirEntry. Par contre, si le vrai code a encore besoin des métadonnées plus tard, alors il vaut mieux laisser metadata dans ExtendedDirEntry que d'appeler à nouveau std::fs::DirEntry::metadata.

Oui, merci encore, c'est beau ^^ c'est vraiment magnifique ^^
Euh...c'est pas demain que je vais réellement appréhender la puissance de Rust, en matière d'orienté objet c'est vraiment au-dessus de tout...

J'ai pas mal de bouquins que je lis et j'essaye de capter avec des projets persos qui, pour le moment, consistent à prendre des programmes écrits en C et les "porter" en Rust.

Là j'ai du mal avec les traits, les closures (bien qu'en perl il y en aie mais je sais pourquoi ça "passe" mieux en perl), en OO j'aime le polymorphisme des fonctions (C++, Java, ...) et là en Rust j'ai vraiment du mal.
Bon, ça fait quelques mois que je "chipottes" mais j'ai du chemin à faire là... c'est pas un réflexe ^^