Discussion :

[Verbalinsurection]

ou mieux encore :

Code :

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

1
2
3
4
5
6
7
8
if (fileName != "NOP")
        {
            cout << "File received, current processing file !" << endl;
            pData = (PMYDATA) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(MYDATA));
            pData->fileNameLink = strdup(fileName.c_str());
            hThread = CreateThread(NULL, 0, lectureFichierGPS, pData, 0, NULL);
            CloseHandle(hThread);
        }

Ca evite d'avoir le fileNameChar

[Spout]

Effectivement, c'est mieux !
Ca donne quoi alors maintenant?

[Verbalinsurection]

Toujours des fuites de mémoires mais le code me plait dejà beaucoup plus...
Je pense que j'ai des fonctions dans mon code qui merdouille....

[Spout]

Il faut que tu regardes à chaque fois que tu utilises à pointeur.
A part ça, je sais pas trop quoi te dire

[Verbalinsurection]

Pfiou...faut que je reprenne tout mon code moi... aller je m'y met, je te tiens au courant et merci encore pour ton aide!

[Verbalinsurection]

ha si une question (ou deux) :
Quand on fait :

Code :

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

1
2
3
FILE *file;
file = fopen(fileName.c_str(), "rb");
fclose(file);

faut il faire appres :

Code :

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

delete(file);

?

[Spout]

Là je n'ai pas la réponse avec certitude, mais à priori je dirais que non, tu n'as pas besoin de faire le delete et que le fclose fait tout ce qu'il faut.
A vérifier

[Médinoc]

Non.

De plus, il ne faut pas faire le fclose() si fopen() a renvoyé NULL:
free(NULL) est garanti ne rien faire, mais fclose(NULL) est un comportement indéfini (et pour cette raison, plante exprès sous Visual 2005).

[Verbalinsurection]

et dans un cas comme celui-ci :

Code :

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

1
2
3
4
{
    char *maxSoftsTmp = c_inifile_get_string("SOFTS", "TOTAL", NULL);
    maxSofts = strtol(maxSoftsTmp, NULL, 0);
}

[Médinoc]

C'est marqué dans la doc

[Verbalinsurection]

OUPS!
ça t'apprendra verbal à pas lire les docs!

[Verbalinsurection]

J'ai ça :

Code :

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

1
2
3
4
5
6
7
while (x <= maxSofts)
        {
            char ch[3];
            sprintf(ch, "%d", x);
            listSofts[x-1] = c_inifile_get_string("SOFTS", ch, NULL);
            x++;
        }

J'ai voulu faire un

Code :

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

free(listSofts);

mais ça plante
Je pense devoir utiliser :

Code :

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

delete(listSofts[]);

Mais la difference avec free() ?


edito :
pour eviter le char ch[3] si je mets ça ça va? :

Code :

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

1
2
3
4
5
while (x <= maxSofts)
        {
            listSofts[x-1] = c_inifile_get_string("SOFTS", (const char*) x, NULL);
            x++;
        }

mais ça plante avec ça...

[Médinoc]

le char ch[3], aucun problème, il est sur la pile et n'est pas utilisé après le while.

Par contre:

• On ne sait pas comment est alloué listSofts
• Il faut supprimer les éléments de listSofts comme suit:
  

  Code C++ :

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

  1
  2
  3
  4
  for(int iSoft=0 ; iSoft<maxSofts ; iSoft++)
  {
  	free(listSofts[iSoft]), listSofts[iSoft]=NULL;
  }

[Verbalinsurection]

Allocation :

Code :

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

1
2
3
4
5
6
7
8
string listSofts[maxSofts];
        while (x <= maxSofts)
        {
            char ch[3];
            sprintf(ch, "%d", x);
            listSofts[x-1] = c_inifile_get_string("SOFTS", ch, NULL);
            x++;
        }

[Médinoc]

Alors là, problème: Le char* retourné par c_inifile_get_string est perdu dans la boucle même.
Change-là ainsi:

Code :

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

1
2
3
4
5
6
7
8
9
string listSofts[maxSofts];
for(x=1 ; x<=maxSofts ; x++)
{
	char ch[3];
	sprintf(ch, "%d", x);
	char * iniStr = c_inifile_get_string("SOFTS", ch, NULL);
	listSofts[x-1] = iniStr;
	free(iniStr);
}

Et tu n'auras rien à faire pour supprimer, puisque le tableau est sur la pile.

[Verbalinsurection]

Merci encore médinoc, peux tu m'expliquer un tout petit plus (ouai j'aime bien quand ça marche mais je prefere encore mieux comprendre).
Tu me dis que le tableau est dans la "pile"... mais le tableau est initialisé avant le { par string ...

[Médinoc]

Ben c'est un tableau de maxSoft objets std::string, tous dans la pile.
Typiquement, chaque objet string doit contenir un pointeur vers ses données, elles allouées sur le tas...

[Verbalinsurection]

Ok merci pour vos explications ça m'aide vraiment.
Par contre j'ai toujours des fuites moi (je parle de mémoire là hein...)

[Spout]

Jette un oeil du côté de DEBUG_NEW pour t'aider à savoir d'où tes fuites () viennent.

[Verbalinsurection]

merci pour l'info mais je n'arrive pas à l'utiliser, je travail sous code::blocks, ming pour la compilation et gdb comme debugger (que je ne sais vraiment pas utiliser...)