redhat-developer / redhat-developer/vscode-java
Refactoring extraction methods leads to naming conflicts
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.3k
- Forks
- 546
- Merge medio
- 20 h 1 min
- PR fusionados (30 d)
- 11
Descripción
[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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce el flujo de Java “Refactor-Extract to method” con la clase proporcionada en la configuración indicada de Windows 11, JDK 17 y VS Code 1.88, utilizando el punto de entrada de la extensión de Java. Rastrea cómo se comprueba el nombre solicitado extractedMethod frente a la clase circundante y, a continuación, verifica que se detecta un conflicto y se muestra una advertencia antes o después de la refactorización.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java, typescript
- Área
- tooling
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100