YOYOCUT 2 - DÉCOUPE FICHIER [VISUAL C++ 2005 EXPRESS]

Commentaire de yoyo269 le 25/04/2006 07:21:15

J'attends vos messages.
N'hésitez pas à me données des idées d'évolution, même si elles peuvent vous paraître farfelues.

YOYO, @+.

Commentaire de yoyo269 le 27/04/2006 19:55:01

Ok! je vais y jeter un oeil.
Merci pour le compliment, c'est sympa.
Une note serait la bienvenue.
PS: j'avoue que j'avais en tête de faire qq chose de mieux que CutKiller en personne ! LOL.

YOYO, @+.

Commentaire de yoyo269 le 27/04/2006 20:12:33

Tant mieux, mais si t'en vois un (ou plus lol) touche en moi un mot.

YOYO, @+.

Commentaire de yoyo269 le 27/04/2006 20:55:06

Ah oui merci. Je vais voir si je vais le faire.
En tous cas pas tout de suite, je suis sur autre chose en ce moment.

YOYO, @+.

Commentaire de SAKingdom le 18/11/2006 21:00:42

Salut. J'ai pas le temps de tout vérifier (trop gros) mais je remarque quelque petites choses en partant:

if(strcmp(Argument, ""))
ce serait pas mieu de faire
if(!Argument[0])
Ou quelque chose dans le genre?

Ensuite:

if(Argument[0] != '\"')
{
char Arg[MAX_PATH];
strcpy(Arg, "\"");

strcpy pour une seul valeur? Pas mieu de faire (à la limite): Arg[0] = '\"'; Arg[1] = 0; ???

Pour faire plus propre, tu peux faire des macro aussi:

CPYCHAR(dest, c) dest[0] = c; dest[1] = 0

Comme je dis, j'ai pas trop le temps donc je n'est pas le temps de confirmer ce que je dis ni d'aller plus loin pour l'instant.
À toi de voir.
Tien moi au courant.

Commentaire de SAKingdom le 18/11/2006 21:48:03

Bon, je me suis trompé.

Par ton if(strcmp(Argument, "")), je suppose que c'est pour tester si il existe quelque chose dans la chaine. Donc c'est pas:

if(!Argument[0])
comme je disais mais
if(Argument[0])

Commentaire de SAKingdom le 19/11/2006 17:29:20

Bon, je remarque encore quelque strcmp(.., "")

Mais là, je vien de tomber sur une horreur:
À partir de la ligne 7776, tu gère ton tableau Fichier en te situant dedans avec sa grosseur. Le problème c'est que tu apelle À CHAQUE FOIS strlen. Tu parcoure toute la chaine à chaque fois et je remarque un nombre important de ce genre de manipulation. À tu oublier de mettre à jour cette section? ;)

Commentaire de yoyo269 le 19/11/2006 18:18:22

SAKingdom >> llllllllllllloooooooooollllllllllllllll !!! olala le boulot ! heureusement que tu me l'as dit. J'avais cette fonction juste pour qu'elle fonctionne au début (avec copier coller etc...) et j'ai oublié de la retoucher. Je vais faire ça de suite parce que c'est vrai que le nombre de strlen() est impressionnant en s'y repenchant dessus. MERCI !!!

wizard512 >> merci pour ton commentaire c'est sympa ! Je précise d'ailleurs que YoyoCut 2 est téléchargeable sur telecharger.com si vous voulez en parler à votre entourage. Une version 3 est prévue mais le temps libre décidera ou non si elle verra le jour (je l'espère entous cas).

YOYO, @+.

Commentaire de SAKingdom le 23/11/2006 17:21:35

Oufffff. Le code est très gros. Je progresse (lentement).

J'ai remarqué une ptite chose pas très importante mais bon.... ce serais interessent de corriger pour une future version.

Dans les procédures de tes contrôles, pour WM_PAINT, tu fais hdc = GetDC(...) et ReleaseDC.

Cependant, je crois que BeginPaint et EndPaint font la même chose:
hdc = BeginPaint(...);
EndPaint(...);

Bon c'est pas grand chose mais ça te sauve 2 appels de fonction pour chaque "repainturage" de chaque contrôle.

Commentaire de SAKingdom le 24/11/2006 00:11:06

Ben c'est à ça que je pensais mais je n'étais sûr de rien (peut-être que ça aurais été utile pour une autre version ou ça corriger quelque chose, qui sais).

Pour ce qui est de la motivation, non j'en ai pas beaucoup je dois dire. Je lis petit bout par petit bout. Pour être franc, je ne comprend pas 95% du programme (son fonctionnement). C'est pour ça que je repère seulement ces petites choses. Étant donné que ton code est très gros, je ne peux pas avoir une vue d'ensemble et repérer les gros bugs potentiels (par exemple, oublier de fermer une instance ou de libérer la mémoire ou mauvaise manipulation d'un poiteur etc.).

Pour ce qui est de commenter la source, bah pas la peine. C'est vraiment beaucoup trop gros et de toute façon, je ne crois pas que ça changera quelque chose. Soi tu comprend le code (avec ou sans commentaire) soi tu le comprend pas (de tout façon, ce n'est pas pour débutant).

