Laffini / Laffini/Java-Matching-Engine-Core

Questions about order modification

Ouverte
#26 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Java
Étoiles
23
Forks
15
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Hi Laffini. When examining the Order class code in the core, I noticed that the side attribute was not marked as final. Upon further investigation, I found that the change order interface provides the ability to modify the side. However, generally speaking, exchanges do not allow modifications to the side. Therefore, I am somewhat puzzled about this. @Laffini

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

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 lire la classe Order dans le core et l’interface de modification d’un ordre mentionnée dans l’issue. Déterminez si le changement de côté est prévu pour ce moteur de matching et, dans le cas contraire, définissez la modification de conception requise ainsi que ses critères d’achèvement. L’issue est actuellement une question et ne mentionne aucun test.

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

Évaluation

Stack technique
java
Domaine
backend
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Calme
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.