Pour apprendre le rust, je le lançais en "pedantic", il faudra quand même trier ce qui est important dans la sortie: cargo clippy --all -- -W clippy::all -W clippy::pedantic.
En lecture (très) rapide, je vois un footgun assez réccurent sur l'usage des adaptateurs en *_or(todo!()) (unwrap_or, ok_or, ...): il vaut mieux utiliser les adptateurs *_or_else(|| todo!()) dès que ce que tu veux mettre dans le todo!() a des effets de bords ou fait des allocations. Plus d'infos ici: https://rust-lang.github.io/rust-clippy/master/index.html#/or_fun_call
[^] # Re: Clippy est un bon début
Posté par YetiBarBar . En réponse au journal PullRequest d'une application en Rust. Évalué à 3.
Je plussoie pour clippy.
Pour apprendre le rust, je le lançais en "pedantic", il faudra quand même trier ce qui est important dans la sortie:
cargo clippy --all -- -W clippy::all -W clippy::pedantic.En lecture (très) rapide, je vois un footgun assez réccurent sur l'usage des adaptateurs en
*_or(todo!())(unwrap_or, ok_or, ...): il vaut mieux utiliser les adptateurs*_or_else(|| todo!())dès que ce que tu veux mettre dans le todo!() a des effets de bords ou fait des allocations. Plus d'infos ici: https://rust-lang.github.io/rust-clippy/master/index.html#/or_fun_call