Citer : Posté le 17/01/2016 19:47 | #

---

Ce serait bien de dire en quoi consiste le jeu et pourquoi pas un hastebin du code histoire de pas avoir à dl tout le truc, j'ai plus le SDK et j'ai pas le temps de le remettre mais j'aimerais bien voir ton code

http://hastebin.com/ pour partager ton code, tu entres le code et tu fais CTRL + S et l'URL à partager apparaît en haut

---

Citer : Posté le 17/01/2016 19:55 | #

---

Je passe l'éponge sur tout le bordel qu'il y a avant le code de ton jeu, quelqu'un de plus compétent que moi te dira si c'est bien ou non x)

Sinon, mais où est ta fonction principale ? Elle a disparu ? Après ça, l'indentation du code lui même semble correcte mais le code peut être optimisé

---

Citer : Posté le 17/01/2016 20:11 | #

---

Alors 1)



2)

Heeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeu, tu as vraiment besoin de 4 conditions pour faire 1 seule chose

3)

C'est des p******* de b******* de c********* de f*** de p*** de sa maman de variables globales! à proscrire immédiatement, NOW, agora, nyní, acum, в настоящее время (je m'assure que le message passe )

4)

J''espère que je n'aurais jamais à utiliser ce code Je ne comprendrais pas l'utilité de la fonction seulement avec le prototype

5)

C'est un peu comme avoir accès au paradis et goto vers l'enfer

J'espère que je n'ai pas été trop gentil sévère mais la programmation requiert une discipline, une rigueur à prendre le plus tôt possible

Citer : Posté le 17/01/2016 20:19 | #

---

J'ai vu mieux comme excuse pour pas regarder son code :P
Alors j'ai pas testé (faut que je lance la machine virtuelle de Windows ça mets 3 ans ça me gonfle) par contre j'ai regardé ton code, je te fais une liste des ce qui va ou pas :
- Tu as deux codes pour gérer le clavier, une partie dans TestLib.c et une autre dans usefull.h, c'est l'un ou l'autre on va dire

- D'ailleurs évite de mettre tout ce pavé au début de TestLib.c, tu crées un nouveau fichier gros_pave.c et tu le mets dedans. Ensuite tu fais un noveau fichier gros_pave.h où tu mets les headers des fonctions de ce gros pavé, tu fais #include "gros_pave.h" au début de TestLib.c et ça fonctionne

- mettre des sauts de ligne pour aérer, par exemple entre les #include et les #define

- quand tu crées une fonction, tu dois renseigner ce quelle renvoie (int, char, float, ...). Si elle ne renvoie rien, tu mets le type void, qui signifie "rien" justement. Pas la peine de faire int ma_fonction() ... return 0;

- quelques petits problèmes d'indentation

- evites les trucs comme çaà moitié sur une ligne, à moité sur 2 lignes
Un truc comme celà est plus propre


- Après c'est un peu plus du chipotage là mais c'est toujours bon à entendre
Les espaces irréguliers c'est moyen


- Les variables X et Y sont définies en local (dans AddIn_main) alors que X2 et Y2 sont déclarées en globale

- Je vois pourquoi tu as fais cela, en fait il faut les déclarer en même temps que X et Y et quand tu appelles ta fonction TPperso2(); tu fais TPperso2(&X2, &Y2);
La ça envoie l'adresse de tes deux variables
puis dans ta fonction TPperso2(); tu utilises * pour modifier X2 et Y2. Au début c'est assez difficile je te l'accorde, on appelle cela les pointeurs.

- tu inclues fixlib.h en plein milieu d'un truc, il est perdu le pauvre

-Ça c'est très bien

- N'oublies pas que c'est GetKey() qui affiche la VRAM à l'écran, sinon il faut utiliser Bdisp_PutDisp_DD()

Citer : Posté le 17/01/2016 20:50 | #

---

Pour le 2), au lieu de faire 'if A, then B, if C, then B, if D, then B', tu fais 'If A OR C OR D, then B', c'est plus logique (j'ai mis OR parce que je me rappelle plus de l'opérateur genre |, me punissez pas svp )

Pour le 3), faut vraiment que tu lises le début du cours, je l'ai appris il y a quelques temps, les variables au début sont des variables globales, utilisables dans tous les fichiers et c'est sale d'après les expérimenté et d'après OC donc tu dois inclure les variables seulement là où tu veux les utiliser, dans une fonction ou ailleurs

Pour le 4), il essaye de t'expliquer qu'on ne comprend pas ce qu'est cette fonction si tu ne lui donnes pas plus d'explications que ça (un nom explicite, des commentaires), je pense.

Et le 5), je laisse quelqu'un d'autre répondre plus précisément vu qu'Intelligide le fait pas

---

Citer : Posté le 17/01/2016 21:26 | #

---

4) Tu comprends bien parce que c'est petit et que tu viens de le coder mais dans un gros projet, ce genre de trucs s'oublient vite

---

Citer : Posté le 18/01/2016 12:24 | #

---

On te soutient, Matt \o/

Citer : Posté le 19/01/2016 22:03 | #

---

Quand il n'y qu'une seule ligne, on peux aussi le mettre sur une ligne. De plus, tu connais surement l'instruction Isz et Dsz en basic. En C, il existe des équivalents :

---