Diagramme etat transition

Soyez le premier à donner votre avis sur cette source.

Vue 5 921 fois - Téléchargée 576 fois

Description

bonjour,
mon programme est un simple éditeur d`état il fait pas tous les cas mais le minimum j ai utilise l'héritage de classe shape pour les composants transition ,trans,état,.. un clic pour le crée comme vous pouvez le déplace librement, pour modifier les propriétés clic a droit choisis propriété la vous pouvez modifier le nom => captions et la taille ,enfin si vous voulez tester aller vers "générer procédure" remplissez le nbr états et le nbr transitions vous allez finir par la compilation et l`exécution de votre diagramme

Conclusion :


j'accepte tous les conseils,Critiques, et propositions d'amélioration...

TTLNOTZ,Aragon

Codes Sources

A voir également

Ajouter un commentaire

Commentaires

Bacterius
Messages postés
3793
Date d'inscription
samedi 22 décembre 2007
Statut
Membre
Dernière intervention
3 juin 2016
6 -
Que des conneries ? (oh tu peux le dire c'est pas interdit ^^) Eh ben si c'est ton point de vue comme tu veux ;) Moi je m'en fiche hein c'est pas moi qui ait fait le programme, maintenant si tu refuses les critiques ...
1. Ah parce que tu mets des pièges pour forcer les gens à regarder le code avant d'exécuter ? Alors là tu sais quoi ? Moi j'ai viré le ";" (Delphi m'a aimablement indiqué sa localisation) et j'ai recompilé sans même regarder le code. Eh oui, moi je teste, puis je regarde le code après :/
2. Oui mais quel intérêt de rajouter une propriété Caption à part créer un nouveau composant et alourdir le programme ? Quel sens ça a ?
3. Le fait que ton programme ne soit pas destiné à la majorité qui n'ait pas Delphi, je peux le concevoir (nous sommes sur DelphiFR, tudieu !), mais perds cette habitude quand même, par exemple si tu veux faire une application professionnelle que tu mettras en téléchargement sur internet par exemple.

Si tu refuses les critiques, on va plus t'en faire. C'est ce qu'on appelle être têtu et borné :)
Et essaye d'améliorer l'orthographe c'est pas possible de te lire sérieux ... un peu de respect quand même ... enfin je m'arrête là sinon je vais m'énerver.

Cordialement,Bacterius !
ttlnotz
Messages postés
2
Date d'inscription
mardi 25 décembre 2007
Statut
Membre
Dernière intervention
29 juillet 2009
-
lol ben pour les trois commentaires je te dit merci mais pour le premier c de n'importe quoi que(des co****) et je te pardon parsque je pense que c'été ta 1ere vue du programme
1: la ; c'est 1 test + est une obligation pour les gens du forum a voir un peu le code avant de passe a l'execution mais j ai admis ta phrase *quand meme....:D*
2: je pense que ta raison il faut mettre des demonstrations
oooooolalà le 2 iem tu sais que les composant sont herite de la classe tshape de + caption c pas 1 label :S la procedure paint dans la classe Tetat pardon pour le 3 iem mon programme n ai pas destiner à la majorite que n'ont pas delphi
Bacterius
Messages postés
3793
Date d'inscription
samedi 22 décembre 2007
Statut
Membre
Dernière intervention
3 juin 2016
6 -
Ah oui non dernière chose importante : tu modifies une unité Delphi pour créer ton diagramme, puis tu recompiles. Ca peut sembler original au premier abord, mais c'est NON ! Déjà, le programme ne sert plus à rien sur les machines qui n'ont pas Delphi d'installé (c'est-à-dire une grande majorité des personnes - et ne me dis pas que le fait que l'on soit sur un forum Delphi change quelque chose, c'est une mauvaise idée, surtout pour ceux qui ont l'intention de faire des projets qu'ils voudraient passer à leurs potes ou un truc du genre ... si ils prennent exemple sur ta compilation au run-time) !
NON NON NON ! Et je crois que la plupart des gens seront d'accord avec moi [faites pas les c*ns les gars] ... Bref voilà, je te propose plutôt un fichier "diagramme.dgm", qui serait sous la forme d'un script, et que ton programme interprêterait. Là, ça serait la classe !

De plus, je crois que je sais pourquoi le programme plante lamentablement : quand tu vas compiler ton projet, l'exécutable est toujours ouvert ! Donc, impossible de remplacer l'ancien exécutable par le nouveau : plantage. Tu as testé, dis-moi ? Sérieusement j'ai des doutes ...

Cordialement, Bacterius !
Bacterius
Messages postés
3793
Date d'inscription
samedi 22 décembre 2007
Statut
Membre
Dernière intervention
3 juin 2016
6 -
2eme fournée :
Qu'apporte le composant TEtat à part un TShape agrémenté d'un libellé en son milieu ? Dans ce cas, tu mettais un TLabel par-dessus, ça évitait un composant/une classe de plus ...
Que de redondances dans le code ! Il n'y a qu'à voir les 3 dernières méthodes de l'unité "Unit1.pas" ... c'est limite la même chose : mets une méthode partagée, et utilise un "tag" pour distinguer les différents composants qui utilisent la méthode partagée, et faire des eventuels changements ...
Un petit effort sur le design n'aurait pas été de refus, mais ce n'est pas le plus important pour l'instant.
Ne pas faire de ton code une "usine à gaz" ... je viens d'aperçevoir la TDirectoryListBox cachée à l'exécution ... renseignes-toi sur les fonctions FindFirst, FindNext et FindClose.
Le zip n'est pas bon !! Il ne faut garder, dans ton cas, que les fichiers ".pas", ".dfm", ".dpr" (et "compil.bat" puisqu'il est utilisé par le programme ... mais bon ... voilà).

Je pense que c'est tout pour l'instant ... si c'est ton premier code c'est pas mal quand même :) pensent que les critiques ne peuvent te faire que du mal à très court terme, mais que du bien à long terme ;)

Cordialement, Bacterius !
Bacterius
Messages postés
3793
Date d'inscription
samedi 22 décembre 2007
Statut
Membre
Dernière intervention
3 juin 2016
6 -
Bonjour,
1. tu aurais pu transformer les "composants" etat, transi, transition et voiture par des "classes", ça aurait évité de devoir les installer ...
2. tu aurais pu tester ton code avant le release ! Après avoir installé les composants, je tombe sur une erreur "';' non autorisé avant 'ELSE'" ... quand même ...
3. des coquilles (avertos et conseils du compilo) à n'en plus finir ... faut l'écouter des fois notre bon vieux compilo ...
4. je vois pas trop ce qu'il faut faire dans ton programme, j'ai défini 2 états avec 1 transition entre la 1 et la 2, et comme je savais pas quoi mettre dans les trois derniers champs ("event", "action" et "condition"), j'ai mis "0" partout, et le programme a freezé (une boucle infinie probablement).

Solutions :
1. transformer les composants en classes.
2. enlever le ';' qui n'a pas sa place dans le code.
3. faire taire le compilateur, qui ne cherche qu'à optimiser ton code (dans la plupart des cas ...)
4. mettre une démonstration.

Bonne chance !

Cordialement, Bacterius !

Vous n'êtes pas encore membre ?

inscrivez-vous, c'est gratuit et ça prend moins d'une minute !

Les membres obtiennent plus de réponses que les utilisateurs anonymes.

Le fait d'être membre vous permet d'avoir un suivi détaillé de vos demandes et codes sources.

Le fait d'être membre vous permet d'avoir des options supplémentaires.