microsoft / microsoft/vscode-java-debug

Refactor: Extract common code from LaunchRequestHandler's delegate

Ouverte
#373 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

ai-triaged debt enhancement
Langage dominant
TypeScript
Étoiles
591
Forks
429
Merge moyen
1 j 9 h
PR mergées (30 j)
19

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par ILaunchDelegate.java aux lignes 25-27, puis examinez les implémentations de launchInTerminal et launchInternally dans le delegate de LaunchRequestHandler. Comparez la logique commune et les différences debug/noDebug décrites dans l’issue ; le travail est terminé lorsque le comportement commun est consolidé et que le delegate reste lisible sans modifier ces comportements de lancement distincts.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
devtools
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.