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

Ajout de la documentation #4

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

Conversation

StaloneLab
Copy link
Member

Petite PR parce que j'avais du temps hier ; j'ajoute à ce repo la documentation du dernier atelier. Cette PR devrait rester ouverte pour review, AMHA ; j'ai un script de rendu HTML en local, mais il dépends d'un fonctionnalité pas encore intégrée dans ZMarkdown donc pas possible de pousser ça ici pour le moment. Dans la mesure du possible, si vous corrigez le Markdown, je ferais un export HTML global avant de merge.

@Situphen Situphen requested review from AmauryCarrade and EtoileFilante and removed request for AmauryCarrade October 6, 2019 21:45
@StaloneLab
Copy link
Member Author

@EtoileFilante : Aucun changement à faire pour toi ? Tu avais pourtant l'air de vouloir changer des choses lors du dernier ZdC.

@EtoileFilante
Copy link
Member

Ça me paraît déjà pas mal, ensuite je n'ai pas eu le temps de reprendre mes différents points encore. Si besoin je ferai une autre PR plus tard ;)

Copy link
Member

@AmauryCarrade AmauryCarrade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Globalement très bon. J'ai fait plein de remarques pour que ce soit encore mieux :D

Certaines concernent aussi le code. Aussi je suis bien entendu ouvert au débat sur ce que je propose :) .

documentation/md/les-bases-de-python.md Outdated Show resolved Hide resolved
documentation/md/les-bases-de-python.md Outdated Show resolved Hide resolved
documentation/md/les-bases-de-python.md Outdated Show resolved Hide resolved
Comment on lines +59 to +67
Enfin, on rappelle les deux types de variables :

- **locale** : disponible uniquement dans la fonction en cours ;
- **interne** : disponible partout où l'objet concerné est présent (pratique pour passer une variable de `initialisation` à `boucle`, par exemple).

Les deux types de fonctions sont de même nature :

- **principale** : disponible pour le fichier en cours ;
- **interne** : peut être utilisée par le jeu lui-même (c'est le cas de `boucle`, par exemple) ou utilisée depuis la variable jeu (`jeu.quitter()` par exemple).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peu clair, je trouve. En fait même moi je ne trouve pas ça clair : on parle bien des variables locales et des attributs d'objets ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clarification souhaitée, en effet, mais je ne sais pas comment m'y prendre.

On parle des variables locales / globales que je regroupe en "locales" ici, et des attributs (variables dites "internes"). Même chose pour les fonctions.

Je ne sais pas à quel point l'abstraire, donc pas bien sûr de comment l'aborder.

documentation/md/les-bases-de-python.md Outdated Show resolved Hide resolved
print("Collision Serpent-Pomme")
```

Vérifiera s'il y a collision entre la pomme et la variable nommée `morceau.position`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peu clair potentiellement quand on ne connaît pas du tout Python (d'où vient ce .position ?).


## Déclaration de l'objet Serpent

Après avoir importé la bibliothèque Zeste de Code, tu auras accès aux fonctions concernant le serpent. Une nouvelle instance de serpent devra ensuite être crée, préférentiellement à l'intérieur du jeu :
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le mot instance n'a jamais été vu avant. Quid ?

Copy link
Member

@Situphen Situphen Oct 7, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+ correction orthographique : "crée" -> "créée"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est vraiment gênant ? Le mot instance me semble assez transparent ici : "une nouvelle instance du serpent" = "un nouveau serpent".

@Situphen : corrigé.

documentation/md/fonctions-du-serpent.md Outdated Show resolved Hide resolved
jeu.serpent.PARTIES.QUEUE
```

Notons que lors de l'itération, les parties sont ordonnées dans le sens inverse (et si tu ne comprends rien à cette phrase, reviens plus tard).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le plus tard est à peine plus bas. Peut-être vaudrait-il mieux reformuler cette partie, ou retirer cette phrase au profit du paragraphe « Position du serpent » (ou y laisser un pointeur : « on verra un peu plus bas comment s'en servir »…)


# Position du serpent

Un itérateur existe et permet d'obtenir tous les morceaux de serpent dans l'ordre allant de la queue à la tête, il s'agit de `serpent.morceaux(taille)` ; le seul argument est la taille du serpent, ou plus simplement le nombre de morceaux à prendre. Pour afficher les positions successives du serpent :
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Par rapport au code : pourquoi demander la taille en paramètre quand elle est stockée dans l'objet de toute façon ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour pouvoir itérer sur une partie des morceaux seulement ; aucune idée de pourquoi je voulais faire ça, mais clairement c'était à dessein. TODO: voir si c'est utilisé.

Co-Authored-By: Amaury Carrade <[email protected]>
@StaloneLab
Copy link
Member Author

J'ai corrigé les problèmes qui me semblent parfaitement justifiés, et laissé un commentaire sur les autres pour débattre.

Pour la position et la modification du code, je fais ça dans un prochain commit avec le "plus tard" que j'ai zappé dans celui-ci.

- `mon_jeu.hauteur`, qui contient la hauteur de la fenêtre ;
- `mon_jeu.evenements`, qui contient la liste des événements déclenchés depuis le dernier tour de boucle.

La variable `evenments` nous intéressera particulièrement, et nous pourrons vérifier si un certain événement est déclenché en utilisant une condition :
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
La variable `evenments` nous intéressera particulièrement, et nous pourrons vérifier si un certain événement est déclenché en utilisant une condition :
La variable `evenements` nous intéressera particulièrement, et nous pourrons vérifier si un certain événement est déclenché en utilisant une condition :

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.

4 participants