microsoft / microsoft/TypeScript
Refactor to extract existing function to outer scope
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 2 T. 4 Std.
- Gemergte PRs (30 T.)
- 132
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Das Issue nennt die verschachtelte Funktion b und die bestehende Fähigkeit zur Umstrukturierung in den äußeren Gültigkeitsbereich; beginne damit, den Einstiegspunkt dieses Refactorings und dessen Tests zu finden. Stelle den Fall mit der verschachtelten Funktion nach und überprüfe anschließend, dass das Refactoring b verschiebt, das erfasste x als Argument übergibt und die Aufrufstelle aktualisiert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- developer-experience
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100