Discussion :

[oodbae_adriano]

Bonjour,

J'ai une fonction dont le code est:

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
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 :

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
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:

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 :

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

1
2
3
4
5
6
 
const char* tmp01=*style;
if (strlen(*tmp01)==0)
{...}
else
{...}

Et j'obtiens l'erreur:

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

[Bousk]

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 ?

[oodbae_adriano]

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 :

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

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

oodbae

[Lintel-oo]

Salut

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

Code :

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

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

Ici de même qu'avec printf :

Code :

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

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 ^^

[oodini]

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

Code :

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

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.