Discussion :

[hurukan]

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 ?

[hurukan]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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()));

[Uther]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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());

[hurukan]

Oui !! Wow !!
Merci !!

[Pyramidev]

Bonjour,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[hurukan]

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...