edunumsec2 / book

Ressources pour l'enseignement de l'informatique au Gymnase.
https://modulo-info.ch
Other
26 stars 22 forks source link

[Contenu]: #76

Open ph3n4t3s opened 2 years ago

ph3n4t3s commented 2 years ago

Page

https://apprendre.modulo-info.ch/content/appr/theme/prog1/cours/7_fonc/fonc.html

Quel est le problème relevé ?

def pythagore(a, b): print(pythagore(3, 4))

Quelle est votre suggestion ?

def pythagore(a, b): return ((a2+b2)**(1/2)) print(pythagore(3, 4))

Code of Conduct

elliotvaucher commented 2 years ago

@ph3n4t3s : merci pour le signalement.

Là, j'imagine que la logique c'était de laisser la fonction vide pour que l'élève la remplisse ?

Je joins @rasql en copie, il pourra nous répondre.

nwolff commented 2 years ago

On pourrait utiliser le mot clé pass et un commentaire ?

def pythagore(a, b):
    pass # remplacer par le vrai calcul

print(pythagore(3, 4))
jppellet commented 2 years ago

Plutôt que pass, je suggérerais ..., alors. ... symbolise mieux qu'il y a quelque chose d'incomplet à faire, alors que pass, traditionnellement, est inséré quand un corps est volontairement vide mais que Python veut quand même quelque chose.

nwolff commented 2 years ago

La doc python dit qu'on peut utiliser pass comme placeholder, ce qui est a mon avis exactement ce qu'on veut faire ici :
https://docs.python.org/3/tutorial/controlflow.html#pass-statements

L'ellipse fonctionnerait aussi, voici une discussion stackoverflow sur le choix entre les deux (pas super conclusive) : https://stackoverflow.com/questions/55274977/when-is-the-usage-of-the-python-ellipsis-to-be-preferred-over-pass

Je vais faire une PR avec pass

nwolff commented 2 years ago

En essayant de faire la PR je découvre que la page contenant cet exercice : https://apprendre.modulo-info.ch/content/appr/theme/prog1/cours/7_fonc/fonc.html est dans l'ancienne version de modulo, il n'y a donc rien à faire ?

jppellet commented 2 years ago

Je laisse @rasql, auteur de cette partie, commenter!

elliotvaucher commented 2 years ago

@nwolff

Bonjour Nicholas. Si, il y a une option pour effectuer la modification que tu proposes.

Même s'il y a une persistence étrange de la page sous https://apprendre.modulo-info.ch/content/appr/theme/prog1/cours/7_fonc/fonc.html (je dis qu'elle est étrange parce que je n'ai juste pas eu le temps d'investiguer pourquoi elle est là), les anciennes versions de modulo existent normalement sous : https://old-apprendre.modulo-info.ch/.

Si tu souhaites effectuer une modification sur une ancienne version, tu peux le faire à partir de la branche old-master, et procéder ensuite normalement. Quand le pull-request sera accepté, je mettrai à jour sous https://old-apprendre.modulo-info.ch/.

J'espère que c'est clair.

Merci beaucoup !

Elliot.