Disons que ya des contraintes beaucoup plus efficaces. Linters et tests sur les hooks precommit / prepush et en bout de ligne de la code review sur les merge request.
Ben typiquement je valide pas la merge request et je demande une modif. Classe inutile dans le contexte. T'as pas besoin de typer une url, le bon type d'une url c'est string sauf use case spécifique du fait du besoin d'un api (DOM par exemple).
Ok je veux bien mais la plus value de forcer un typage systématique elle est où ? Le rapport avantage / inconvénient est pas si évident que ça en pratique, pour moi ça relève beaucoup du dogme.Envoyé par Pyramidev
C'est pas le sujet, il s'agit d'épurer le code. C'est du même ordre que d'avoir un commentaire qui décrit l'objet d'une variable alors que le naming est explicite. C'est inutile la plupart du temps.Envoyé par Uther
Tu peux mais c'est de l'overkill. Tu as seulement besoin d'une fonction de validation.Envoyé par Uther
C'est beaucoup, beaucoup, beaucoup trop long. Le simple fait d'exposer tes méthodes te prend la moitié de l'écran. Une classe c'est un moule à objet, c'est pas une lib.Envoyé par Sodium
Partager