microsoft / microsoft/TypeScript
Inline function refactoring
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 2g 4h
- PR unite (30g)
- 132
Descrizione
Search Terms
inline function method refactoring
Suggestion
I would like a refactoring that inlines a function from
1: function foo() { return 42; }
2: function bar() { const meaningOfLife = foo(); }
to
1: function bar() { const meaningOfLife = 42; }
Use Cases
This is a very common refactoring and thus widely used while cleaning up code.
Examples
In the above code sample, block 1, line 1: selecting foo, the user should be able to inline this function to every occurence and optionally delete the function definition.
In the above code sample, block 1, line 2: selecting foo, the user should be able to inline the function to this occurence, and, if it's the only one, optionally delete the function definition.
I am not sure how the option can be handled in vscode. Eclipse brings up a pop-up with the two options, but afaik vs code tries to be minimalist.
Checklist
My suggestion meets these guidelines:
- This wouldn't change the runtime behavior of existing JavaScript code
- This wouldn't be a breaking change in existing TypeScript / JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. new expression-level syntax)
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dagli esempi di funzioni inline e dal comportamento richiesto per selezionare una dichiarazione di funzione o un sito di chiamata. Esamina come l’editor TypeScript gestisce attualmente i refactoring e determina la semantica per sostituire le occorrenze, preservare il comportamento e rimuovere facoltativamente la definizione. L’issue non indica file né test, quindi è ancora necessario identificare il punto di ingresso dell’implementazione e i controlli di completamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, typescript
- Ambito
- tooling
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100