Proposal. Simplify point 4.5 (Filter)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 35/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- documentation
Direzione di ricerca
Inizia dal punto 4.5 (Filter) nella documentazione sulle best practice e confronta gli esempi esistenti “cattivo” e “buono” con l’espressione proposta. Il lavoro è completato aggiornando l’esempio se il filtro più breve viene accettato e preservando le indicazioni previste dalla sezione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In this part of best practice I found a part of code
// bad
inbox.filter((msg) => {
const { subject, author } = msg;
if (subject === 'Mockingbird') {
return author === 'Harper Lee';
} else {
return false;
}
});
// good
inbox.filter((msg) => {
const { subject, author } = msg;
if (subject === 'Mockingbird') {
return author === 'Harper Lee';
}
return false;
});
But, it seems that we can write this part of code better:
inbox.filter((msg) => {
const { subject, author } = msg;
return subject === 'Mockingbird' && author === 'Harper Lee';
});
Isn't it?
Thanks,
Andrii
- Lingua principale
- JavaScript
- Stelle
- 148k
- Fork
- 26.6k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedForse già presa @bodapatisaikrishna l’ha presa 25 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
airbnb/javascript#3237 · 6 commenti ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Forse già presa @Developer-shivamMishra l’ha presa 16 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
airbnb/javascript#3152 · 4 commenti ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Forse già presa @dataCenter430 l’ha presa 220 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
airbnb/javascript#3238 · 8 commenti ·
-
Upgrading eslint-plugin-react-hooksForse già presa @weihongyu12 l’ha presa 366 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 38/100
airbnb/javascript#3186 · 3 commenti · 2 reazioni ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
airbnb/javascript#3173 · 9 commenti · 1 reazione ·
Tutte le issue di airbnb/javascript
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
enhancement
Difficoltà 1/5 1-3 ore Idoneità per principianti 85/100
holepunchto/hrpc-test#28 ·
-
good first issue
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
anoopcodehack/DevBoard#616 ·
I maintainer di solito rispondono entro 2 giorni
-
idnits3
Difficoltà 2/5 1-3 ore Idoneità per principianti 63/100
ietf-tools/idnits#319 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
521xueweihan/HelloGitHub#3939 ·