Discussion :

[Arioch]

Bonjour !

Bon , je vais probablement me faire déchiqueter sur place par @sver ou wiztricks mais tant pis, j'expose mon problème.

Pour des besoins de comparaison de dates entre elles, j'ai besoin de calculer le nombre de jours séparant la date du jour d'une date donnée.

Pour ce faire, j'ai une classe Converter qui, entre autres, dispose d'une méthode statique nommée gap_with_today :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
from datetime import datetime
from typing import ClassVar
 
class Converter:
    TODAY: ClassVar[str] = datetime.now().strftime("%Y-%m-%d")
 
    @staticmethod
    def gap_with_today(date: str) -> int:
        return (
            datetime.strptime(Converter.TODAY, "%Y-%m-%d").date().day
            - datetime.strptime(date, "%Y-%m-%d").date().day)

Voici mon test :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
from datetime import datetime
import pytest
from app.converter import Converter
 
@pytest.fixture
def conv() -> Converter:
    return Converter()
 
class TestConverter:
    @pytest.mark.parametrize(
        "date,expected",
        [
            ("2022-01-01", 0),
            ("2022-12-31", -365),
        ],
    )
    def test_gap_with_today(self, conv: Converter, date: str, expected: int) -> None:
        Converter.TODAY = "2022-01-01"
        assert conv.gap_with_today(date) == expected

Comme on le voit, dans le test je force la date du jour à devenir 2022-01-01. Lorsque le test passe sur le 1er élément de la fixture parametrize de pytest, le résultat attendu donne bien 0 en retour (2022-01-01 - 2022-01-01). Pour le second élément de la fixture, je m'attends à avoir -365 comme résultat de la différence en nombre de jours entre 2022-01-01 et 2022-12-31.

Mais le test passe au rouge :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

FAILED tests/unit/test_converter.py::TestConverter::test_gap_with_today[2022-12-31-365] - AssertionError: assert -30 == -365

La méthode retourne -30 en écart alors forcément, pytest se plaint.

Chuis pas doué avec les dates, ça vous l'aurez remarqué mais avez-vous un moyen simple de comparer deux dates formatées %Y-%m-%d de manière plus efficace ?

[tyrtamos]

Bonjour,

je vais probablement me faire déchiqueter sur place par @sver ou wiztricks mais tant pis, j'expose mon problème.

J'adore !


Petit test sur les genoux:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
import datetime
 
d1 = datetime.date(2022, 1, 1)
d2 = datetime.date(2022, 1, 1)
print(d2-d1)
0:00:00

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
import datetime
 
d1 = datetime.date(2022, 1, 1)
d2 = datetime.date(2022, 12, 31)
print(d1-d2)
-364 days, 0:00:00

[umfred]

une de tes erreurs c'est le .day qui renvoie le nombre de jour du mois de l'année et pas le nombre de jour depuis le début de l'année comme tu devais le croire (sinon c'est le yday du .timetuple() https://docs.python.org/fr/3/library...date.timetuple )

[Sve@r]

Une autre erreur aussi, c'est de mettre TODAY de ton objet Converter en tant qu'attribut statique. Si tu crées deux instances de Converter, le TODAY sera le même pour les deux. C'est peut-être pas forcément adéquat... (ce qui influe alors sur la méthode statique gap_with_today() qui, dès lors, ne sera, elle non plus, plus statique)...

[Arioch]

Merci à tous pour vos retours !

J'ai mis en place ceci dans le code et les tests passent au vert.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
from datetime import datetime, date
from typing import ClassVar
 
class Converter:
    TODAY: ClassVar[str] = datetime.now().strftime("%Y-%m-%d")
 
    @staticmethod
    def gap_with_today(date_: str) -> int:
        today: date.fromisoformat = date.fromisoformat(Converter.TODAY)
        old_date: date.fromisoformat = date.fromisoformat(date_)
        return (today - old_date).days

[Arioch]

L'objet timedelta ??? (zut, c'était vraiment trop facile de te déchiqueter... )

Alors, timedelta, c'est ce que j'avais regardé en premier mais je n'ai pas lu la bonne doc et elle ne parlait pas de fromisoformat que j'ai utilisé.

Une autre erreur aussi, c'est de mettre TODAY de ton objet Converter en tant qu'attribut statique. Si tu crées deux instances de Converter, le TODAY sera le même pour les deux. C'est peut-être pas forcément adéquat... (ce qui influe alors sur la méthode statique gap_with_today() qui, dès lors, ne sera, elle non plus, plus statique)...

Dans le cas du mec déchiqueté il n'y aura jamais plus d'une instance de la classe Converter.

[Sve@r]

Dans le cas du mec déchiqueté il n'y aura jamais plus d'une instance de la classe Converter.

Ca c'est pas une bonne approche. Le but d'un objet est d'être réutilisable dans différentes configurations. Si les codeurs de la classe str avaient dit "bah c'est pas grave si la chaine est statique car chez-moi je n'utilise qu'une seule instance" ben personne ne l'utiliserait.
Tu as le droit d'avoir des attributs statiques mais uniquement si ces attributs répondent à une problématique associée à ton objet, pas à ta façon de l'utiliser. Si moi je me dis "ah il a l'air pas mal cet objet Converter" mais que derrière je rajoute "dommage que son TODAY soit statique car j'en aurais bien eu besoin dans plusieurs instances" quelque part tu as raté ton objet.

[Sve@r]

Bonjour

J'adore !

Moi aussi

Bon , je vais probablement me faire déchiqueter sur place par @sver ou wiztricks mais tant pis, j'expose mon problème.

Ouais vas-y, no-limit/no-fear

mais avez-vous un moyen simple de comparer deux dates formatées %Y-%m-%d de manière plus efficace ?

L'objet timedelta ??? (zut, c'était vraiment trop facile de te déchiqueter... )