Prevent an implementation for `int` class from operating on `bool` values
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez par reproduire l’exemple avec classes 0.4.0, puis suivez le dispatch à partir de la déclaration de la typeclass et de l’enregistrement de render.instance(int). C’est terminé lorsque render(True) n’utilise plus l’implémentation de int et lève à la place NotImplementedError pour le cas bool non spécifié.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Context. Consider the following piece of code.
from classes import typeclass
@typeclass
def render(data_value) -> str:
"""Pretty-print a value."""
@render.instance(int)
def _render_int(data_value: int) -> str:
return f'🔢 {data_value}'
render(True) == '🔢 True'
(this should be runnable as-is on classes 0.4.0.)
I would have expected this code to fail with a NotImplementedError because the bool case wasn't specified; but instead, that case is handled by the int implementation because
In [3]: issubclass(bool, int)
Out[3]: True
Decision. In an int implementation, recognize if the provided value is actually a bool and refuse to process that value.
Consequences. I have been writing in Python for quite a few years now and I might even have encountered this relationship between int and bool types before, but that is not quite a type of thing that I keep in my short-term memory to be able to instantly recognize.
I believe this change can help avert bugs which might be hard to trace otherwise.
- Langage dominant
- Python
- Étoiles
- 730
- Forks
- 30
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de dry-python/classes
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
dry-python/classes#496 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
dry-python/classes#494 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 52/100
dry-python/classes#492 · 2 commentaires ·
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
dry-python/classes#481 · 5 commentaires · 4 réactions ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
dry-python/classes#386 ·
Toutes les issues de dry-python/classes
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
stephrobert/dsoxlab#238 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
sublimehq/package_control#1780 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
nwg-piotr/nwg-displays#145 ·