Le C++ nous mène-t-il au suicide ???

Question

Lol question tr&#232;s explicite :)
Donc je comprend plus rien du tout :)
Desfois &#231;a marche, desfois non, desfois il plante, desfois non, desfois il compile, desfois il tape des crises... Help Me !!! Lol

Donc en faite, j'explique le principe, il charge un fichier que vous choisissez, il le coupe en ligne, il &#233;tudie chaque type de ligne et affiche le type que vous souhaitez, bref, il marche pour certain fichier, et pour d'autre, il oublie une ligne, enfin bref regarder par vous m&#234;me...
J'ai tester en mettant des pauses partout, afficher les diff&#233;rentes valeurs de la ligne au cours du traitement, rien &#224; faire, ma ligne se perd dans la m&#233;moire :)

Le code est fait sous Dev C++, d&#233;sol&#233;, c'est que la structure...
Le fichier qui merde, c'est Ligne.cpp.
Donc ouvrer l'exe, ou compiler le 'like you want' et tester avec diff&#233;rents fichiers et explikez to me svp :p

ZIP : http://membres.lycos.fr/Joky/Scanner%20C++%20File.zip


void Aurevoir( void ); //Bonne journ&#233;e

cs_Joky · Answer

Les fautes j'men fou un peu, mais y'en a qui sont vraiment honteuses...
Ouvrer  = Ouvrez...


void Aurevoir( void ); //Bonne journ&#233;e

cs_Joky · Answer

J'ai remarquer que le fichier Ligne.cpp merde &#224; partir de la ligne 119...
Et je pense que c'est &#233;troitement lier &#224; la fonction TrouverProchainMot...


void Aurevoir( void ); //Bonne journ&#233;e

Zootella · Answer

'l'archive est d'un format corrompue...", windows nous tue bien avant ton code !

cs_Joky · Answer

Lol en effet &#231;a craint, j'pense pas qu'elle soit corrompue, c'est juste ton log qui t&#233;l&#233;charge... j'avais les m&#234;mes probl&#232;mes avec DAP sur themexp.org fin bref, jv&#233; tout mettre sans zip :)

http://membres.lycos.fr/Joky

Voir C++ ;)


void Aurevoir( void ); //Bonne journ&#233;e

cs_Joky · Answer

Voila donc faut aller voir dans Scanner C++ ( Ebauche )
Et y'a les fichier n&#233;cessaire.


void Aurevoir( void ); //Bonne journ&#233;e

cosmobob · Answer

salut,

oublie jamais un truc 'desfois il plante, desfois non' : ton code est
bugg&#233;, et comme ds 90% des cas c'est du a une mauvaise utilisation des
pointeurs.

primo : je sais pas si t'as fait du java avant, mais il semblerait que
t'aies pas compris certaints trucs : un char *Line = new char[512]; est
inutile !! car tu connais a la compilation la taille de ton tableau.
Ecris plutot : char Line [512]; ca te permettra de pas faire de
delete[] par la suite.

deuxio, ben des le main ca commence mal : 

  CLigne **cLigne; 

  *cLigne = new CLigne[NbrLigne-1];  



exemple typique d'&#233;criture dans une zone de m&#233;moire non allou&#233;e ...

Fais plutot CLigne * cLigne = new CLigne[NbrLigne-1]; et ensuite
utilise &cLigne si tu veux un CLigne** ... La dans ce que tu fais
tu d&#233;clares un pointeur sur un pointeur de CLigne, et en faisant
*cLigne = ... tu &#233;cris dans ce qui est point&#233; par ton pointeur de
pointeur, mais pb, celui ci pointe vers qqc d'al&#233;atoire a la base, et
en tout cas certainement pas vers une zone m&#233;moire disponible. Qd ca
plante pas a cet endroit, c'est de la chance. 

Tu peux egalement faire (mais la 1ere solution est meilleure ...)

  CLigne **cLigne; 

  cLigne = new CLigne*;


  *cLigne = new CLigne[NbrLigne-1];  

mais la ca te permet peut etre de mieux comprendre pourquoi ce que t'as fait ne va pas.



ensuite un pb etant li&#233; a la partie C de ton code, si tu fais

     szLigne = new char[512];

    strcpy(szLigne, szValue);

