Etrange phénomème paradoxale :)

Question

Pourquoi quand j'execute mon prog depuis le dossier Release il plante et quand je le lance depuis VS il ne plante pas..

J'ajoute qu'il plante lors d'un appel &#224; une fonction qui charge un
fichier, mais j'ai pris soin de mettre le chemin complet du fichier
avec GetModuleFileName

Donc je ne comprend pas tr&#232;s bien ^o)





//Signature en construction

BruNews · Answer

Quasi minuit c'est bient&#244;t l'heure du crime pas des extralucides.

ciao...
BruNews, MVP VC++

cs_Joky · Answer

J'ai trouver o&#249; &#231;a merder mais j'comprend pas pourquoi....

MessageBox(NULL, "Ligne 114", 0, 0);

    Buff = new char[dwByteToRead+1];

    MessageBox(NULL, "Ligne 116", 0, 0);



il m'affiche la boite Ligne 114 mais pas la Ligne 116

Pourtant quand je le lance depuis VS, il continue sa route...



//Signature en construction

steve_clamage · Answer

Trace dwByteToRead, verifie si le new &#224; echou&#233; (catch std::bad_alloc) ...

cs_Joky · Answer

B&#233; moi j'veux bien lol , mais le probl&#232;me c'est que j'peux pas trop
jouer avec le debugger vu qu'il plante uniquement quand je le lance
depuis l'explorateur

Avec VS, il marche impec....

//Info
MessageBox(hWnd,"

BruNews · Answer

et vaut combien ce brave 'dwByteToRead' ???

ciao...
BruNews, MVP VC++

steve_clamage · Answer

Ouai, affiche le dans un messagebox avant le new au moins.

cs_Joky · Answer

dwByteToRead = GetFileSize(hFile, NULL);

    char Buffer[512];

    wsprintf(Buffer, "ByteToRead : %d", dwByteToRead);

    MessageBox(0,Buffer,0,0);



Il vaut 152... enfin pour le fichier hein :) il varie...

cs_Joky · Answer

Et il vaut exactement la m&#234;me chose que lorsque que j'execute depuis VS...

BruNews · Answer

envoie toute la fonction.
Tu es sur de la validit&#233; de hFile ?

ciao...
BruNews, MVP VC++

cs_Joky · Answer

Bon je garantie rien pour la propret&#233; lol :)

Et oui je suis du hFile puisque je le test :p



BOOL LoadUser(char *szName, LPUSER hUser)

{

    HANDLE hFile;

    DWORD dwByteToRead;

    DWORD dwByteRead;    

    char *Buff;

    

    char Path[1024];



    char *d = Path + GetModuleFileName(NULL, Path, 1024);

    while(*d != '\')    *d--;

    *++d=0;



    strcat(Path, szName);

    hFile = CreateFile( Path,

                        GENERIC_READ,

           
           
FILE_SHARE_READ,

                        NULL,

                        OPEN_EXISTING,

                        0,

                        0);

    if(hFile == INVALID_HANDLE_VALUE)

    {    

        CloseHandle(hFile);

        return FALSE;

    }

    

    dwByteToRead = GetFileSize(hFile, NULL);



    Buff = new char[dwByteToRead+1];



    if(!ReadFile(hFile, Buff, dwByteToRead, &dwByteRead, NULL))

    {

        MessageBox(NULL, "Erreur dans la lecture du fichier.", "Erreur", 0x10);

        CloseHandle(hFile);

        return FALSE;

    }

    CloseHandle(hFile);

    CreateUser(hUser, szName);

    

    LPSHORTCUT lpRaccourci = InitialiserShortcut();



    int iSeparateur=0;

    char *c[2];



    c[iSeparateur] = strtok(Buff, ";");

    while(c != NULL)

    {

        iSeparateur++;

        if(iSeparateur>1)

        {

            Remplir(lpRaccourci, c[0], c[1]);

            AjouterRaccourci(hUser, lpRaccourci);

            lpRaccourci = InitialiserShortcut();

            iSeparateur = 0;

        }

        c[iSeparateur] = strtok(NULL, ";");

        if(!c[iSeparateur])

            break;

    }

    

    delete[] Buff;

    if(!hUser)

        return FALSE;



    return TRUE;

}

BruNews · Answer

if(hFile == INVALID_HANDLE_VALUE)
    {    
        CloseHandle(hFile);
        return FALSE;
    }

1ere erreur, on ne close pas un handle invalide puisqu'il ne pointe sur rien.

ciao...
BruNews, MVP VC++

cs_Joky · Answer

Ah okay, j'pensais qu'il aurait &#233;t&#233; mal initialis&#233; donc le Close quoi 

C'est bon &#224; savoir &#231;a d&#233;j&#224; :p

J'fais un test

BruNews · Answer

Buff = new char[dwByteToRead+1];
Faut tester validit&#233; de Buff toujours.


Buff[dwByteToRead] = 0; A AJOUTER

on verra ensuite.

ciao...
BruNews, MVP VC++

steve_clamage · Answer

A mettre dans un bloc try.

BruNews · Answer

Teste toujours si lecture sans erreur de cette mani&#233;re, c'est la plus sure:

dwByteRead = 0;
ReadFile(hFile, Buff, dwByteToRead, &dwByteRead, NULL);
CloseHandle(hFile);
if(dwByteToRead != dwByteRead) ERREUR DE LECTURE

ciao...
BruNews, MVP VC++

BruNews · Answer

steve > new ne retourne pas NULL si on ne modifie rien ???

Si envoie une erreur alors lui pr&#233;f&#233;rer malloc sans aucun doute, surtout qu'il n'y a pas de constructeur.

ciao...
BruNews, MVP VC++

cs_Joky · Answer

Buff = new char[dwByteToRead+1];

    if(!Buff)

    {

        MessageBox(NULL, "Invalidit&#233; Buff", NULL, NULL);

        return FALSE;

    }

    Buff[dwByteToRead] = 0;


Il passe m&#234;me pas par la condition...

Il plante avant...

Sauf avec VS

J'upload tout sur mon site attend lol

Mais tu critiques pas le fouilli hein :)

