Rule proposal: `no-useless-recursion`

Metriche repository

Star

5022 stelle (5022 stelle)

Metriche merge PR

Merge medio 4h 30m (Merge medio 4h 30m)26 PR mergiate in 30 g (26 PR mergiate in 30 g)

Descrizione

function foo(bar) {
  if (bar.baz) {
    return foo(bar.baz);
  }

  return bar;
}

function foo(bar) {
  while (bar.baz) {
    bar = bar.baz:
  }

  return bar;
}

function foo(bar) {
  for (; bar.baz; bar = bar.baz) {}

  return bar;
}

function foo(bar) {
  if (Array.isArray(bar)) {
    return bar.map(baz => foo(bar));
  }

  return bar;
}

Guida contributor

Direzione di ricerca

L'issue propone una nuova regola ESLint per rilevare la 'ricorsione inutile', in cui una funzione chiama ricorsivamente se stessa in coda con lo stesso argomento o con un argomento trasformato, senza un caso base. La sfida principale è definire con precisione la ricorsione 'semplice'. I commenti di revisione (ad es. #1078) suggeriscono di discutere l'ambito, ad esempio se segnalare solo le chiamate tail recursive o qualsiasi auto chiamata senza una condizione di arresto. Controllare le regole ESLint esistenti come no useless return o no useless catch per i pattern. Consultare la documentazione di eslint plugin unicorn sulla creazione di regole (in lib/rules/) e le convenzioni di test. Coinvolgere i maintainer per chiarire la definizione prima dell'implementazione.

Tech stack

javascript

Dominio

developer experiencetooling

Tipo issue

Funzionalità

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

3

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

3-5 giorni

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

In attesa del maintainer

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Abbastanza chiara

Prerequisiti

AST knowledgeESLint plugin developmentJavaScript

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

45