Réforme dynamique : Corrige le problème de non création de conditions lorsqu'un profil est présent deux fois #194
Conversation
Allan-CodeWorks
force-pushed
the
Fix_reforme_double_profil
branch
3 times, most recently
from
8bc8c85 to
3fd88e4
Compare
| def condition_already_exists_in_node(profil_condition, conditions_in_node_data) -> bool: | ||
| for condition_type in profil_condition.keys(): | ||
| if condition_type in conditions_in_node_data: | ||
| conditions_with_operator_fields = ['age', 'quotient_familial', 'situation_handicap'] |
There was a problem hiding this comment.
cette définition on pourrait la remonté avant le for, ça rendrait un peu mieux j'ai l'impression, t'en pense quoi ?
|
|
||
| def add_profil_with_conditions(data: dict, profil: dict): | ||
| data['profils'][profil['type']].update(conditions_to_node_data(profil['conditions'])) | ||
| def condition_already_exists_in_node(profil_condition, conditions_in_node_data) -> bool: | ||
| for condition_type in profil_condition.keys(): |
There was a problem hiding this comment.
Pas obligé, mais pour facilité la lecture ce que je fais parfois c'est réduire le nombre de branche possibles le plus rapidement possible (1). Par exemple là jusqu'à la toute fin de la méthode on doit gardé en tête qu'est-ce qui se passe si profil_condition.keys() est vide. On pourrait mettre un early return pour enlever ce cas des possible. Ça permetrait en plus ligne 118 de return directement operator in conditions_in_node_data[condition_type]
(1) oui ça fait deux fois possible :-)
There was a problem hiding this comment.
Je ne vois pas bien ce que tu veux dire :
- si
profil_condition.keys()est vide, on ne rentre pas dans le for - Je peux déjà return comme tu le dis en ligne 118 (et merci, je l'ai fait)
There was a problem hiding this comment.
Justement, mais là on doit garder en tête que si c'est vide faudrait regarder après le for
Alors que si on fait avant le for
if len(profil_condition) == 0:
return False
On est sûr à 100% qu'on rentre dans le for, et on ecarte à 100% le cas profile_condition est vide.
Ça change pas grand chose et c'est même plus long, mais ça facilite la lecture (enfin je trouve)
There was a problem hiding this comment.
Encore une fois pas obligé, voici un blog post qui parle entre autre de cet argument.
https://freedium.cfd/https://betterprogramming.pub/are-early-returns-any-good-eed4b4d03866
The last argument that I'm going to make for using early returns is peace of mind for the programmer. By using early returns, you get invalid cases out of the way first (bouncer pattern), put a blank line in there next, and then you can focus on the "real" body of the function.
There was a problem hiding this comment.
D'accord, je comprends!
C'est un article intéressant et je vois maintenant l'intérêt pour le cas ici, je trouve effectivement que c'est pas plus mal de rendre ce code un peu plus facile à lire et je pense que c'est le cas avec ce "early return"
Merci!
Allan-CodeWorks
force-pushed
the
Fix_reforme_double_profil
branch
from
3fd88e4 to
5a9fce5
Compare
Allan-CodeWorks
force-pushed
the
Fix_reforme_double_profil
branch
from
5a9fce5 to
761a411
Compare
…entiques pour les aides avec profiles identiques
Allan-CodeWorks
force-pushed
the
Fix_reforme_double_profil
branch
from
761a411 to
c4acf35
Compare
Description
Si un fichier
yamlprésente deux profils de même type, la deuxième condition écrase la première.