redhat-developer / redhat-developer/vscode-java
Refactoring extraction methods leads to naming conflicts
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 2.3k
- Forks
- 546
- Merge moyen
- 20 h 1 min
- PR mergées (30 j)
- 11
Description
[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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Reproduisez le flux Java « Refactor-Extract to method » avec la classe fournie sur la configuration indiquée composée de Windows 11, JDK 17 et VS Code 1.88, en utilisant le point d’entrée de l’extension Java. Suivez la manière dont le nom demandé extractedMethod est vérifié par rapport à la classe englobante, puis vérifiez qu’un conflit est détecté et qu’un avertissement est affiché avant ou après la refactorisation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java, typescript
- Domaine
- tooling
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100