Mais j'y pense, j'ai pas encore donné de note moi. Allez, va pour un 10.

Commentaire de vecchio56 le 20/12/2006 00:26:33 administrateur CS

Effectivement, SAKingdom, tu es très courageux
Franchement un seul fichier contenant tout le code, c'est pas très malin
Je sais pas si t'a fait une petite phase d'analyse et de conception (a priori je dirais que non), mais ca aurait été utile.
La c'est très difficile de comprendre l'architecture de ton projet

Enfin si t'arrive à te comprendre c'est l'essentiel, mais ne reste pas trop longtemps sans toucher à ton projet, sinon un beau jour tu va vouloir le modifier, et là tu ne va plus rien comprendre

Commentaire de yoyo269 le 20/12/2006 09:36:03

Salut vecchio56,

c'est vrai que cette méthode n'est pas la mieux, mis on va dire que c'est ma façon de coder bien à moi ! ;)
Je comprends très bien que mon code ne soit pas très lisible pour quelqu'un d'autre que moi, mais moi avec l'habitude je me relis sans aucun souci. Même en reprenant un code plusieurs mois plus tard.
Je vais quand même essayer de changer mon habitude, pour utiliser plusieurs .cpp.
Sinon qu'appelles-tu une phrase d'analyse et de conception ?
Ca m'intéresse beaucoup !

Merci d'avance pour ta réponse et merci aussi pour tes remarques ! :D

YOYO, @+.

Commentaire de CHAR As Human le 27/12/2006 23:05:33

400 fois mieu que cut killer

Commentaire de yoyo269 le 24/01/2007 20:21:20

Nouvelle mise à jour.
Principale avancée, la nouvelle option qui permet de générer des fichiers auto-recomposables. C'est-à-dire que YoyoCut n'est pas nécessaire pour recomposer le fichier d'origine.
Essayez et vous verrez ! ;)

YOYO, @+.

Commentaire de ndubien le 12/04/2007 20:36:44

Bonjour yoyo269,

je vous écrit pour vous signaler une défaillance dans votre programme (il s'agit d'une défaillance minime). J'ai un fichier nommé : "a.f i l e" de 132ko, le programme le coupe correctement mais lors de la recomposition de ce fichier, votre programme crée un fichier nommé différement : "a.f" de 0ko. Je pense que ce problème sera simple à résoudre.

En tout cas, il s'agit d'une très bonne source, car elle fonctionne bien, est accueillante...

A bientôt.
ndubien

Commentaire de yoyo269 le 12/08/2007 12:40:09

Nouvelle mise à jour !

YOYO, @+.