Discussion :

[chris670]

1) j'ai créé un buffer[10]

char buffer [10]

2) je demande une chaine alphanumerique
3) je l'enregistre dans buffer

cin>>buffer;
int n = strlen(buffer);

4) je crée un **p

char **p=new char*[n];

5) et j'alloue de la mémoire pour n (**p)

COMMENT FAIT ON CORRECTEMENT

6) je voudrai copier dans chaque p la chaine enregistrée dans buffer

est-ce juste de procéder comme suit :

for (i=0;i<n;i++)
(p+i)=buffer;

et lorsque je veux afficher le contenu

for (i=0;i<n;i++)
cout<<**(p+i);

7) delete p;
est ce juste de libérer la mémoire comme ceci


merci

[bigboomshakala]

salut

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	char buffer [10];
	cin>>buffer;
 
	int taille = 5;
	int n = strlen(buffer); 
 
	char **p=new char*[taille]; 
	for (int i=0; i<taille; i++)
	{
		p[i] = new char[n];
		strcpy(p[i],buffer);
		cout<<p[i]<<endl; 
	}

sans oublier les include qui vont bien.

(NOTE : sinon plus simple et plus robuste - utiliser std::string et std::vector)

[Luc Hermitte]

Correctement,
- Tu ne remplis pas un buffer à taille limité avec l'opérateur d'extraction appliqué sur un flux (>>)
- Tu n'enchaines pas des new explicites sans t'occuper de gérer les éventuelles exceptions.

a- => std::string (souple) ou std::istream::getline avec limitation du nombre de caractères (inutilement rigide)
b- => divers algos (/bidouilles) plus ou moins complexes (FAQ, non ?), ou vecteur de pointeurs intelligents (dans le cas général), ou dans le cas présent : vecteur de chaines (std::vector<std::string>).