Joffrey Fouché - fin verif date et début algo by joffrey3 · Pull Request #39 · bcalou/tp-python-doomsday-rule

joffrey3 · 2023-11-15T15:39:54Z

No description provided.

bcalou

Des choses à corriger mais c'est bien avec un soin apporté à la clarté. Attention à vérifier pycodestyle

bcalou · 2023-11-17T09:14:40Z

__main__.py

+    """main function ask a date, if the date is incorrect ask again,
+    when the date is correct,
+    main function ask for the day of the date and return it to the user"""
+
+    while True:
+        input_date: str = input(
+            "Please enter a date if you want to know it day on week")
+        if (date.is_valid_date(input_date)):
+            break
+    print(input_date + " is a " + algorithm.get_weekday_for_date(input_date))



bcalou · 2023-11-17T09:15:11Z

doomsday/algorithm.py

@@ -1,2 +1,96 @@
+import doomsday.date as date
+
+WEEK_DAY_STR = ("Sunday", "Monday", "Tuesday", "Wednesday",


Eviter la référence au type dans le nom des variable eux-mêmes (str)
Weekday c'est sans espace, petit détail

bcalou · 2023-11-17T09:16:16Z

doomsday/algorithm.py

+TAB_CHANGE_FOR_CENTURY = (2, 0, 5, 3)
+TAB_ANCHOR_DAY_IN_MONTH = (10, 21, 0, 4, 9, 6, 11, 8, 5, 10, 7, 12)


Pas sûr de la signification de "TAB" ici

bcalou · 2023-11-17T09:17:19Z

doomsday/algorithm.py

+TAB_CHANGE_FOR_CENTURY = (2, 0, 5, 3)
+TAB_ANCHOR_DAY_IN_MONTH = (10, 21, 0, 4, 9, 6, 11, 8, 5, 10, 7, 12)
+
+NUMBER_ADD_IF_ODD = 11


Très bien les constantes jusqu'ici, attention cela dit elles ne se valent pas toutes. Ce 11 est très spécifique, très local. Il peut être déclaré juste là où on en a besoin.
Le haut du fichier est plus adapté aux constantes très générales

bcalou · 2023-11-17T09:17:58Z

doomsday/algorithm.py

+YEAR = 0
+MONTH = 1
+DAY = 2


Pas mal pour fluidifier la suite en effet

bcalou · 2023-11-17T09:22:48Z

doomsday/date.py

    return True
+
+
+def List_of_possible_error(error_code: int, error_text: str) -> bool:


pas de majuscule dans le nom d'une fonction

Et c'est une fonction qui fait une action, donc à renommer, ce n'est pas juste une liste

bcalou · 2023-11-17T09:23:32Z

doomsday/date.py

+    return False
+
+
+def Month_last_day(month_number: int, year: int) -> int:


pas de majuscule

bcalou · 2023-11-17T09:24:09Z

doomsday/date.py

+    return MIN_MONTH_LAST_DAY
+
+
+def get_leap(year: int) -> bool:


is_leap (on pose une question)

bcalou · 2023-11-17T09:24:32Z

doomsday/date.py

+
+
+def get_leap(year: int) -> bool:
+    """change the variable if the year is leap"""


Commentaire inexact . "Change" the variable ?

bcalou · 2023-11-17T09:26:21Z

doomsday/algorithm.py

+
+def add_century_step(number: int, century: int) -> int:
+    """we add a number different with the century"""
+    return number+TAB_CHANGE_FOR_CENTURY[(century // CENTURY) %


Attention tu dois avoir des erreurs de formattage avec pycodestyle (espacement autour du +)

bcalou · 2023-11-24T11:32:49Z

Beaucoup d'efforts pour avoir un code très clair, félicitations.
On peut toujours améliorer, peut-être alléger certains passages (beaucoup de sous-fonctions, c'est bien mais c'est un équilibre à trouver).
Mais c'est un très bon début pour ces TP.

joffrey3 added 3 commits November 15, 2023 16:37

fin verif date et début algo

2ac38e8

fin de l'agorithme du jour

5b439de

fin

a653eeb

joffrey3 marked this pull request as ready for review November 17, 2023 08:34

bcalou reviewed Nov 17, 2023

View reviewed changes

fin avec la revue corrigée

b686387

		@@ -1,2 +1,96 @@
		import doomsday.date as date

		WEEK_DAY_STR = ("Sunday", "Monday", "Tuesday", "Wednesday",

		TAB_CHANGE_FOR_CENTURY = (2, 0, 5, 3)
		TAB_ANCHOR_DAY_IN_MONTH = (10, 21, 0, 4, 9, 6, 11, 8, 5, 10, 7, 12)

		return True


		def List_of_possible_error(error_code: int, error_text: str) -> bool:

		return False


		def Month_last_day(month_number: int, year: int) -> int:



		def get_leap(year: int) -> bool:
		"""change the variable if the year is leap"""

Conversation

joffrey3 commented Nov 15, 2023

Uh oh!

bcalou left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bcalou commented Nov 24, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants