Discussion :

[Goundy]

Bonjour les gars

Alors voilà j'ai un problème, je programme un bot irc en ce moment mais il y a un truc que je ne comprend pas:

Code :

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

1
2
3
4
/*recv  & parse(buffer);...*/
/*disant que la commande du user se trouve dans char *cmd*/
if (sscanf (cmd, "!qaddl %s", &texte)) printf ("je suis dans la condition");
else printf ("je ne suis pas dans la condition");

Ceci fonctionne correctement MAIS si j'envoi la commande: "!qadd" seulement ça entre quand même dans la condition et j'ai une erreur de segmentation ...

Pourquoi est-ce que la condition est satisfaite ?

Merci

[Médinoc]

1. [s]scanf("%s"), c'est le mal. Tu dois limiter la taille ([s]scanf("%10s") par exemple).
2. Quel est le type de texte ? Généralement, on ne met pas de & pour un %s. Si texte est un tableau de caractères, ça passe, mais dans le cas d'un pointeur, c'est le comportement indéfini (généralement un crash) assuré.

[Goundy]

Médinoc c'est bien un tableau de caractères le texte.

[Emmanuel Delahaye]

if (sscanf (cmd, "!qaddl %s", &texte)) printf ("je suis dans la condition");
else printf ("je ne suis pas dans la condition");[/code]
Ceci fonctionne correctement MAIS si j'envoi la commande: "!qadd" seulement ça entre quand même dans la condition et j'ai une erreur de segmentation ...

Pourquoi est-ce que la condition est satisfaite ?

Ton test n'est pas assez précis. sscanf() renvoi le nombre de conversions réussies. C'est donc '==1' qu'il faut tester et non '!= 0'...

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
19
20
#include <stdio.h>
 
int main (void)
{
#if 0
   char const *cmd = "!qaddl xxx";
#else
   char const *cmd = "!qaddl";
#endif
   char texte[32] = "";
   int n = sscanf (cmd, "!qaddl %s", texte);
 
   printf ("n = %d\n", n);
   if (n == 1)
      printf ("je suis dans la condition : '%31s'\n", texte);
   else
      printf ("je ne suis pas dans la condition\n");
 
   return 0;
}

Il faut apprendre à écrire du code de test... tu n'es plus un débutant depuis le temps...

[Goundy]

Merci beaucoup
Je crois que je dorénavant je vais mieux utiliser sscanf
Merci encore les gars

[Goundy]

Re !
Désolé pour le double post, j'ai une autre question :/

Code :

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

int x = sscanf (cmd, "!addl %s", texte);

Si cmd contient "!qaddl" seulement x est quand même = à 1
Comment ça se fait si aucune conversion ne peut avoir lieu? j'avoue que je suis paumé

voilà ma partie du code:

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
 
#define		_QADDL	"!qadd %[^\n]"
 
if (sscanf (ptr, _QADDL, quote)==1){
						printf ("strlen(%d)", strlen(quote));
						xx = strchr(ptr, ' ') + 1;
						settozero (quote);
						strcpy (quote, xx);
						/* add a single line */
						gard = add_quote (quote, nik);
							if (gard != 0){
								logit ("mysql error\n");
 
							}
						print ("added: %s\n", quote);
					}

Oubliez j'ai trouvé mon erreur :/
Merci