Discussion :

[z980x]

Bonjour à tous,

voici mon pb, j'obtiens une erreur de segmentation lors de la compilation de mon projet.

Je vous mets les parties qui posent problème:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 pt_ll mes_liaisons=NULL, aux1=new liste_liaisons;
 
  for(int t=0; t<case_tab;t++) {
    cout << tab_villes[t] << endl;
 
    strcpy(aux1->ville_source,"Philippe");
    cout << aux1->ville_source;
 
    aux1->lien = mes_liaisons;
    mes_liaisons=aux1;
 
  }

Dans le .h, définissant une liaison:

Code :

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

1
2
3
4
5
6
7
8
9
10
typedef struct liste_liaisons;
typedef liste_liaisons*pt_ll;
struct liste_liaisons {
  char *ville_source;
  int energie;
  bool endommage;
  int type;
  char *ville_desti;
  pt_ll lien;
};

C'est la ligne:

Code :

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

strcpy(aux1->ville_source,"Philippe");

qui pose problème... Je ne comprends pas pourquoi...

Merci de me filer un coup de main !

Philippe

[Pragmateek]

Le strcpy fait tout foirer:

Code :

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

aux1->ville_source="Philippe"

tout simple devrait marcher.

Pourquoi?

J'en sais rien,il suffit d'attendre l'avis des pros!

EDIT:Ca doit être un probléme d'initialisation.

EDIT: En effet :

Code :

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

1
2
aux1->ville_source=new(char);
strcpy(aux1->ville_source,"Philippe");

marche.

[z980x]

Ouah, mon sauveur !!!!

Merci, ca marche parfaitement bien !!

Merci beacoup, franchement !

Philippe

[Luther13]

Toujours initialiser un pointeur que l'on declare.

[Luc Hermitte]

Bizarre. Il me semblait avoir déjà répondu.

EDIT: En effet :

Code :

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

1
2
aux1->ville_source=new(char);
strcpy(aux1->ville_source,"Philippe");

marche.

Certainement pas. Cela passe à l'exécution uniquement parce que tu dois compiler en mode débug. Il faudrait réserver bien plus de caractères que cela, et sous forme d'un tableau dynamique qui plus est.
Bref, un truc compiqué à écrire proprement et à maintenir.
Les std::string sont tes amies.