exemple typique de buffer overflow car tu ne controles pas la taille de
szValue ... N'utilises jamais strcpy, prefere la version s&#233;curis&#233;e
strncpy ...

tu devrais avoir:

     szLigne = new char[512];


    strncpy(szLigne, szValue,512);

    szLigne[512-1] = 0; 

cela dis tu devrais avoir un char szLigne[512]; comme champs de ta classe, ca t'eviterait un new avec une taille fixe !!!



bon d&#233;ja une correction de ces quelques pb vont peut etre t'&#233;viter certains plantages ..

a+

cs_Joky · Answer

H&#233;h&#233;h&#233; merci :p pour toutes ses remarques, j'v&#233; vite corrig&#233; le bordel et j'dis quoi...
C'est vrai que les pointeurs lol c'est quand m&#234;me ce qu'il y a de plus merdique :p

@tte de suite 


void Aurevoir( void ); //Bonne journ&#233;e

cosmobob · Answer

et controle la taille des tableau que tu manipules !!!!

ta fonction void CLigne::TrouverMotSuivant(char *c, char *Mot) est un exemple typique de non controle.

si tu veux pas t'emmerder ou que tu r&#233;ussis pas a t'en sortir avec les
tableaux de char, fais du C++ justement et utilise les std::string ....

cs_Joky · Answer

J'voudrais savoir si c'est correct &#231;a lol xD

  int NbrLigne = cFile.CompterLigne();
  CLigne *cLigne[NbrLigne-1];
  cLigne[0] = new CLigne(2, "David");
  cLigne[0]->Afficher();
  system("PAUSE");
  delete cLigne;

donc on commence par compter les lignes..., on cr&#233;e un tableau de pointeur qui ne point sur rien pour l'instant, et ensuite le premier &#233;l&#233;ment on l'initialise... Normalement c'est bon la ??? :)


void Aurevoir( void ); //Bonne journ&#233;e

cs_Joky · Answer

delete[] cLigne plutot...


void Aurevoir( void ); //Bonne journ&#233;e

cosmobob · Answer

ca, ca doit pas compiler : CLigne *cLigne[NbrLigne-1]; la taille du tableau doit etre connu a la compilation si tu fais qqc comme ca (ca doit etre une constante quoi, la NbrLigne-1, au moment de compiler, le compilo sait pas ce que ca vaut !) du coup le reste est pas consistant, puisque tu écris cLigne[0] = new CLigne(2, "David");, donc en fait cLigne est supposé ici etre un CLigne** ... utilise le type vector de la STL, t'auras plus de pb avec les tableaux .... ce que tu fais est plus du C que du C++...... ex: #include vector cLigneArray; cLigneArray.push_back( CLigne(2, "David") ); // ajout d'une CLigne au tableau cLigneArray[0].Afficher(); etc...

cosmobob · Answer

enfin si tu utilises la STL, il faut que tes classes d&#233;finissent proprement le constructeur par copie, et l'operateur de copie:

le CLigne& operator=(const CLigne& b); // operateur d'affectation

et CLigne::CLigne(const CLigne& b); // constructeur par copie



quand tu ne les d&#233;finis pas, et que tu fais une allocation dynamique
dans le constructeur, tu peux facilement obtenir un plantage de ta
classe; essaie ca:



{

    CLigne a;

    CLigne b(a);

}

tu verras ca va planter :)

(lors de la destruction de b, il va essayer de faire un delete[]
szLigne; alors que la zone m&#233;moire d&#233;sign&#233;e par szLigne a d&#233;ja &#233;t&#233;
d&#233;sallou&#233; lors de la destruction de a)

a+

cs_Joky · Answer

Mon dieu lol c'est affreuxxxxxxxxxxxx
J'suis assez mal barr&#233; donc :p

Il est donc temps que j'm'int&#233;resse &#224; ces vecteur la j'sais pas trop quoi :p
En tout cas merci beaucoup de t'investir lol moi ca fait longtemp j'me serais casser :p


void Aurevoir( void ); //Bonne journ&#233;e

Le C++ nous mène-t-il au suicide ???

13 réponses

Votre réponse