Discussion :

[barucca]

Bonjour a tous!!!

Bon, j'ai encore un probleme avec des chaines. Je suis désolée d'avance car je sais que c'est un sujet traité maintes fois, mais j'ai vraiment du mal....

voila mon code :

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
22
23
 
 
char * p;
char** temp;
int i=0;
 
if (machin)
{
	p = "ma premiere chaine";
	 if (p)    
	 {		
		do
		{			
 
         //on copie p dans temp
			int taille = strlen(p);
			strncpy(temp[i],p,taille);
         //on passe a l'erreur suivante
			p = "ma prochaine erreur";
			i++;
		}while(p)		
	 }
}

En fait, j'ai la chaine p qui change de valeur, et je voudrais mettre dans un tableau de chaine (temp) toutes les valeurs de p.

Mon probleme est que j'ai une erreur lorsque je fais "strncpy(temp[i],p,taille);"

Comment dois je faire pour copier ma chaine p dans mon tableu de chaine temp à l'indice i???

Merci d'avance,
Barucca

[Laurent Gomila]

Avec les bons outils c'est tout de suite plus facile :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <string>
#include <vector>
 
std::string p:
std::vector<std::string> temp;
 
p = "ma premiere chaine";
 
do
{
    temp.push_back(p);
    p = "ma prochaine erreur";
}
while(p != "");

Je t'invite à jeter un oeil à la FAQ pour plus de détails.

[lepoutho]

je suis tout a fait d'accord, mais il n'empeche qu'à titre pédagogique
c'est pas mal de maitriser les char* avant de passer au string et stl.
Enfin ça n'est qu'un avis personnel qui en soit est tout à discutable, j'en
conviens volontiers.
Aussi la permière chose que je remarque en lisant le code est que les
allocations de mémoire et désallocation ne sont pas faites.

si ça peut t'aider ...

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
22
23
24
25
26
27
28
29
30
31
32
33
34
 
 
const unsigned short NB_ERREURS = 1000;
 
int _tmain(int argc, _TCHAR* argv[])
{
	char * erreurCourante = NULL;
	char ** tabErreurCourante = new char * [NB_ERREURS];
	for (int index=0;index<NB_ERREURS;index++) tabErreurCourante [index] = NULL;//indispensable  ...
	bool maCondition = true;
 
	int indexErreur =0 ;
	if (maCondition)
	{
		erreurCourante = "premiere erreur";
		if(indexErreur<NB_ERREURS)
		{
			tabErreurCourante[indexErreur] =new char [strlen(erreurCourante)+1] ; //+1 pour le caractere \0
			sprintf(tabErreurCourante[indexErreur],"%s",erreurCourante);
			indexErreur+=1;
		}
		else
		{
			//allocation insuffisante !!!
		}
	}
	//nettoyage
	for (int index=0;index<NB_ERREURS;index++)
	{
		delete [] tabErreurCourante[index];
	}
	delete [] tabErreurCourante;
	return 0;
}

NB) p , temp , i -> c'est mal !!!!!
A+

[barucca]

bonjour,

merci a vous deux pour votre aide. J'utiliserais les srting et les vectors lorsque je ferais du code qui me sera propre, mais pour le moment je modifie du code existant et je dois garder les char*....
lepoutho : je vais regarder ton exemple et m'en inspirer, j'espere que je vais y arriver!!

merci beaucoup en tout cas.

[barucca]

ca marche, merci!!

[Luc Hermitte]

je suis tout a fait d'accord, mais il n'empeche qu'à titre pédagogique c'est pas mal de maitriser les char* avant de passer au string et stl.
Enfin ça n'est qu'un avis personnel qui en soit est tout à discutable, j'en
conviens volontiers.

Vi.
A titre pédagogique, je préfère que l'"étudiant" maitrise l'algorithmique avant les méandres de la gestion de la mémoire. Comme on faisait en Ada, Pascal, ... quoi.
Et pour ça, rien de tel que d'utiliser les abstractions fournies par le langage.