banc de poissons

Bonjour à tous !
J'ai eu une idée : de faire un programme modélisant (au moins grossièrement) un banc de poissons : avec plusieurs poissons (pixels) et un pixel que l'on peut faire bouger sur tout l'écran (je l'appelle pointeur).
En fait je programme sur TI mais je ne me suis mis au C que depuis quelques semaines : j'ai déjà réalisé ce programme en basic mais j'ai abandonné le projet à cause de la vitesse...

Le principe est le suivant :

Si un poisson détecte le pointeur à une certaine distance, définissable, il s'en éloigne jusqu'à être au delà de cette distance.
Le groupe de poissons se forme de cette façon : chacun va en direction du poisson le plus proche et s'arrête à une distance définissable aussi.

Je ne sais pas si je suis très clair, aussi je joins le code que j'ai élaboré (pour l'instant, il n'y a qu'un seul poisson). Il comporte une erreur, le problème c'est que je ne sais pas laquelle. Si quelqu'un pouvait m'aider à la trouver, ainsi que pour la suite du programme...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82

short xa,ya,xta,yta,yy,xx,y,x,yt,xt; /* Variables de déplacement */ char get; /* Variable de test clavier */ short cy,cx,tecom; /* Variables de comptage */   moduldep() /* Fonction de déplacement */ { /* du pointeur */ get=ngetchx(); if((get==81)&&(y>2)) y-=2; if((get==82)&&(x>2)) x-=2; if((get==84)&&(y<98)) y+=2; if((get==88)&&(x<158)) x+=2; }   bacomp() /* Fonction de comptage des */ { /* distances */ cy=ya-y; cx=xa-x; if((cy<=10)&&(cx<=10)) { tecom=1; if(cy<0) { if(ya<99) ya++; } else { if(ya>1) ya--; }   if(cx<0) { if(xa<159) xa++; } else { if(xa>1) xa--; } } else tecom=0; }   // Main Function void _main(void) { clrscr(); xa=xta=79; ya=yta=49; x=y=xt=yt=1; DrawPix(xa,ya,A_NORMAL); DrawPix(x,y,A_NORMAL);   dep: moduldep(); DrawPix(x,y,A_XOR); DrawPix(xt,yt,A_XOR); yt=y; xt=x;   ba: bacomp(); if (get=8) goto zzz; if(tecom) { DrawPix(xa,ya,A_XOR); DrawPix(xta,yta,A_XOR); yta=ya; xta=xa; goto ba; } goto dep; zzz: }

---

Citation:

---

Envoyé par virgiledu95

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32

// Main Function void _main(void) { clrscr(); xa=xta=79; ya=yta=49; x=y=xt=yt=1; DrawPix(xa,ya,A_NORMAL); DrawPix(x,y,A_NORMAL);   dep: moduldep(); DrawPix(x,y,A_XOR); DrawPix(xt,yt,A_XOR); yt=y; xt=x;   ba: bacomp(); if (get=8) goto zzz; if(tecom) { DrawPix(xa,ya,A_XOR); DrawPix(xta,yta,A_XOR); yta=ya; xta=xa; goto ba; } goto dep; zzz: }

---

---

Un Force 10 comme ça, d'entrée de jeu. Bravo!

http://emmanuel-delahaye.developpez.com/goret.htm

Yeah terrible ton echelle.
Desormais quand je ferai un code, je me baserai sur ca pour faire un code goret level 0. ^^

Je trouve quand meme l'echelle un peu dure, mais ce n'est que mon avis.

Echelle 8 : plus qu'un return par fonction

J'ai deja vu des codes a toi avec plusieurs return...

Citation:

---

l'expression 'comme un goret' signifie 'salement', 'de façon crade' etc.

---

Un code peut être crade et efficace à la fois...

Mais un code illisible est bon pour la poubelle

je n'arrive pas à savoir quel est le niveau le pire et le mieux... parce que je ne sais pas si je dois prendre cette remarque comme un compliment ou comme une critique... et si c'est une critique merci d'argumenter

Le pire c'est le Force 10...

Et pourquoi ben parce que y'a des goto dans tous les sens, tout est dans le main, ya des variables globales en pagaille, pas d'indentation...

Ton code est très mauvais, et regroupe une bonne partie de ce qu'il ne faut absolument pas faire en C.
Mais tu es débutant, tu as programmé sur ta calculette en basic à coup de goto donc celà ne peut donner que de mauvais résultats.
Donc, avant de te lancer dans un programme, apprends l'algorithmique sérieusement, ouvre un bon bouquin de C (pas ceux de Delannoy :aie: ) et mets toi au boulot.
Reviens dans quelques temps nous montrer un code un peu plus correct (tu te demanderas alors comment tu as pu écrire de telles horreurs) et on sera là pour résoudre tes problèmes.
Bon courage ;)

