Copier un char* dans un char[xx]

**ViRouF** · 21/03/2007, 13h55

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 !

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

**DarkBear** · 21/03/2007, 13h57

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)

**ViRouF** · 21/03/2007, 14h00

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

Code :

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

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

**josse95** · 21/03/2007, 14h18

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

Tu dois faire:

Code :

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

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

**Médinoc** · 21/03/2007, 15h29

Envoyé par josse95

Tu dois faire:

Code :

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

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 :

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

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).

**josse95** · 21/03/2007, 15h49

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 !