BruNews · Answer

ok mais je look demain, now je vais dodo.
Mets le lien.

ciao...
BruNews, MVP VC++

steve_clamage · Answer

BruNews > avant (pre-standard), new retournait 0, maintenant il
jette une exception de type std::bad_alloc. C'est sur que pour 
l'utilisation qui en est faite (c'est du C) autant utiliser malloc et
verifier la valeur retour.

cs_Joky · Answer

http://membres.lycos.fr/Joky/Desktop%20Multi%20User.rar

Bonne nuit l'ami :p

Reve pas trop de moi :)

BruNews · Answer

et encore, en prog API &#231;a va se transformer en HeapAlloc(), il gagnera encore qlqs cycles, hi hi.

ciao...
BruNews, MVP VC++

BruNews · Answer

Pour sur, t'es pas assez belle.
A demain.

ciao...
BruNews, MVP VC++

cs_Joky · Answer

Ouaip j'vais essayer malloc donc

mais l&#224; j'ai un soucis :)

c'est comment qu'on fait ???



Buff = (char*)malloc(sizeof(dwByteToRead)+1);

ou Buff = (char*)malloc(dwByteToRead+1);

J'pense que c'est &#233;quivalent.... enfin mes pens&#233;es on va les oublier un peu

cs_Joky · Answer

Je suis une Blonde Platine moi :o

BruNews · Answer

sizeof(dwByteToRead)+1 DONNERA 5 !!!
va pas me laisser dormir la vedette.

Buff = (char*) malloc(dwByteToRead+1);

marche pas ton lien !!!

ciao...
BruNews, MVP VC++

cs_Joky · Answer

http://membres.lycos.fr/Joky/

Prend Desktop Multi User.rar :)

Y'a l'executable tu peux d&#233;j&#224; t'amuser &#224; voir ce que &#231;a donne lol

Mais pas trop hein

Il va couter cher lui :p

BruNews · Answer

ok c'est pris.

ciao...
BruNews, MVP VC++

cs_Joky · Answer

N'emp&#234;che que m&#234;me avec un malloc lol

Il plante...

C'est trop bizarre la vie xD

vecchio56 · Answer

Le problème ne vient pas du tout du new, ca se situe plutot dans la boucle à la fin de la fonction. Sans doute un pb dans une de tes fonctions de raccourcis

cs_Joky · Answer

B&#233; comment ca se fait qu'il s'arr&#234;te au new aussi...J'vais remettre un peu d'ordre la dedans :p

if(!Meilleur("Joky")) return ERREUR;

magic_Nono · Answer

en mode d&#233;bug, il y a protection de la m&#233;moire

chose n'&#233;tant pas en mode release



(raison du plantage &#224; l'exec)



tu vois que &#231;a a du bon, BN,

des classes prot&#233;geant les utilisation directes....

___________________________________________________________
Magicalement
Nono

BruNews · Answer

Absolument et d&#233;finitevement NON, si on ne devait faire que des classes par crainte de mal coder, on ne serait pas en train d'en discuter devant un PC mais au phone. Il n'y aurait pas de kernel ni aucun driver ni m&#234;me de GDI dont une tr&#232;s grande partie est carr&#233;ment en ASM.
Il faut &#233;crire son cahier des charges et plan de dev avant de commencer &#224; coder, voila ce qu'il faut d&#233;duire et pas autre chose sinon goto framework .net de suite.

ciao...
BruNews, MVP VC++

magic_Nono · Answer

je suis pr les 2, BN tu le sais

C++ orient&#233; objet incluant assmb qd n&#233;cessaire



mais qd je vois ce genre de pb, 

ou mm ceux qu'on me pose directement,

je dis qu'il faut prot&#233;ger et v&#233;rifier les alloc sans cesses



d'ou les macros de gestion mem

qui &#233;taient une de mes premieres publi sur ce site



et l&#224; pas de classe, juste une v&#233;rif SYSTEMATIQUE de bonne ex&#233;cution


___________________________________________________________
Magicalement
Nono

Etrange phénomème paradoxale :)

33 réponses

Votre réponse

Discussions similaires