probleme avec strncpy

bonjour

quelqu'un pourrais me dire pourquoi mon appli plante quand j'utilise strncpy
avec un exemple de mon erreur pour une meilleur comprehension

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28

FILE *fptr; fptr = fopen(Chdata,"r"); // ouverture du fichier if (fptr != NULL) { char ligne[100]; while (fgets (ligne, sizeof ligne, fptr) != NULL) { char *controlstr; char *valeurstr; size_t countstr = strlen(ligne); // strncpy(controlstr,ligne,4); char *refbox1 = "A001"; char *refbox2 = "B001"; if (controlstr != NULL) { if (strcmp(controlstr, refbox1) == 0) { gtk_combo_box_append_text(Box1data, ligne); } if (strcmp(controlstr, refbox2) == 0) { gtk_combo_box_append_text(Box2data, ligne); } } } fclose(fptr); // fermeture du fichier }

---

merci

tout simplement parce que "controlstr" est un pointeur non initialisé qui pointe sur rien....

Citation:

---

Envoyé par Vincent Rogier

tout simplement parce que "controlstr" est un pointeur non initialisé qui pointe sur rien....

---

la j'ai besoin d'explication ?:calim2:

si je comprend bien, tu utilises des fonctions dont tu ne comprends pas les types de données mis en jeu...

Je te conseille de relire des cours, tuto, bouquin sur le C et particulièrement le concept de chaines de caractères et de pointeur en C.

au lieu de t'expliquer ici ces concepts la, regarde la faq C, les cours et tutos de la rubrique C, ca y seront mieux expliqué que ce que je peux faire en 3 lignes...

j'ai deja lu la faq et les tutos, mais il ya quelque qui m'echappe:calim2:

Bon en 3 lignes on peut tout de meme dire :
Pour pouvoir copier les 4 premiers caracteres de "ligne" a l'adresse pointée par "controlstr" il est necessaire de reserver un espace mémoire :
au lieu de
char *controlstr;

reservez un espace mémoire
char controlstr[5] ;
ou
char *controlstr;
controlstr=malloc(5);

puis initialiser cet espace mémoire
memset(controlstr,NULL,5);
8O

Le memset n'est plus utile dès l'instant où tu as initialisé soit un tableau, soit ton pointeur à l'aide de malloc.

Il faudrait également expliquer pourquoi si on veut copier les 4 premiers caractères de "ligne" il faut réserver un tableau de 5 char.

Cette réponse en plusieurs parties n'aidera pas forcément le PI, et je lui conseille également de relire la FAQ ou des tutoriels sur les chaînes de caractères et/ou les pointeurs.

Mais pour faire court :
Une chaîne de caractère doit se terminer par le caractère nul '\0'.

Le man de strncpy nous dit :

Citation:

---

The strncpy() function is similar (to strcpy), except that not more than n bytes of src are copied. Thus, if there is no null byte among the first n bytes of src, the result will not be null-terminated.

---

Le code à utiliser sera donc du style :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

char *controlstr = malloc (5 * sizeof(char)); //sizeof(char) toujours égal à 1 // ou bien : // char controlstr[5];   strncpy (controlstr, ligne, 4); controlstr[4] = '\0';   /* traitement... */   // si on a utilisé malloc : free (controlstr);

---

merci benoitra

mon appli ne plante plus, mais maintenant je me retrouve avec 8 ou 9 caracteres dans "controlstr" ce qui rend la comparaison impossoble:calim2:

Citation:

---

Envoyé par BSans

strncpy (controlstr, ligne, 4);
controlstr[4] = '\0';

[/code]

---

le rajout de cette ligne fais plante mon appli ou n'affiche qu'une partie du contenu du fichier

Il faudrait que tu repostes le code modifié.

À quel moment survient ce plantage ?

Citation:

---

Envoyé par BSans

Il faudrait que tu repostes le code modifié.

À quel moment survient ce plantage ?

---

Code:

---

1 2	controlstr[4] = '\0';

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

FILE *fptr; fptr = fopen(Chdata,"r"); // ouverture du fichier if (fptr != NULL) { char ligne[100]; while (fgets (ligne, sizeof ligne, fptr) != NULL) { char controlstr[5]; char *valeurstr; size_t countstr = strlen(ligne); strncpy(controlstr,ligne,4); controlstr[4] = '\0'; char refbox1[5] = "A001"; // ici :aie: char refbox2[5] = "B001"; // et ici :aie: if (controlstr != NULL) { if (strcmp(controlstr, refbox1) == 0) { gtk_combo_box_append_text(Box1data, ligne); } if (strcmp(controlstr, refbox2) == 0) { gtk_combo_box_append_text(Box2data, ligne); } } } fclose(fptr); // fermeture du fichier }

---

probleme resolu
merci a vous

Ok, donc le problème ne vient à priori pas de "controlstr[4] = '\0';", mais de

Code:

---

1 2 3

char refbox1[5] = "A001"; char refbox2[5] = "B001";

---

Ta précédente déclaration était bonne :

Code:

---

1 2 3

char *refbox1 = "A001"; char *refbox2 = "B001";

---

Si tu veux passer par des tableaux, alors il faut utiliser cette déclaration :

Code:

---

1 2 3

char refbox1[5] = {'A','0','0','1','\0'}; char refbox2[5] = {'B','0','0','1','\0'};

---

Citation:

---

Envoyé par benoitra

puis initialiser cet espace mémoire
memset(controlstr,NULL,5);

---

0 mais certainement pas NULL. NULL n'a de sens que pour un pointeur.

Et au passage

Code:

---

1 2	char controlstr[5] ; memset(controlstr,0,5);

---

s'écrit plus simplement :

Code:

---

char controlstr[5] = "" ;

---

@ BSans: Les 3 déclarations que tu as citées sont correctes. La première est équivalente à la dernière.

La seconde est à éviter car tu fais pointer un pointeur sur une zone mémoire non modifiable. Si jamais tu essayes de modifier cette zone ton compilateur ne te dira rien mais tu auras un segfault à l'exécution.
Si tu veux utiliser la deuxième déclaration il est mieux de déclarer tout ça en 'const'.


@ lrgtk : Tu pourrais éviter l'appel à "controlstr[4] = '\0';" si tu avais initialisé ta variable au moment de la déclaration.

Code:

---

char controlstr[5] = "";

---

Cette ligne est inutile car ton tableau est 'statique'. Ta condition sera toujours vraie.

Code:

---

if (controlstr != NULL)

---

A quoi te sert la variable valeurstr ? ;)