microsoft / microsoft/TypeScript
extract function including traling comment that is not selected
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
Bug Report
extract function including traling comment that is not selected
🔎 Search Terms
🕗 Version & Regression Information
- This is a bug in 4.3.2-latest version
When I use 'extract to method in class C' in following code, the selected region does not contain comment '/end/', but when I excecute 'extract to method in class C', the extractd method contains comment '/end/'. Is this a bug?I think it would better comment not selected appear in newMethod.
⏯ Playground Link
Playground link with relevant code
💻 Code
class C{
a() {
let z =1;
/*start*/let a1= {x:1};
let y =10;
z=42
return a1.x+ 10 /*end*/
}
}
🙁 Actual behavior
class C{
a() {
let z = 1;
/*start*/
let __return;
({ __return, z } = this.newMethod(z));
return __return; /*end*
}
private newMethod(z: number) {
let a1 = { x: 1 };
let y = 10;
z = 42;
return { __return: a1.x + 10 /*end*/
, z };
}
}
🙂 Expected behavior
class C{
a() {
let z = 1;
/*start*/
let __return;
({ __return, z } = this.newMethod(z));
return __return; /*end*/
}
private newMethod(z: number) {
let a1 = { x: 1 };
let y = 10;
z = 42;
return { __return: a1.x + 10 , z };
}
}
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
Riproduci il comportamento extract-to-method dal TypeScript Playground collegato usando l’esempio fornito con class C, confrontando la regione selezionata con il metodo generato. Traccia il punto di ingresso del refactoring extract-function e aggiungi la copertura per il commento finale; il lavoro è completato quando il commento rimane con l’istruzione return originale invece di comparire nel metodo estratto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- devtools
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100