DECOUPAGE CHEMIN FICHIER (FORUM)

Commentaire de BruNews le 10/08/2005 12:57:59 administrateur CS

Salut,

eh bien tu n'es pas en forme ?

Dans ta GetFileType(), il y a de la mémoire allouée par ta strtoupper(), je ne vois pas le free() correspondant. Comme quoi il ne faut jamais retourner de chaine allouée dans une func c'est source d'oubli de libération de mémoire à tout coup, faut passer buffer que la func remplira. De plus ça oblige à bosser avec le mode d'alloc de la func, on peut ne pas vouloir de la CRT.

En C on connait les pointeurs, pas besoin d'itérateurs comme dans GetFileName() et les autres, ça fait parcourit les chaines plusieurs fois. 1 seul compteur suffit pour controler la longueur qund tu écris afin d'éviter un overflow.

A revoir, bonne continuation.

Commentaire de DeAtHCrAsH le 10/08/2005 13:58:53

BruNews> Faut dire que je dors pas beaucoup ces temps ci (debout 5h, coucher 23h30)
Sinon pour les itérateurs je sais que j'aurais pu passer par les pointeurs en décellant le "Null-terminated string" mais le gars est un débutant. C'est plus dans l'optique de travailler sur des tableaux que sur des pointeurs.

Steve_Clamage> Pour le cast je suis obligé car je passe par un compilateur C++. Il n'accèpete pas de transtypé un (void *) en (char *).
Sinon pour l'entete ctype.h, elle peut etre ommise car déjà présente dans stdlib.h.

Commentaire de DeAtHCrAsH le 10/08/2005 14:39:31

Steve_Clamage> Le gars bosse sur un projet C++. J'ai fait le code en C mais dans un .cpp car il ne va pas l'utiliser tel quel.

BruNews> Vala je viens de corriger le probleme de la fuite mémoire. J'ai monter le buffer dans les params de la fonctions.

Commentaire de DeAtHCrAsH le 10/08/2005 14:50:12

Steve_Clamage> lol t'es l'inspecteur gadget ? Tout ce qui "marche" en C "marche" en C++. Pas besoin de ces directives de compilations. Le gars va tout simplement récuperer le code et je ne suis meme pas sure qu'il va le copier tel quel.
Cette source est plus a titre d'apprentissage que de réutilisation.

Faut que j'aille au poste pour plus de question ? ^_^

Commentaire de steve_clamage le 10/08/2005 15:05:02

C'est pas grave je voulais dire, on peut se contenter de mettre le code en extern "C", le C s'interface tres bien avec le C++.

Commentaire de steve_clamage le 10/08/2005 15:26:57

Les guillemets ? non je penses pas, ca compile ca en C++ ?

struct
{
int a;
int b;
} s = { .b = 5 };

Mais tu as raison, c ansi est à peu de chose prés (cast du void* par exemple) compatible avec C++.

Commentaire de ledruid le 10/08/2005 20:41:16

Je confirme, c'est suite à ma demande sur le forum que il y à eu la gentillesse  de me donner un coup de main, et effectivement je suis sur un projet en c++ avec quelques contraintes ( mais je suis un vrai débutant .. lol).

Et effectivement le but n'est pas de le copier tel quel, mais plutot de m'en servir pour "apprendre".

Commentaire de vecchio56 le 10/08/2005 20:59:37 administrateur CS

Une déclaration de ce type pourrait aussi être utile si on a besoin de faire des concaténations par la suite:

char* GetFileDrive(const char* FilePath, char *tFileDrive)
{
  while(*FilePath != '\\') *tFileDrive++ = *FilePath++;
  *tFileDrive = 0;
  return tFileDrive;
}