microsoft / microsoft/vscode-java-debug

Refactor: Extract common code from LaunchRequestHandler's delegate

Offen
#373 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

ai-triaged debt enhancement
Vorherrschende Sprache
TypeScript
Sterne
591
Forks
429
Ø Merge
1 T. 9 Std.
Gemergte PRs (30 T.)
19

Beschreibung

See ILaunchDelegate.java#L25-L27

    CompletableFuture<Response> launchInTerminal(LaunchArguments launchArguments, Response response, IDebugAdapterContext context);

    CompletableFuture<Response> launchInternally(LaunchArguments launchArguments, Response response, IDebugAdapterContext context);

There is common logic in implementation for both interfaces. The main differences in implementation for debug/noDebug are:

  • launchInTerminal: It constructs different command string, and then take different actions according to response.
  • launchInternally: It starts debuggee process in different ways.

Refactor work should be done to make code clean and readable.

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

Beginne mit ILaunchDelegate.java in den Zeilen 25-27 und untersuche anschließend die Implementierungen von launchInTerminal und launchInternally im Delegate von LaunchRequestHandler. Vergleiche die gemeinsame Logik und die im Issue beschriebenen debug/noDebug-Unterschiede; abgeschlossen ist die Aufgabe, wenn das gemeinsame Verhalten konsolidiert ist und der Delegate lesbar bleibt, ohne diese unterschiedlichen Startverhalten zu ändern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
devtools
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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