redhat-developer / redhat-developer/vscode-java
Extracting local variables and refactoring 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 "20"
- click “Refacator-Extract local variable”
- new variable name " existingVariable"
public class OriginalClass {
public void method() {
int existingVariable = 10;
// extract variable "20" , name :"existingVariable"
System.out.println(20);
}
}
refactoring result:
Reason: When extracting local variables, there was no detection of naming conflicts between the new variable 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 la refactorización indicada „Extract local variable“ en VS Code con las versiones de Java y de la extensión indicadas, utilizando el ejemplo proporcionado OriginalClass y un nombre existingVariable. Siga el punto de entrada de la refactorización para determinar dónde se comprueban los conflictos. Se considera terminado cuando el conflicto se detecta antes o después de la refactorización y se muestra una advertencia, en lugar de producir silenciosamente el resultado conflictivo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java, typescript, vscode
- Área
- developer-experience, 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