Discussion :

[Loupsio]

voila un logiciel entièrement de ma création, c'est un Mastermind tel qu'ils sont dans les concours infirmiers , c'est a peu près le plus difficile que je sache faire
je l'ai fait testé, ils sont tous résolvables, et j'aimerais avoir votre avis et savoir ce qui peut être améliorée au niveau du code et de la presentation,

www.megaupload.com/?d=NXV25M4K

[Modulpro]

Salut,

J'ai téléchargé l’exécutable : lors du premier choix, "r" affiche bien les règles, mais "j" et "q" provoquent la fermeture de la fenêtre (erreur non gérée ou fin du programme ?). Impossible donc de jouer.
Je ne sais pas sous quoi tu as compilé ton code, moi je suis sous Windows XP 32.

Salut

PS : serait il possible d'avoir le code plutôt que l'exécutable ?

[Loupsio]

c'est bizarre chez moi ca marche, je suis sur windows vista et j'utilise free pascal I.D.E 2.4.2
voici le code
www.megaupload.com/?d=RXKTDNZH

et un nouvel upload de l'executable
www.megaupload.com/?d=RSX08TQQ

edit:
j'ai telecharger l'ancien lien et il s'arrêtait bien aussi quand je faisais "jouer", j'ai donc essayé ce nouveau lien et la ca marche,
j'avais du changer quelque chose, ce que je ne comprend pas c'est que entre les 2 fichiers je n'ai rien fait pour corriger ce bug et le 2 eme marche... enfin bon le principal c'est que ca marche

[Alcatîz]

Bonjour,

N'oubliez pas qu'une application de téléchargement vous permet de proposer vos contributions à toute la communauté :
http://pascal.developpez.com/telecha...80/Free-Pascal

[Modulpro]

Salut,

Bon l'exécutable fonctionne apparemment bien. J'ai téléchargé le code, je l'ai regardé vite fait (sans trop entrer dans la logique du jeux), mais voila quelques conseils pour en améliorer la lisibilité :

- Utiliser des procédures pour regrouper une série d'instructions indépendantes du programme lui même. Par exemple toute la partie "jouer" pourrait être rassemblée dans une procédure. Cela permet de simplifier le corps du programme, et d’effectuer plus facilement d'éventuelles modifications.

- La structure

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
if choix = 'j' then {...}
else if choix = 'r' then {...}
else if choix = 'q' then {...}
else {...}

peut être remplacée par quelque chose de la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
Case choix of
  'j' : begin {...} end;
  'r' : begin {...} end;
  'q' : begin {...} end;
  else begin {...} end;
end;

- De même, les boucles de la forme

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
i := 1;
  repeat
     {...}
     i := i + 1;
  until i = 4;

peuvent êtres remplacées par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
For i:=1 to 3 do 
  begin
    {...}
  end;

- enfin l'indentation du code peut être améliorée (elle n'est pas facile à mettre en place ici à cause de la longueur du programme, mais ce problème va s'arranger si tu le découpes en procédures ou fonctions).

Voila, c'est tout ce que j'ai vu pour l'instant.

[Loupsio]

merci des conseil, je re-posterais la nouvelle version dès que j'aurais fais les modifications

[Loupsio]

voila j'ai changé la structure
ainsi que les boucles
il me reste plus qu'a améliorer l'indentation et les procédures, mais je vois pas ce que sont les procédures

executable:
www.megaupload.com/?d=RSX08TQQ

code:
www.megaupload.com/?d=RXKTDNZH