printf( "%s", const char*) -> segmentation fault

Bonjour,

J'ai une fonction dont le code est:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

void Force::create_pair(const char *style, const char *suffix) { delete [] pair_style; if (pair) delete pair;   int sflag; pair = new_pair(style,suffix,sflag);   if (sflag) { char estyle[256]; sprintf(estyle,"%s/%s",style,suffix); int n = strlen(estyle) + 1; pair_style = new char[n]; strcpy(pair_style,estyle); } else { int n = strlen(style) + 1; pair_style = new char[n]; strcpy(pair_style,style); } }

---

Elle fonctionne bien. Elle est tiree du code de "Lammps" (http://lammps.sandia.org). Je veux rajouter un printf pour inspecter la fonction:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

void Force::create_pair(const char *style, const char *suffix) { printf(" la valeur de style est %s \n",*style); delete [] pair_style; if (pair) delete pair; int sflag; pair = new_pair(style,suffix,sflag); if (sflag) { char estyle[256]; sprintf(estyle,"%s/%s",style,suffix); int n = strlen(estyle) + 1; pair_style = new char[n]; strcpy(pair_style,estyle); } else { int n = strlen(style) + 1; pair_style = new char[n]; strcpy(pair_style,style); } }

---

Mais j'obtiens lors de la compilation l'erreur:

Citation:

---

Segmentation fault

---

A quoi cela est il du? la variable est elle vide?
C'est possible car je veux justement verifier la valeur de cette variable, car on etudie le fichier d'entree du programme entier (lammps), mais j'aimerais savoir comment verifier que la variable est vide ou pas.

J'ai voulu introduire une boucle de verification:

Code:

---

1 2 3 4 5 6

const char* tmp01=*style; if (strlen(*tmp01)==0) {...} else {...}

---

Et j'obtiens l'erreur:

Citation:

---

error: invalid conversion from 'char' to 'const char *'

---

Quelqu'un peut il m'indiquer la syntaxe pour:
1) afficher un const char* avec printf
2) faire fonctionner la boucle conditionelle de verification de la taille
3) m'aider

Merci
Oodbae

Bonjour,

char* style est un char*
*style est donc un char*

Tu essayes de donner la valeur d'un char à un char*

strlen a pour signature strlen(const char*), tu l'utilises avec un char

%s du printf attend un char*, tu lui fournis un char : seg fault (à l'exécution et non à la compilation)

btw: si tu es en C++, pourquoi ne pas utiliser std::string qui t'empêcherait de manipuler ce genre de pointeurs ?

Salut bousk,

merci pour la reponse.
1) j'ai vu sur de nombreux forums que std::string serait plus simple, mais ce code n'est pas de moi (de l'universite de Sandia), donc je ne change pas la structure
2) j'Ai refait quelques test, notamment avec ta correction sur le char/char*, merci, et l'erreur "segmentation fault" reapparait. J'ai l'impression que la ligne de code suivante provoque l'erreur "segmentation fault". Est-ce que c'est possible, sachant que le texte "the function..." est bien affiche, mais le programme bloque apres et refuse d'afficher un simple " printf("ww \n"); "

Code:

---

printf(" the function Force::create_pair is being called with the arguments: \n");

---

oodbae

Salut ;)

A première vue je dirais tout d'abord ceci :

Code:

---

1 2	printf(" la valeur de style est %s \n",(rien)style);

---

De plus si tu codes en c++ utilise std::string ça simplifie énormément la tâche !

Enfin je crois que si tu testes strlen(qqch) avec qqch qui est un pointeur nul (ou sur aucune donnée), tu auras un segfault :mrgreen:

Ici de même qu'avec printf :

Code:

---

1 2 3 4 5 6

const char* tmp01=*style; if (strlen((rien ici)tmp01)==0) {...} else {...}

---

Avec évidemment tmp01 qui pointe sur quelque chose de correct ^^

Citation:

---

Envoyé par Lintel-oo

A première vue je dirais tout d'abord ceci :

Code:

---

1 2	printf(" la valeur de style est %s \n",(rien)style);

---

De plus si tu codes en c++ utilise std::string ça simplifie énormément la tâche !

---

Quand il faut faire du formatage de données numériques, c'est moins vrai.