Proposal. Simplify point 4.5 (Filter)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 35/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- documentation
Piste de recherche
Commencez par le point 4.5 (Filter) de la documentation des bonnes pratiques et comparez les exemples existants « mauvais » et « bon » avec l’expression proposée. Le travail est terminé lorsque l’exemple est mis à jour si le filtre plus court est accepté, tout en préservant les indications prévues par la section.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- JavaScript
- Étoiles
- 148k
- Forks
- 26.5k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedPeut-être pris @bodapatisaikrishna l’a pris il y a 27 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
airbnb/javascript#3237 · 6 commentaires ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Peut-être pris @Developer-shivamMishra l’a pris il y a 18 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
airbnb/javascript#3152 · 4 commentaires ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Peut-être pris @dataCenter430 l’a pris il y a 223 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 48/100
airbnb/javascript#3238 · 8 commentaires ·
-
Upgrading eslint-plugin-react-hooksPeut-être pris @weihongyu12 l’a pris il y a 368 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
airbnb/javascript#3186 · 3 commentaires · 2 réactions ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
airbnb/javascript#3173 · 9 commentaires · 1 réaction ·
Toutes les issues de airbnb/javascript
Issues similaires
-
feature request
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
docs hacktoberfest help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
pacocartones/free-llm-api-hub#347 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
cytoscape/cytoscape.js#3536 ·
Les mainteneurs répondent en général sous 15 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
cuttle-cards/cuttle#1440 ·
Les mainteneurs répondent en général sous 1 jour
-
URL-encoded parser builds nested objects nothing readsPeut-être pris @Senthil455 l’a pris aujourd’hui. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
Janani-bn/CivicFix#168 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour