JEU DE LA VIE DE CONWAY

Commentaire de petifa le 11/10/2008 13:55:15

Slt tucobouch,
ca m'a l'air cool ton code surtout que le jeu de la vie est assez sympatique comme projet.
Par contre j'ai pas pu executer le programme car il y a des erreurs à la compilation (dues aux @Override):
The method adjustmentValueChanged(AdjustmentEvent) of type new AdjustmentListener(){} must

override a superclass method

pourquoi est-ce que tu n'as pas implémenté ta classe mainFrame avec Component listener
#public class MainFrame extends JFrame implements ComponentListener, AdjustmentListener



Sinon quelques remarques :

monde.java : ligne 88
#public synchronized void tue()
dans cette fonction, tu devrait aussi réinitialiser nbVoisinesVivantes de chaque céllules en le mettant

a 0

monde.java : ligne 107
#return cellules[x][y];
fais attention si x et y ne sont pas valide tu as avoir une exception. Tu devrais mettre :
#if (x >= 0 && x < this.largeur && y >= 0 && y < this.hauteur)
# return cellules[x][y];
#else
# return null;

monde.java : ligne 114
Tu as une variable inutilisée...
#public synchronized void update(double time)

Dans cette fonction tu fais un prétraitement pour compter le nombre de voisins qu'à une cellule,

pourquoi ne pas le faire à la fin de la fonction pour avoir des cellules ayant le bon nombre de voisin.

Dans ton cas si une cellule disparait, les cellules voisines la compte toujours parmis leur voisine.


monde.java : ligne 126
tu créé des variable pour rien ...
#int nb=nbVoisinesVivantes(x,y,c.isEnVie());

monde.java : ligne 183
inutile de créer une variable,
#Cellule c=cellules[xc][yc];
utilise plutot
#if ((cellules[xc][yc]).isEnVie())

cellule.java : Tu devrais mettre un constructeur par défaut.


MainPrg.java : ligne 155, variable non utilisée

Tu aurais aussi pu mettre deux textBox pour spécifier la hauteur et les dimentions et les modifier à sa guise et aussi un bouton pour initialiser aléatoirement la grille plutot que de le mettre en "dur" dans le code.

Sinon code sympa

Commentaire de petifa le 11/10/2008 21:17:01

oki pour tes remarques sauf la numéro 3,
tu devrais tester directement dans les classes c'est plus propre, parce que si tu dois appeler n fois ces fonctions tu devras faire autant de fois les tests.

Après pour la mise à jour des nombre je trouve plus logique qu'ils soient mis à jour en même temps que les valeurs des cellules vivantes, sinon les valeurs ne devraient pas être misent dans les cellules.

Commentaire de petifa le 11/10/2008 23:43:39

OKi, je regarderai a nouveau et plus précisément ton code plus tard, histoire de trouver d'autres choses à améliorer :p

Commentaire de tucobouch le 14/10/2008 21:38:39

Bonjour,
oui effectivement il y avait un prb au démarrage de l'appli... c'est désormais corrigé. Merci

Commentaire de petifa le 15/10/2008 00:09:01

il n'y a pas d'ordre pour appliquer les règles, a un moment donnée toutes les cellules appliquent les règles, c'est pour ça que le nombre de voisin est calculé avant de changer l'étant d'une cellule. Une fois que les voisins des cellules sont comptés on change les états.