microsoft / microsoft/TypeScript

Refactor to extract existing function to outer scope

Aperta
#26,479 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Domain: LS: Refactorings In Discussion Suggestion
Lingua principale
Go
Stelle
111k
Fork
14.3k
Merge medio
2g 4h
PR unite (30g)
132

Descrizione

TypeScript Version: 3.1.0-dev.20180813

Code

function a(x: number) {
    function b(y: number) {
        return x + y;
    }
    return b(1);
}

Expected behavior:

Refactor on b to convert to:

function a(x: number) {
    return b(1, x);
}
function b(y: number, x: number) {
    return x + y;
}

Actual behavior:

No such refactor.
We already have the ability to refactor code to an outer scope. So a workaround is to highlight the entire body of b and refactor to a function in global scope, then delete b and rename the refactored function to b, then manually update call sites.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

La issue nomina la funzione annidata b e la capacità esistente di effettuare il refactoring verso lo scope esterno; inizia individuando il punto di ingresso di quel refactoring e i relativi test. Riproduci il caso della funzione annidata, quindi verifica che il refactoring sposti b, passi x catturato come argomento e aggiorni il sito della chiamata.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
typescript
Ambito
developer-experience
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.