Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exos Pierre #9

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Exos Pierre #9

wants to merge 29 commits into from

Conversation

Piotr-V
Copy link

@Piotr-V Piotr-V commented Mar 15, 2017

Non optimisés.

@Jonathancollinet
Copy link
Owner

Alors pour git c'étais pas ça mais tu t'es rattrapé sur la fin.

Pour l'exo 5 tu peut le faire encore plus simplement, sans forcément créer des variables;
Pour l'exo 6 tu peut encore simplifier les conditions.
Pour l'exo 9, la méthode 2 est la mieux.
Il manque l'exo 10.

Le reste c'est du tout bon !

Piotr-V added 5 commits March 20, 2017 10:16
Suppression des variables.
Simplification des conditions.
@Piotr-V
Copy link
Author

Piotr-V commented Mar 20, 2017

Je pense être pas mal sur les exos 5 et 6. J'ai compris en discutant avec les autres que je m'étais fourvoyé sur le 10.

@Jonathancollinet
Copy link
Owner

Alors tout niquel sauf:

L'exo 10: si je te met 5000 fruits & légumes différents, feras-tu 5000 if ?

Sinon cool ! Une fois que tu aura résolu ça tu pourra passer à la suite numéro 2.

@Jonathancollinet
Copy link
Owner

Malgrès le refacto, toujours pareil:

L'exo 6: avec les ternary imbriqués ça deviens illisible; Dans ces situations une structure de condition suffit;

L'exo 10: si je te met 5000 fruits & légumes différents, feras-tu 5000 cases ?

@Piotr-V
Copy link
Author

Piotr-V commented Mar 27, 2017

Eric nous a dit que les codes tenaient en 1 ligne jusqu'à l'exo 9, c'est pour ça que je t'ai envoyé l'exo 6 en ternaire. J'attendais de te renvoyer le 7, 8, 9 en ternaire également pour t'envoyer le 10 enfin corrigé avec un objet. Mais je n'ai pas encore trouvé le moyen de tronquer mes boucles, à partir de l'exo 7, pour tout faire tenir en 1 ligne.

@joysoft33
Copy link
Collaborator

joysoft33 commented Mar 27, 2017 via email

@joysoft33
Copy link
Collaborator

joysoft33 commented Mar 27, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants