sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-iterator-to-array`

Chiusa

#2567 aperta il 20 feb 2025

Vedi su GitHubGuida contributor
 (2 commenti) (3 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

Description

Iterator#toArray() is more clear to me that coverts iterator to array than [...iterator], especially when iterator is a long expression.

Examples

// ❌
[...map.values()].map(value => value * 2)

// ✅
map.values().toArray().map(value => value * 2)
map.values().map(value => value * 2).toArray()

Proposed rule name

prefer-iterator-to-array

Additional Info

Full list of Iterators https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Iterator#description

Guida contributor

Direzione di ricerca
Esamina l'implementazione di codice di regole simili come 'prefer array flat map' nel repository eslint plugin unicorn per i pattern di codice. La regola deve rilevare l'uso dell'operatore spread sugli iterabili (es. [...map.values()]) e suggerire l'uso di .toArray() invece. Utilizza selettori AST come 'SpreadElement' o 'CallExpression' per identificare il pattern. Il problema fornisce esempi chiari di ciò che dovrebbe essere segnalato e di ciò che è preferibile.
Tech stack
javascripttypescript
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ESLint plugin developmentAST basicsJavaScript iterators
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70