cs_Julien39
Messages postés6414Date d'inscriptionmardi 8 mars 2005StatutModérateurDernière intervention29 juillet 2020
-
4 juin 2011 à 17:50
cs_jojolemariole
Messages postés519Date d'inscriptionmercredi 21 mars 2007StatutMembreDernière intervention19 décembre 2016
-
6 juin 2011 à 16:41
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
cs_jojolemariole
Messages postés519Date d'inscriptionmercredi 21 mars 2007StatutMembreDernière intervention19 décembre 201625 6 juin 2011 à 16:41
Salut,
J'abonde dans le sens du commentaire précédent pour ce qui est de la qualité du code.
Encore une fois, par pitié, surveillez les évènements MouseReleased ou bien MouseClicked + MouseDragged plutôt que seulement MouseClicked. 1 fois sur 2 mon "clic" n'est pas pris en compte...
La résolution qui tu as ajoutée fonctionne, mais il y a un petit problème au niveau de l'affichage. Si j'ai fait 6 essais, et qu'elle trouve une solution en 5 essais, mon 6è essai reste visible.
La méthode de résolution que tu as ajoutée n'est pas très efficace, un algo "intelligent" ne dépasse pas les 5 essais. -> 5-guess de Knuth (un Dieu de l'algorithmique).
cs_Julien39
Messages postés6414Date d'inscriptionmardi 8 mars 2005StatutModérateurDernière intervention29 juillet 2020371 4 juin 2011 à 17:50
Le produit final est convenable mais le coté conception n'est pas formidable, tu aurais pu isoler une partie métier (même petite) et créer tes pions en étendant JPanel.
Sinon, tu ne respectes pas les normes d'écriture en Java et tes noms de fonction ne sont pas assez explicites comme cette fonction rd qui retourne un int, sans javadoc...
De plus les parties de code commentées donnent un coté non terminé à ton code.
6 juin 2011 à 16:41
J'abonde dans le sens du commentaire précédent pour ce qui est de la qualité du code.
Encore une fois, par pitié, surveillez les évènements MouseReleased ou bien MouseClicked + MouseDragged plutôt que seulement MouseClicked. 1 fois sur 2 mon "clic" n'est pas pris en compte...
La résolution qui tu as ajoutée fonctionne, mais il y a un petit problème au niveau de l'affichage. Si j'ai fait 6 essais, et qu'elle trouve une solution en 5 essais, mon 6è essai reste visible.
La méthode de résolution que tu as ajoutée n'est pas très efficace, un algo "intelligent" ne dépasse pas les 5 essais. -> 5-guess de Knuth (un Dieu de l'algorithmique).
4 juin 2011 à 17:50
Sinon, tu ne respectes pas les normes d'écriture en Java et tes noms de fonction ne sont pas assez explicites comme cette fonction rd qui retourne un int, sans javadoc...
De plus les parties de code commentées donnent un coté non terminé à ton code.