Allocation d'un tableau 2D

Version imprimable

Bonjour,

j'essai d'allouer un tableau à 2D en connaissant la taille d'une seule dimension.

et sur chaque ligne je met une chaine de caractère dont je ne connais pas la taille.

le problème et que lorsque je fais l'initialisation par une boucle for j'ai toujours la dernière chaine dans tous mon tableau.

voici mon code :

Code:

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
35
36
37
38
39
40
41
42
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
    char** Aj;
    char num[2];
    char toto[4]="Bon";
    char tmp[4];
    int i,n=7,j=5,k;
 
    Aj=(char**) malloc (n*sizeof(char*));
 
    for(i=0;i<n;++i)
       Aj[i]=(char*) malloc (sizeof(char)*j);
	/*for(i=0;i<n;i++)
        Aj[i]="Bonjo";*/
   /* for(i=0;i<n;++i)
		for(k=0;k<j;++k)
			Aj[i][k] = 'a';*/
   for(i=0;i<j;i++)
   {
	   for(k=0;k<n;++k)
	   {
           strcpy(tmp,toto);
           itoa(i,num,10);
           strcat(tmp,num);
           Aj[i] = tmp ; 
	   }
   } 
 
   //for(i=0;i<n;i++)
    for(i=0;i<j;++i)
		printf("valeur dans le tableau : %s\n",Aj[i]);
 
   //for(i=0;i<n;++i)
     //  free(Aj[i]);
   free(Aj);
 
    return 0;
}

02/07/2008, 12h56
KindPlayer

?

perso je comprends pas tres bien ce que tu veux faire. Tu veux allouer un tableau de pointeur sur chaine de caractere c'est ça? Mais avec quelles chaines de caractères?
02/07/2008, 13h13
fleffle
Citation:
Envoyé par ambessa

Bonjour,

j'essai d'allouer un tableau à 2D en connaissant la taille d'une seule dimension.

Code:

1 2 3 4 5 6 7 8 9 int n=7,j=5; Aj=(char**) malloc (n*sizeof(char*)); for(i=0;i<n;++i) Aj[i]=(char*) malloc (sizeof(char)*j); }
Moi je vois que tu alloues un tableau de 7 lignes ayant chacune 5 colonnes.
Me trompe peut être. Je pige pas bien ton but en fait. Le résultat que tu souhaites.
02/07/2008, 13h16
diogene
Citation:

j'ai toujours la dernière chaine dans tous mon tableau.

C'est normal : tu écrases la valeur de l'adresse des chaînes allouées pour mettre celle de la chaîne temp :

Code:

Aj[i] = tmp ;

Tu dois copier la chaîne dans temp dans la chaîne d'adresse Aj[i] (strcpy)

Il faut effectivement libérer les Aj[i]) avant Aj. Supprimer les commentaires
Code:

1 2 3 //for(i=0;i<n;++i) // free(Aj[i]); free(Aj);
Note : caster le retour de malloc est inutile. Par contre, tester le retour de malloc pour vérifier que l'allocation a réussi est impératif.
02/07/2008, 14h19
Lucien63

Attention à la taille de ton tableau de char tmp il a une taille de 4.

Code:

char tmp[4];

Tu y copie une chaine de 3 char + \0 ce qui fait 4 char

Code:

strcpy(tmp,toto);

Jusqu'a la ça va !

Ensuite tu concane avec une autre chaine de 2 char

Code:

strcat(tmp,num);

Là ! c'est pas bon, tu écris en dehors de ton tableau !

Voici mon code corrigé :

