LE TRADITIONNEL JEU DU PENDU :)

Commentaire de John Dogget le 20/04/2008 19:36:18

Lu.

Quand tu n'as qu'une instruction dans une boucle, ce n'est pas utile de mettre un begin et un end autour de cette instruction.

Exemple :

for I := 0 to Length(usedword) - 1 do
begin
     WinW := WinW + WState.Cells[I, 0];
end;

-> Tu peux enlever le begin et le end, le compilo comprendra très bien ce que tu veux faire.

for I := 0 to Length(usedword) - 1 do
     WinW := WinW + WState.Cells[I, 0];

Ca reviens assez souvent dans ton code.

;)

Commentaire de John Dogget le 20/04/2008 19:40:04

Arf, t'as oublié de mettre les images dans le zip (pendu1,2 etc)

Commentaire de Bacterius le 20/04/2008 20:03:10

J'ai téléchargé le zip, il y a bien les images dedans. As-tu pensé à les mettre dans le dossier *\Images\
Ou * représente le dossier ou sera placé Pendu.exe, et éventuellement le code source ?
Il faut mettre dedans toutes les images jpg. ;)

Cordialement, Bacterius !

Commentaire de Bacterius le 20/04/2008 20:28:41

J'ai juste un petit problème avec la StringGrid, comment enlever le vilain surlignement bleu sur la première cellule ? je n'y arrive pas :(
Vous avez des idées ?
Merci d'avance

Cordialement, Bacterius !

Commentaire de Bacterius le 20/04/2008 21:10:08

Super ! Merci Cantador, ca change carrément tout le look de l'application :)

Cordialement, Bacterius !

Commentaire de cirec le 21/04/2008 01:04:50 administrateur CS

il y a plein de petites erreurs dans ton code
l'utilisation de Application.Terminate est à proscrire
les procédure Button4Click à Button29Click peuvent et doivent se résumer à une seule procedure.

Il y a encore d'autre code répétitif que l'on peut supprimer.

on ne met qu'une seul instruction par ligne

J'ai fait une grande partie des modifications (mais pas tout^^)

si tu veux voir ce que ça donne donne moi une adresse Mail en MP et je t'enverrai le code ;)

en dehors de ces petites erreurs c'est pas mal tu progresses à petits pas ^^

Commentaire de JulioDelphi le 21/04/2008 10:38:08 administrateur CS

oula, je viens de voir une listbox minuscule en "visible := false" qui traine ! Si tu ne l'affiche pas, utilise une TStringList a la place !

Commentaire de Bacterius le 21/04/2008 18:03:46

Ah aussi Cirec je t'envoie mon adresse mail par mp, et je regarderai ton optimisation après avoir fait la mienne, pour pouvoir comparer :) pour voir si je progresse bien !

Cordialement, Bacterius !

Commentaire de Bacterius le 21/04/2008 20:36:01

Merci beaucoup Cirec j'ai reçu ton email, et j'avais terminé de faire mon optimisation. On a eu a peu près les mêmes idées, sauf sur l'histoire des tags. Et j'ai procédé un peu différemment pour la vérification des images, au début du programme.

Ton code :

if Not FileExists(Dirr + 'Images\Pendu_1.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_2.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_3.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_4.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_5.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_6.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_7.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_8.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_9.jpg') then
    FileError;
  if Not FileExists(Dirr + 'Images\Pendu_10.jpg') then
    FileError;


Mon code :

(Je déclare une variable "I" locale)

for I := 1 to 10 do if FileExists(Dirr + 'Images\Pendu_' + IntToStr(I) + '.jpg') = False then EventImageNotFound;

J'ai également placé cet évenement dans OnFormShow
Parce que j'ai pensé que si on le place dans FormCreate, il ne pourrait pas fermer l'application car elle n'aura pas commencé ^^'
Car dans le fichier projet, on crée les forms avant Application.Run
Mais je suppose que ca doit marcher dans FormCreate ^^

Je posterai la version optimisé bientôt

Au fait Cirec ^^' quand j'ai ouvert ton code optimisé, il était imbibé de ta version de Delphi, et donc tous les composants ont disparu ^^ sauf le shape et 2 boutons ... Mais j'ai quand meme le code :)
Ca me fera réécrire tout :p

Cordialement, Bacterius !

Commentaire de Bacterius le 21/04/2008 20:45:52

C'est bon j'ai téléchargé WinZip 9 :) J'espère qu'il les prend en charge

