Discussion :

[beekeep]

Bonjour à tous,

j'essaye de récupérer la liste des sections d'un fichier à l'aide la la FAQ :
http://cpp.developpez.com/faq/vc/?pa...leSectionNames

contrairement à l'exemple je suis obligé de caster le premier paramètre en LPTSTR,
je sais pas si c'est ça qui pose problème mais en tout cas ça ne fonctionne pas .. ça plante avant la fin de la boucle.

voici mon code actuel:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
        char *szReturnedString= new char[32000];
        int nSizeRet= GetPrivateProfileSectionNames( LPTSTR( szReturnedString), 32000, p_sFilePath);
 
        int nSection=0,npos=0;
        while(npos<nSizeRet)
        {
            TRACE("%d-", strlen(szReturnedString+npos)+1);
            npos+=(strlen(szReturnedString+npos)+1);
            nSection++;
        }
        delete szReturnedString;

output:

2-2-2-2-2-1-1-2-2-Error: failed to load message box prompt string 0x0000.

Merci d'avance

[3DArchi]

Salut,
Remplace char * par TCHAR et enlève le cast.

[white_tentacle]

et

Code :

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

delete[] szReturnedString

[3DArchi]

Et wcslen ou mieux _tcslen dans TCHAR.h
+ verif echec allocation

[beekeep]

Merci pour vos réponses ça marche

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
        TCHAR szReturnedString[32000];
        int nSizeRet= GetPrivateProfileSectionNames( szReturnedString, 32000, p_sFilePath);
 
        int nSection=0,npos=0;
        while(npos<nSizeRet)
        {
            int iSectionLength = _tcslen(szReturnedString+npos);
 
            CString sSection = _T("");
            for(int i=npos; i<npos+iSectionLength; i++)
            {
                sSection.AppendChar(szReturnedString[i]);
            }
 
            this->m_vMaterial.push_back( new CMaterial( sSection ) );
 
            npos += iSectionLength+1;
        }

la partie récupération des noms de section n'est peut-être pas idéale mais ça fonctionne ..

[white_tentacle]

allouer 32000 TCHAR sur la pile, c'est beaucoup (avec sizeof(wchar_t) == 4, ça nous ferait 128ko sur la pile, gros risque de stack overflow) --> il faudrait mieux recourir à un tableau alloué sur le tas.

[beekeep]

oui c'est beaucoup je vais le réduire,
mais vu qu'on ne connais pas la taille avant de créer le buffer je vois pas comment faire ce que tu dis.

[white_tentacle]

Si tu le réduis, ça a des chances de ne plus marcher.

De toute façon, c'est une vieille API moisie (héritée de Win16) qu'il ne faut plus utiliser.

Bref, tu n'a pas d'autres chois que de passer un buffer très grand, en espérant qu'il soit assez grand (t'as pas vraiment de moyen de vérifier qu'il est assez grand).

Mon propos, c'était de dire qu'allouer ce tableau sur la pile est une mauvaise idée. Faire plutôt

Code :

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

1
2
3
TCHAR* szReturnedString = new TCHAR[32000];
...
delete[] szReturnedString;

Ce qui l'alloue sur le tas, et le libère.

Idéalement, il faudrait mieux un auto_tab_ptr pour s'assurer de libérer la mémoire, mais ce n'est pas dans std, donc on va dire que ça suffit pour l'instant .

[3DArchi]

t'as pas vraiment de moyen de vérifier qu'il est assez grand

Ce sera borné par la taille du fichier .ini. Mais ça peut encore faire très grand... Après, si tu sais à peu près comment est constitué ton fichier tu peux t'en servir... mais ce sera toujours dangereux

[beekeep]

Mon propos, c'était de dire qu'allouer ce tableau sur la pile est une mauvaise idée. Faire plutôt

Code :

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

1
2
3
TCHAR* szReturnedString = new TCHAR[32000];
...
delete[] szReturnedString;

ok je vais faire ça.

De toute façon, c'est une vieille API moisie (héritée de Win16) qu'il ne faut plus utiliser.

non mais on ne m'a pas laissé le choix, j'aurai jamais choisi ce format de fichier à la noix sinon.
Puis elle est utilisée dans le reste du projet que je reprend ..

Merci bien

[Médinoc]

Pour le auto_ptr, je me demande si on ne pourrait pas faire ceci:

Code C :

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

1
2
3
4
5
6
7
typedef TCHAR TrenteDeuxMilleTChars[32000];
 
...
 
TrenteDeuxMilleTChars *pTab = new TrenteDeuxMilleTChars;
TCHAR * szReturnedString = *(*pTab)[0];
std::auto_ptr<TrenteDeuxMilleTChars> deleteTchars(pTab);

Ou, beaucoup plus simple, utiliser un std::vector<TCHAR>.

PS: Sous Windows (comme sous Java, d'ailleurs), les wchar_t font seulement deux octets. La première version Unicode de Windows supportait seulement UCS-2, alors ensuite, compatibilité oblige, ils ont été obligés de passer en UTF-16 au lieu de UCS-4...