Code:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
    char** Aj;
    char num[2];
    char toto[6]="Bon";
    char tmp[6];
    int i,nl=7,nc=5,k;
 
	//allocation des lignes
    Aj=(char**) malloc (nl*sizeof(char*));
 
	//test de l'allocation
	if(Aj == NULL)return -1;
 
	//allocation des colonnes
    for(i=0;i<nl;++i)
	{
        Aj[i]=(char*) malloc (sizeof(char)*nc);
 
		//test de l'allocation
		if(Aj[i] == NULL)return -1;
	}
 
	//affectation d'une chaine de caractère sur chaque lignes 
    for(i=0;i<nc;i++)
	{
        for(k=0;k<nl;++k)
        {
           strcpy(tmp,toto); //copie de la chaine "Bon" dans tmp
           itoa(i,num,10); //num contient un indice en base 10
           strcat(tmp,num); //concatenation "Bon" + le numero en base 10
		   strcpy(Aj[i],tmp); //copie du contenu de tmp a la ligne i
	   }
   } 
 
	for(i=0;i<nc;++i)
        printf("valeur : %s\n",Aj[i]); //affichage de chaque ligne du tableau
 
	//liberation
    for(i=0;i<nl;++i)
        free(Aj[i]);
    free(Aj);
 
    return 0;
}

Merci beaucoup pour votre aide.

Autre chose

Code:

1
2
3
4
5
6
7
        for(k=0;k<nl;++k)
        {
           strcpy(tmp,toto); //copie de la chaine "Bon" dans tmp
           itoa(i,num,10); //num contient un indice en base 10
           strcat(tmp,num); //concatenation "Bon" + le numero en base 10
	   strcpy(Aj[i],tmp); //copie du contenu de tmp a la ligne i
	}

A quoi sert la boucle for k ?
Tu fait nl fois la même chose ???

02/07/2008, 14h30
ambessa

je veux ajouter à ma chaine un indice de sorte à avoir :

Bon0
Bon1
Bon2
...
02/07/2008, 14h38
Lucien63

Ça j'ai bien compris !
mais tu ne te sert pas de k mais de i dans ton code.
tu as une boucle for en trop.
02/07/2008, 14h44
Emmanuel Delahaye
Citation:

Envoyé par ambessa

j'essai d'allouer un tableau à 2D en connaissant la taille d'une seule dimension.

et sur chaque ligne je met une chaine de caractère dont je ne connais pas la taille.

C'est pas super clair. Ce que je comprends, c'est que tu veux créer un tableau dynamique de pointeurs sur chaines, c'est ça ?
Code:

1 2 3 4 5 6 Entrer le nombre de chaine : 3 Entrer la chaine 1 : hello Entrer la chaine 2 : wild Entrer la chaine 3 : world
et ensuite, par exemple, on peut afficher

Code:

hello wild world

c'est ça ?
02/07/2008, 14h45
ambessa

oui, vous avez raison. :roll:
02/07/2008, 14h47
ambessa

Non pas tout à fait je veux avoir le nombre collé à ma chaine.
02/07/2008, 14h50
Emmanuel Delahaye

Citation:

Envoyé par ambessa

Non pas tout à fait je veux avoir le nombre collé à ma chaine.

Quel nombre ? Collé où ? avant, après ?

Il faut être précis, sinon, on ne peut rien coder de juste. La programmation, c'est pas du hasard...

Ce que je voulais te dire c'est que tu aurai pu remplacer ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
	//affectation d'une chaine de caractère sur chaque lignes 
  for(i=0;i<nc;i++)
	{
      for(k=0;k<nl;++k)
        {
           strcpy(tmp,toto); //copie de la chaine "Bon" dans tmp
           itoa(i,num,10); //num contient un indice en base 10
           strcat(tmp,num); //concatenation "Bon" + le numero en base 10
	   strcpy(Aj[i],tmp); //copie du contenu de tmp a la ligne i
       }
   }

Par ça :

Code:

1
2
3
4
5
6
7
8
	//affectation d'une chaine de caractère sur chaque lignes
    for(i=0;i<nl;i++)
        {
           strcpy(tmp,toto); //copie de la chaine "Bon" dans tmp
           itoa(i,num,10); //num contient un indice en base 10
           strcat(tmp,num); //concatenation "Bon" + le numero en base 10
	   strcpy(Aj[i],tmp); //copie du contenu de tmp a la ligne i
        }

02/07/2008, 15h01
ambessa

ok.
mais est ce que cela me fais écrire sur chaque ligne?
02/07/2008, 15h34
Lucien63

Citation:

Envoyé par ambessa

mais est ce que cela me fais écrire sur chaque ligne?

Ce sont les fonctions strcpy et strcat qui copie dans tes lignes.

A moins que je n'ai pas bien compris ce que tu voulais faire !