Cordialement, Bacterius !

Commentaire de Bacterius le 21/04/2008 22:03:22

Oui pas mal de boulot
J'ai posté une version intermédiaire, je continue l'optimisation néanmoins
La capture d'écran n'a pas fonctionné, c'est peut-être parce que c'est le même nom de fichier, je vais changer une lettre et réessayer.
J'ai oublié de préciser que cette version prenait en charge les accents (la lettre 'e' prend les accents aigu et grave) pas les circonflexe, je l'ai oublié - prochaine version.

:) L'optimisation continue

Cordialement, Bacterius !

Commentaire de Mowglinaute le 24/04/2008 18:03:06

Il est très bien ton jeu, je te donne la décoration du Grand Pingoin, allez bonne continuation à ta programmation.

Dr. Joseph

Commentaire de Bacterius le 25/04/2008 20:37:27

Re, je ne pourrai peut être pas poster le zip aujourdhui, Internet Explorer m'envoie un message 'Out of memory at line : 6', et tous les mots clés renvoient un message rouge expliquant les contraintes (3-100 caractères, 1 tiret, 1 espace) alors qu'elles sont respectées.
Désolé pour le double post, j'ai tenté de mettre le zip 2 fois.
Pourvu que ca marche ^^

Cordialement, Bacterius !

Commentaire de Bacterius le 28/04/2008 18:09:16

Bonjour,
je ne comprends pas bien ta question : une démo ?
normalement tu devrais pouvoir compiler et éxecuter sans problème. Mais si tu n'y arrives pas, je peux t'envoyer le zip avec l'éxecutable inclus par mail, envoie moi une adresse email par message privé.

Merci :)

Cordialement, Bacterius !

Commentaire de JulioDelphi le 02/05/2008 20:01:55 administrateur CS

ya 1001 choses à modifier dans le code ^^
Je suis en train d'en faire l'inventaire ... et c'est long :p
Mais pour un premier programme, c'est assez sympa donc je prends le temps de temps en temps de regarder ligne par ligne le code.
Je posterais les modifs à apporter des que j'aurais fini :p
En attendant, continue a le mettre à jour !

Commentaire de JulioDelphi le 03/05/2008 10:29:27 administrateur CS

ha ... bon bah j'arrête alors.

Commentaire de cirec le 03/05/2008 12:59:56 administrateur CS

je n'en doute pas mais les corrections étaient servies sur plateau d'argent avec explications etc.

c'est juste mais tout le monde sait bien que c'est pas toujours facile de se plonger dans le code d'une autre personne (surtout avec plusieurs instructions par ligne) ... donc quand on le fait on s'attend quand même à ce qu'on tienne compte.

Voilà c'est tout c'est dommage de revoir les mêmes erreurs.

Commentaire de cantador le 03/05/2008 13:45:22

@cirec:
autre sujet :
ce code est classé sous :
Jeu, Pendu, Débutant, Image, Traditionnel
et quand on tape Pendu dans le moteur de recherche il ne retrouve rien...

???

Commentaire de cantador le 03/05/2008 15:14:21

@cirec:
oui j'avais vu pour google (on trouve tout sur ce machin..)
Pour le moteur du site, autrement dit il faut taper exactement le titre de la source ??

et quand on voit de temps en temps des titres de source plus ou moins alambiqués.

j'ai d'ailleurs à peu près le même souci pour retrouver des discussions sur le forum..

Bref, je m'interroge :

Le moteur de recherche, il sert encore à qlqchose ?