Discussion :

[darktimide]

Bonjour
Je souhaite mettre la taille de mon tableau à zero alors que je suis en pleine boucle:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
char mdc[100]="0";
char exit[4]="exit";
while(1)
{
        if(strcmp(mdc,exit)==0)
        {break;}
 
        read(fdaccept,mdc,sizeof(mdc));
        printf("Message recu du client:%s \n",mdc);
        memset(mdc,'\0',sizeof(mdc));
}

J'ai reussi a initialiser tout les caracteres du tableau lors de la declaration et avec le memset.
Mais je souhaite mettre la taille de mon tableau à zero.
Fonctionnement de mon prog:
- Il compare mdc(="0") avec exit(="exit")
- Si mdc == exit on sort de la boucle
- Sinon on lit les données recus, on les affiches et on remet à zero les champs du tableau

Le probleme:
- si mdc ="banane"
- si mdc ="exit"
je ne peux pas sortir de ma boucle
car mdc != exit
Je suppose que le prog l'interprete comme cela:
si mdc ="banane" -> mdc = "banane"
si mdc ="exit" -> mdc ="exitne" ou mdc="exit\0\0"
Voilà pourquoi je souhaite reduire la taille de mon tableau ou s'il ya une autre solution je suis ouvert.
Je vous remercie d'avoir lu entierement ce poste.

[diogene]

si mdc ="banane" -> mdc = "banane"
si mdc ="exit" -> mdc ="exitne" ou mdc="exit\0\0"

Ce sera "exit\0..." puisque le tampon a été préalablement rempli par des 0 par memset(). Or comme le strcmp() n'ira pas plus loin que le premier '\0' pour la comparaison, cela ne devrait pas poser de problèmes.

[darktimide]

Enfaite ça ne marche pas... justement :o
J'ai testé mon code avant d'avoir posté quand meme.
Ça ne fonctionne pas.
Merci quand même.

Edit: Je viens de tester en envoyant cette fois des le debut exit et rien ne se passe.
Une erreur au niveau du if ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
char mdc[100]="0";

if(strcmp(mdc,exit)==0)
        {break;}

        else{

        read(fdaccept,mdc,sizeof(mdc));                   //mdc="exit"
        printf("Message recu du client:%s \n",mdc);
        memset(mdc,'\0',sizeof(mdc));
        }

[droggo]

Hoa,

Déjà, quand je vois ceci :

Code :

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

char exit[4]="exit";

4 caractères réservés pour un mot qui en fait 5, sans oublier le zéro terminal nécessaire, ce qui nous amène à 6 caractères, je ne vais pas plus loin dans la lecture.

[darktimide]

J'ai modifié comme me la proposé droggo, rien a faire ça ne fonctionne pas.
Juste droggo tu me dis de mettre à 6, je ne vois pas pourquoi.
char exit[4]="exit"

exit[0]=e
exit[1]=x
exit[2]=i
exit[3]=t
exit[4]=\0

Peux tu m'indiquer mon erreur stp?

[droggo]

Joe,

Parce que tu débordes de la mémoire réservée, d'où comportement indéfini.

[darktimide]

Je ne comprends pas bien à quel moment je déborde de la mémoire pourrais tu m'expliquer plus en detail et même en mettant à 6 mon tableau, le probleme reste toujours le même.
Merci d'avoir répondu.

[Obsidian]

Bonjour,

Je ne comprends pas bien à quel moment je déborde de la mémoire pourrais tu m'expliquer plus en detail

Quand tu écris exit[4], tu réserves de la place pour quatre caractères uniquement, numérotés de 0 … à 3 ! Si tu inclus le zéro final au rang 4, cela fait bien cinq caractères en tout et non quatre :

Code :

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

1
2
3
4
5
6
 
exit[0]='e';   // Premier caractère 
exit[1]='x';   // Second caractère
exit[2]='i';   // Troisième caractère
exit[3]='t';   // Quatrième caractère
exit[4]='\0';  // Cinquième caractère

et même en mettant à 6 mon tableau le probleme reste toujours le même.
Merci d'avoir répondu.

En mettant ton tableau à 6, ça devrait fonctionner, sauf si l'émetteur n'envoie pas le zéro final… ce qui est probable si tu utilises « strlen() » pour mesurer la taille de la chaîne à envoyer.

Sinon, dans un cas comme dans l'autre, il te manque le terminateur : soit parce que tu as déclaré exit[4] et qu'il n'y plus de place pour lui, soit parce que l'émetteur ne l'a pas transmis avec la chaîne. Donc, strcmp va pour ainsi dire toujours échouer puisque la comparaison va impliquer ce qui ce trouve après, ce qui est indéfini par définition.

En particulier, dans le premier cas, même si l'émetteur envoie proprement « exit » suivi de son zéro, le test échouera quand même parce que cette chaîne sera inférieure à « exit » suivi de caractères non-nuls qui erreraient en mémoire.

[darktimide]

Merci beaucoup pour ta réponse,

Au final j'ai trouvé le probleme, c'était par rapport au memset qui était mal placé dans la boucle, j'ai aussi en meme temps enlevé le tableau exit et je compare directement mon tableau mdc à la chaine de caratere "exit":

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
while(1)
{
 
        if((strcmp(mdc,"exit"))==0)
        {printf("fin");break;}
 
        else
{
        memset(mdc,'\0',sizeof(mdc));
        read(fdaccept,mdc,sizeof(mdc));
        printf("Message recu du client:%s \n",mdc);
 
}
}

[Obsidian]

Merci beaucoup pour ta réponse,

Au final j'ai trouvé le probleme,

C'est gentil de penser à poster ta solution et à remercier les intervenants, mais il faut que tu lises ces réponses quand même parce que c'est la deuxième fois que tu passes à côté de la cause réelle de tes bugs (l'autre étant ici).

• Ton tableau « exit » est mal dimensionné. La chaîne écrite en dur fonctionnera toujours, certes. Mais ça ne fait que contourner le problème sans le résoudre ;
• Tu ne lis pas le code de retour de read() ;
• On ne sait pas comment est écrit le programme de l'émetteur.

A contrario, le memset() placé là où il l'est n'empêche pas du tout ton programme de fonctionner.

[darktimide]

Il est vrai que dès que j'ai trouvé la solution je n'ai pas lus les réponses avec attention,
Pour le read il est vrai que je dois vérifier le code retourné.
J'aurais sûrement dû donner des précisions sur les données reçues par le read et des indications sur le fonctionnement du programme émetteur ><'.
Merci beaucoup encore.