redhat-developer / redhat-developer/vscode-java

Inline method refactoring leads to syntax errors

Offen
#3,597 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

potential-duplicate
Vorherrschende Sprache
TypeScript
Sterne
2.3k
Forks
546
Ø Merge
20 Std. 1 Min.
Gemergte PRs (30 T.)
11

Beschreibung

[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
  1. select "staticMethod()"
  2. click “Refacator-inline method”
  3. class MyClass {

private static int value = 10;
// inline method
public static void staticMethod() {
System.out.println("Value: " + value);
}
}
class AnotherClass {
private int value = 20;
public void callStaticMethod() {
new MyClass().staticMethod();
}
}
refactoring result:
image

Reason: During inline method refactoring, the issue of calling fields was not detected

Expected behavior: Check for grammar issues before and after inline refactoring, and provide warning prompts

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Es wird keine Quelldatei oder kein Test genannt. Beginnen Sie damit, das Inline-Methoden-Refactoring unter Windows 11 mit JDK 17, VS Code 1.88 und Java extension 1.29 zu reproduzieren, und verfolgen Sie dann den Einstiegspunkt des Inline-Methoden-Refactorings. Als erledigt gilt die Aufgabe, wenn Feldaufrufe vor oder nach dem Refactoring erkannt werden und eine Warnung für daraus resultierende Grammatikfehler bereitgestellt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java, typescript
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.