probleme acces

Question

Bonjour,



Voila j'ai cr&#233;e la proc&#233;dure suivante :

----------------------------------------------------------------------------

void ReversD(char *D_ptTextNoReverse)

{

    char D_cTextTmp[500]="";

    char D_cCar;

    int D_iSize,D_ia,D_ib=0;

    

    D_iSize=strlen(D_ptTextNoReverse);

    D_ia=D_iSize-1;

    

    //Remise des lettre dans le bon sens.

    while(D_ib<=D_iSize)

    {

        D_cCar=D_ptTextNoReverse[D_ib];

        D_cTextTmp[D_ia]=D_cCar;

        D_ia--;

        D_ib++;

    }

    strcpy(D_ptTextNoReverse,D_cTextTmp);

}

----------------------------------------------------------------------------



Tous fonctionne, il affiche a l'&#233;crant la variable modifi&#233; avec le text
dans le sens inverse et puis pouf juste apr&#232;s mon programme plante
(surement un acces violation) mais je ne vois pas ou. Je l'ai d&#233;j&#224;
utiliser dans un de met programme mais la la variable D_ptTextNoReverse
etait d&#233;clar&#233;e en global (char D_cText[350]) et donc on ne passait pas
en paramettre un pointeur de type char mais bien la variable elle m&#234;me.
Je soup&#231;one que le probl&#232;me doit se trouver la mais je ne vois pas
comment. Je voulait juste g&#233;n&#233;raliser ma fonction pour la mettre dans
mon .h snif lol



Merci d'avance.

MuPuF · Answer

Par hasard, as-tu mis le char null a la fin de la chaine ?
Je crois pas moi en regarde ton prog, tiens nous informé

deck_bsd · Answer

non il n'y est pas, mais la n'est pas le probl&#232;me vu que cette
proc&#233;dure marche tr&#232;s bien quand je transmet direct une variable et sa
taille (ex :char variable[10]) mais pas avec un pointeur char. je pense
que le probl&#232;me vient de la. Surtout que j'ai fait ce que tu a dit mais
que le r&#233;sultat est le m&#234;me, plantage :'( snif

BruNews · Answer

Sur chaine vide '"" tu plantes &#224; tout coup:

char D_cCar;
int D_iSize,D_ia,D_ib=0;
    
D_iSize=strlen(D_ptTextNoReverse);
D_ia=D_iSize-1;


DONC:
D_iSize = 0
D_ia = -1
D_ib = 0


while(D_ib <= D_iSize) S'EXECUTERA !!!
D_cCar=D_ptTextNoReverse[D_ib]; // OK
D_cTextTmp[D_ia] = D_cCar; // BADABOUM, D_cTextTmp[-1] !!!

Je t'avais pourtant d&#233;j&#224; donn&#233; une fonction FIABLE et autrement mieux cod&#233;e:
void __stdcall ReverseStr(char *psz)
{
  char *c = psz, a;
  while(*c) c++;  while(psz < --c) {a *psz; *psz++ *c; *c = a;}
}

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

24Karas · Answer

bon j'te l'ai faite en un peu plus optimis&#233; je pense



FONCTION :



void Reverse(char *szInput)

{

    //vars

    char *sPtrStart;

    char *sPtrEnd;

    char cTmp;

    unsigned int uLen;



    // while there is letter to exchange

    uLen = strlen(szInput);            // length of the input string

    sPtrStart    =
szInput;            //
start of the string

    sPtrEnd        = szInput+uLen-1;    // end of the string

    while (sPtrStart < sPtrEnd)

    {

        // exchange letters

        cTmp        = *sPtrStart;

        *sPtrStart    = *sPtrEnd;

        *sPtrEnd    = cTmp;



        // move pointers

        sPtrStart++;

        sPtrEnd--;

    }

}



USAGE :

int main (void)

