Copier un char* dans un char[xx]

Bonjour,


Pourriez-vous me dire comment copier le contenu de la chaine str1, en chaine1 de la structure déclarée. J'ai pas le choix des types et faut que ça fonctionne ... j'ai essayé tout ce qui me venait en tête ! 8O

Code:

---

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

#include <iostream>   using namespace std;   typedef struct { char chaine1[10]; } maStructure; typedef maStructure * strucPtr;   int main(){   char * str1 = "blablabla"; strucPtr str2;   strcpy(str2[0].chaine1,str1);   cout << str2[0].chaine1 << endl;   return 0; }

---

Ce code compile, mais j'ai un segmentation fault ... :(

Merci d'avance :oops:

As-tu essayé sans le [0] sur ta str2 ?

(désolé je peux pas tester j'ai pas de compilo sous la main mais ton [0] ne parait inutile)

C'est bien ça normalement, puisque c'est un tableau de structure. Si je ne mets pas le [0], lors de la compilation :

Code:

---

main.cpp:15: error: request for member `chaine1' in `str2', which is of non-class type `maStructure*'

---

strucPtr définit un pointeur;
str2 n'a donc pas de place allouée.

Tu dois faire:

Code:

---

1 2	str2 = (strucPtr) malloc(sizeof(*str2)); strcpy(str2->chaine1,...);

---

Tu déclares str2 comme étant un pointeur sur un (ou plusieurs) maStructure, mais tu ne le fais pointer nulle part.

Merci beaucoup de votre aide et de votre explication, ça fonctionne.

:yaisse2:

oui c'est vrai les malloc :aie: (j'oublie tous le temps )

par contre le [0] est bien inutile et remplace ton . par ->

Citation:

---

Envoyé par josse95

Tu dois faire:

Code:

---

1 2	str2 = (strucPtr)malloc(sizeof(*str2)); <snip>

---

---

Inapproprié pour le C++ :

• Utilisation d'un cast de type C là où un static_cast suffirait.
• Utilisation de malloc() alors qu'en C++, on dispose du new qui n'exige aucun cast.

De plus, je déconseille d'utiliser le simple strcpy() pour un buffer limité. Une méthode sûre en C est:

Code:

---

1 2	str2->chaine1[0] = '\0'; strncat(str2->chaine1, str1, sizeof(str2->chaine1)-1);

---

À noter qu'on peut encore utiliser mieux que sizeof(), genre une macro type ARRAYSIZE() (celle-ci est fournie par Microsoft).

C'est vrai qu'on est dans le forum C++. Mais bon, tout ça, c'est du chipotage (ou du fignolage si tu préfères).
A part éventuellement l'utilisation du strncat à la place du strcpy qui fera que ton programme ne plantera pas (en cas de débordement) mais ne fera quand même pas ce que tu attends qu'il fasse !