redhat-developer / redhat-developer/vscode-java
Refactoring extraction methods leads to naming conflicts
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 2.3k
- Fork
- 546
- Merge medio
- 20h 1m
- PR unite (30g)
- 11
Descrizione
[provide a description of the issue]
Environment
Operating System: Windows11
JDK version: 17
Visual Studio Code version: 1.88
Java extension version: 1.29
Steps to Reproduce:
- select "extracted();'
- click “Refacator-Extract to method”
- new method name " extractedMethod() "
class ExtractMethodTest {
public void originalMethod() {};
public void methodA() {
// Extracted method
extracted();
}
private void extracted() {
extractedMethod();
}
private void extractedMethod() {
// TODO Auto-generated method stub
}
public void methodB() {
extracted();
}
}
refactoring result:
Reason: When extracting the method, there was no detection of naming conflicts between the new method name and the context
Expected behavior: Check for naming conflicts before and after refactoring, and provide warning prompts
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 flusso Java “Refactor-Extract to method” con la classe fornita sulla configurazione indicata composta da Windows 11, JDK 17 e VS Code 1.88, utilizzando il punto di ingresso dell’estensione Java. Traccia il modo in cui il nome richiesto extractedMethod viene verificato rispetto alla classe circostante, quindi verifica che venga rilevato un conflitto e che venga mostrato un avviso prima o dopo il refactoring.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java, typescript
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100