{

    //vars

    char *szBuff;



    szBuff = (char *)malloc(12);

    strcpy (szBuff,"1234567890A");

    fprintf (stderr,"%s
",szBuff);

    Reverse(szBuff);

    fprintf (stderr,"%s
",szBuff);

    free (szBuff);



    return (0);

}



marche tr&#232;s bien ici.

enfin test rapide pas que &#231;a &#224; foutre



++

24Karas · Answer

oupss trop rapide le brunews

deck_bsd · Answer

Oui Brunews, mais je la comprend pas ta solution c'est pour ca, d&#233;j&#224; le while(*c) c'est quoi la condition??? lol

deck_bsd · Answer

BrunNews , pk D_iSize = -1, il ne prend pas la taille de la chaine
vide, mais celle qui est pass&#233; en param&#232;tre donc D_iSize n'est pas
&#233;gale a -1.

BruNews · Answer

Si chaine vide
D_iSize vaut 0 donc D_ia=D_iSize-1; met bien D_ia &#224; -1, non ?

char *c = buf;
c pointe sur adresse de buf donc sur buf[0]
So buf est "ABC"
c pointe sur adresse du 'A'
donc *c vaut 'A'
while(*c) c++;
tant qu'il ne pointe pas sur 0 final on pousse c, ok ?

Faut absolument &#233;tudier les pointeurs si tu veux du code performant.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

deck_bsd · Answer

BuNews, dezo de te contredire mait j'utilise la chaine vide comme tu
dit est simplement un chaine de car ou je vais mettre le mot
reconstituer temporairement avant de l'affecter a la bonne variable, je
l'initialise a vide c'est tout. Le strlen est la taille de la chaine qu
l'on re&#231;oit a moin d'etre un fiaifait malin personne ne voudrai
inverser un text vide lol ;).



Pour ton exemple j'avai tout compris sauve le while (*c) je ne l'avait
pas vu comme &#231;a. Enfaite si je me trompe pas, la tu utilise
l'arith&#233;mitque des pointeurs c'est bien cela? en faisant pas exemple
c++ c'est &#233;gale a c+1 fois la taille de c(donc du type) ce qui fait
avancer et donc passe a la lettre suivante c'est bien cela?

BruNews · Answer

ben oui, c++ vaut c = c+ 1 donc pointe 1 octet plus loin en m&#233;moire.

Nenni pour ton explication, une fonction doit &#234;tre fiable dans tous les cas, apte &#224; recevoir une chaine vide aussi.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

deck_bsd · Answer

Croti, le probl&#232;me n'est mm pas la, ta proc&#233;dure et la mienne
fonctionne bien, le probl&#232;me est a l'affichage, je fait un
printf("%s",text); et boum ca plante, si je le supprime en l'aissant
l'apelle de la fonction cela marche, donc c'est ca qui ne va pas, et le
pis c'est que si je le met juste apr&#232;s l'apelle de la proc&#233;dure ca
m'affiche le texte a l'enver et puis ca plante, mais si je le change de
place, ca met des caract&#232;re al&#233;atoire et ca ne plante pas, je comprend
absolument pas le probl&#232;me.

BruNews · Answer

Tu &#233;crases une zone m&#233;moire avec ta fonction, tout d&#233;pend donc o&#249; se trouve l'adresse de d&#233;part du buffer &#224; retourner pou savoir ce qui sera &#233;cras&#233;.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

deck_bsd · Answer

Oki merci BruNews, en faite j'ai retir&#233; l'apelle d'une fonction
randomize que j'avai faite, ca marche impec, je l'ai remise apr&#232;s ca
marche encore impec :s comment pourrai je faire pour que cela ne se
reproduise plus? car  ca fait perdre du temps , on croi que c'est
dans le code mais pas du tout.

BruNews · Answer

Une erreur sur un prog se situe toujours dans son code, quand on ne trouve pas vite fait faut recommencer le tout et on y gagne en temps.
Soit tu utilises un debuggueur pour tracer l'ex&#233;cution de ton prog.
Soit, comme moi, je fais chaque fonction s&#233;par&#233;ment dans des mini progs de test en essayant de passer absolument nimporte quoi en param et &#231;a ne doit pas planter. Une fois au point je l'int&#232;gre au vrai prog. L'avantage est que tu te constitueras un jeu de fonctions solides et pr&#234;tes &#224; l'emploi.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

deck_bsd · Answer

Oui , mais je ne suis pas reponsable du placement des donn&#233;es en
m&#233;moires, donc c'est pas ma faute si ici j'&#233;crasai une zone de m&#233;moire.
h&#233;h&#233; BruNews on a du a un moment capter la mm id&#233;e, c'est ce que je
fait aussi surtout pour mes fonction et proc&#233;dure que je g&#233;n&#233;ralise et
que je met ds un header perso. Ici j'avai juste vite repris le
programme de test de ma fonction randomize et j'avait ajouter le test
de celle-ci. Ce que je ne comprend pas cette fonction et cette
proc&#233;dure est cr&#233;e ca, il n'y a rien pourtant ds ma fonction randomize
qui ferait ca, c'est une b&#234;te fonction lol il n'y a rien de sp&#233;ciale.

Probleme acces

15 réponses

Votre réponse

Discussions similaires