Lisibilité et simplifications #25
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bonjour, c'est ma première participation à ce repo. Voici mes corrections : * Remplacer les .format(), obsolètes, par des f-strings, plus optimisées et plus lisibles. Note : il faut tourner Python 3.6+ (mise à jour de 2016) ; * Remplacer les max(t1, t2) - min(t1, t2) par le plus simple abs(t1 - t2) (valeur absolue). C'est plus court et plus compréhensible ; * Supprimer les indentations présentes sur les lignes vides (PEP 8) ; * Il y a beaucoup trop de parenthèses, sans toujours de cohérence, c'en est presque illisible. J'ai essayé de résoudre le problème au mieux. J'ai aussi simplifié certaines conditions (ex. l. 197) ; * Remplacer les & par des and (aucune raison d'utiliser l'opérateur bit à bit &). * Ajouts d'espaces entre les opérateurs pour la lisibilité. En règle générale j'ai appliqué les conventions de Python, c'est-à-dire celles de la PEP. Ce programme est en grande partie codé comme si c'était un autre langage. Certaines lignes sont beaucoup trop longues (jusqu'à 169 caractères dans un while !). La règle veut qu'on ,ne dépasse pas 79 char, mais c'est rarement applicable -- essayons néanmoins de limiter la longueur. Dernier point : peut-être que ce serait une bonne idée de renommer les variables "k" et "l" en quelque chose de plus explicite ? Merci de proposer à tout le monde l'accès à CovidTracker !
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bonjour, c'est ma première participation à ce repo.
Voici mes corrections :
Remplacer les .format(), obsolètes, par des f-strings, plus optimisées et plus lisibles. Note : il faut tourner Python 3.6+ (mise à jour de 2016) ;
Remplacer les max(t1, t2) - min(t1, t2) par le plus simple abs(t1 - t2) (valeur absolue). C'est plus court et plus compréhensible EDIT : c'est aussi plus optimisé (2 fois plus rapide sur des flottants) ;
Supprimer les indentations présentes sur les lignes vides (PEP 8) ;
Il y a beaucoup trop de parenthèses, sans toujours de cohérence, c'en est presque illisible. J'ai essayé de résoudre le problème au mieux. J'ai aussi simplifié certaines conditions (ex. l. 197) ;
Remplacer les & par des and (aucune raison d'utiliser l'opérateur bit à bit &).
Ajouts d'espaces entre les opérateurs pour la lisibilité.
En règle générale j'ai appliqué les conventions de Python, c'est-à-dire celles de la PEP. Ce programme est en grande partie codé comme si c'était un autre langage. Certaines lignes sont beaucoup trop longues (jusqu'à 169 caractères dans un while !). La règle veut qu'on ne dépasse pas 79 char, mais c'est rarement applicable -- essayons néanmoins de limiter la longueur.
Dernier point : peut-être que ce serait une bonne idée de renommer les variables "k" et "l" en quelque chose de plus explicite ?
Merci de proposer à tout le monde l'accès à CovidTracker !