Merci : au moins, ça a le mérite d'être clair.
Je vais faire ce que tu me conseille : finir mon livre de C.
Je mets le forum en favoris et je reviendrai plus tard...
Si par contre, en attendant, tu pouvais me signaler les erreurs les plus grosses dans mon code pour que je bosse dessus, ça m'arrangerait bien !

Skyrunner te les a enumérées :
=> La présentation : le code est mal indenté, défaut peut-être mineur, mais une bonne présentation aide à la compréhension du code, à la lecture de la structure du programme.

=> Beaucoup plus grave (et même rédhibitoire): un code bourré de goto est complètement illisible, montre une mauvaise étude du problème, une méconnaissance totale de l'algorithmique et fait que personne ne voudra lire ton code pour essayer de la comprendre, (tu as vu les réactions).

=> Méconnaissance du C :
void _main(void) : jamais vu !!! où as-tu trouvé ça, si c'est dans ton bouquin, direction poubelle ! Il y a de bons livres d'apprentissage, regarde sur le site, on t'en conseillera de bons.

Les fonctions ne retournent pas de valeurs, ton compilo devrait le signaler

Utilisation de variables globales, à proscrire absolument, il faut prendre tout de suite les bonnes habitudes, elles sont rarement vraiment nécessaires et dans 99% des cas, on peut s'en passer.

==> Enfin, pour le forum, publication d'un code non compilable, oubli des fichiers de déclarations de fonctions (.h), aucune explication sur ce que sont censées faire ces fonctions, manque de précisions : Mon code comporte une erreur, mais tu ne nous donnes aucune pistes sur le comportement du programme, et de ce que tu en attendais.

Citation:

---

Envoyé par virgiledu95

Merci : au moins, ça a le mérite d'être clair.
Je vais faire ce que tu me conseille : finir mon livre de C.
Je mets le forum en favoris et je reviendrai plus tard...
Si par contre, en attendant, tu pouvais me signaler les erreurs les plus grosses dans mon code pour que je bosse dessus, ça m'arrangerait bien !

---

En tout cas bonne attitude. Bravo.

Désolé pour ces oublis
mon compilateur est TIGCC C and ASM SDK Version 0.96 Beta 6
je mets en include :

Code:

---

#include <tigcclib.h>

---

mais pour ce qui est de

Code:

---

1 2 3 4

void _main(void) {   }

---

, le compilateur me l'insère automatiquement lorsque je crée un fichier !

Et tu as choisi ce compilateur comment?
C'est juste une question.

Si tu veux developper en C standard, tourne toi vers un IDE type Code::Blocks et un compilateur type GCC.

Un tres bon livre.
http://c.developpez.com/livres/#L2100487345

LA référence.

C'est du C pour calculatrice, il vaut mieux abandonner à mon avis, et faire des choses sérieuses sur un vrai compilo, sinon tu vas prednre rapidement de mauvaises habitudes.
Extrait de la DOc:

Citation:

---

III:\ La fonction principale :
Comme nous l'avons déjà expliqué au Chapitre précédent, la fonction principale doit systématiquement être nommée _main.
Cette fonction est déclarée comme étant de type [FONT=Courier New]void[/FONT]. (Elle est précédée du modificateur de type [FONT=Courier New]void[/FONT].). Cela signifie que cette fonction ne renverra aucun résultat. De plus, elle n'admet aucun argument, puisque l'argument (placé entre parenthèses à la suite du nom de la fonction est de type [FONT=Courier New]void [/FONT]lui aussi).
Ceci est parfaitement logique : puisque cette fonction est la fonction principale, c'est elle qui est active dès le lancement du programme, et elle ne peut donc pas être lancée par une autre fonction.
En C, passer des arguments à un programme en le lançant est possible, mais cela ne se gère pas de la même façon dans le code source qu'en TI-BASIC.

---

La norme C impose que la fonction main retourne un résultat, donc apprendre le C sur les calculettes de ce type est une absurdité car ton code n'est pas portable.

Justement, la fonction principale d'un programme en C sur TI s'appelle _main et non main pour la bonne raison qu'elle n'est pas standard.

En clair, "ce n'est pas un bug, c'est une feature".

Mon conseil: Pour les TI, va sur un autre forum, celui-ci est surtout conçu pour les architectures plus standards.
Par exemple, tu peux aller sur http://www.tigen.org, l'un des gourous les plus intégristes de TI-GCC s'y trouve.



PS: ça reste du code super-gore, même si je ne suis pas d'accord avec tous les critères d'Emmanuel...