rust-lang/rust-clippy

Lint `set_*` and `get_*` methods that do nothing but access the field

Open

#1673 aperta il 12 apr 2017

Vedi su GitHub
 (18 commenti) (3 reazioni) (0 assegnatari)Rust (1391 fork)batch import
A-lintL-styleT-middlegood first issue

Metriche repository

Star
 (10.406 star)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

It is not idiomatic in Rust to have setters and getters. Make the field public instead.

If the type only has a get_foo method but not a set_foo method, suggest naming it foo instead.

Guida contributor

Direzione di ricerca
Studia i lint esistenti di clippy per getter/setter, traccia come controllano i corpi dei metodi e l'accesso ai campi, poi implementa un nuovo lint per rilevare metodi `get *` e `set *` che restituiscono o assegnano soltanto un campo.
Tech stack
rust
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65