Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Implémenter une class Professeur #6

Closed
MathieuSoysal opened this issue Jan 19, 2021 · 0 comments · Fixed by #30
Closed

✨ Implémenter une class Professeur #6

MathieuSoysal opened this issue Jan 19, 2021 · 0 comments · Fixed by #30
Assignees
Labels
enhancement New feature or request

Comments

@MathieuSoysal
Copy link
Member

MathieuSoysal commented Jan 19, 2021

Afin d'intégrer une recherche par professeur.
Cette fonctionnalité contient :

  • une HashMap<String, Professeur> qui contient l'ensemble des professeurs, (enregistrés via les cours contenue dans l'emploie du temps)
  • la class Professeur possédera une méthode get délégué de sa HashMap cette méthode aura deux variables String en paramètre, l'un pour le nom et l'autre pour le prénom du professeur, ceux-ci sont recueillis afin de trouver à quelle valeur clef de la hashMap il est fait référence. Pour au final que la méthode get retourne l'objet Professeur correspondant. (note: Faire attention à formater les valeurs donnés caractère spéciaux remplacés par un espace, et chaque lettre mit en majuscule)
  • l'attribut professeur de la class Cours n'est plus de type String mais de type Professeur.

l'avantage de ce changement est de ne pas utiliser le .equals qui devient lourd lorsqu'il est répété à chacun des professeurs de chacun des cours de l'emplois du temps. Mais de passer par le == qui donnera un gain de temps drastique.

@MathieuSoysal MathieuSoysal added the enhancement New feature or request label Jan 19, 2021
@MathieuSoysal MathieuSoysal self-assigned this Jan 27, 2021
@MathieuSoysal MathieuSoysal linked a pull request Jan 28, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant