[Débutant] Initialiser un tableau de chaine de caractères

**tomasi** · 01/11/2006, 15h14

Bonjour,

je débute sur les chaines de caractères en C, j'ai suivi la doc du site en utilisant la fonction strcpy mais cela ne fonctionne pas.

Voici le code en question, j'essaie d'initialiser le tableau de chaines de 5 caractères nom[N][5] à 0 mais cela ne fonctionne pas

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define N 10
 
char nom[N][5];
int absi[N];
int ord[N];
 
void initTab() //initialise les tableaux à 0
{
  int i;
 
  for(i=0;i<N;++i)
    strcpy(nom[i][5],"0");
 
  for(i=0;i<N;++i)
  {
    absi[i]=0;
    ord[i]=0;           
  } 
}
 
void affiche() //affiche les tableaux
{
  int i,j;
 
  for(i=0;i<N;++i)
    printf("%s",nom[i][5]);
  printf("\n");
 
  for(i=0;i<N;++i)
    printf("%d",absi[i]);
  printf("\n");
 
  for(i=0;i<N;++i)
    printf("%d",ord[i]);  
  printf("\n");       
}
 
int main()
{ 
 
  initTab();
  affiche();
 
  system("pause");
 
  return EXIT_SUCCESS;
}

merci pour votre aide

**Franck.H** · 01/11/2006, 15h22

Salut,

en fait tu peux les initialiser directement pendant leur déclaration comme suit:

Code :

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

char nom[N][5] = {0};

sinon tu peux aussi le faire de cette manière:

Code :

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

1
2
3
4
5
 
for (i = 0; i < N; i++)
{
   memset (nom[i], 0, 5);
}

**Gruik** · 01/11/2006, 15h23

Salux

Tu veux initialiser avec un caractere "0" ou mettre des caracteres nuls (\0) dans les chaines?

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define N 10

char nom[N][5];
int absi[N];
int ord[N];

void initTab() //initialise les tableaux à 0
{
  int i;
     
  for(i=0;i<N;++i)
    strcpy(nom[i]/* [5] */,"0");
      
  for(i=0;i<N;++i)
  {
    absi[i]=0;
    ord[i]=0;           
  } 
}

void affiche() //affiche les tableaux
{
  int i,j;
     
  for(i=0;i<N;++i)
    printf("%s",nom[i]/* [5] */);
  printf("\n");
      
  for(i=0;i<N;++i)
    printf("%d",absi[i]);
  printf("\n");
  
  for(i=0;i<N;++i)
    printf("%d",ord[i]);  
  printf("\n");       
}

int main()
{ 
  
  initTab();
  affiche();
  
  system("pause");

  return EXIT_SUCCESS;
}

**hegros** · 01/11/2006, 15h24

#define N 10
char nom[N][5];
<...>
strcpy(nom[i][5],"0");

Les indices débutent à zéro nom[i][5] est hors borne le dernier élément de la ligne est à l'indice 4.

Dans inittab et affichtab

**tomasi** · 01/11/2006, 16h05

c'est bon, j'arrive à initialiser le tableau

maintenant je cherche à le remplir avec des valeurs rentrées au clavier
tant que la valeur rentrée n'est pas vide il doit stocker cette valeur dans la première case libre (=0) et également stocker 2 autres valeurs dans 2 autres tableaux au même indice que dans le premier

il doit y avoir un souci dans les boucles car il y a une boucle infinie si la chaine est et il ne remplit pas le tableau dans le cas contraire

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define N 10
 
char nom[N][5];
int absi[N];
int ord[N];
 
void initTab() //initialise les tableaux à 0
{
  int i;
 
  for(i=0;i<N;++i)
    strcpy(nom[i],"0");
 
  for(i=0;i<N;++i)
  {
    absi[i]=0;
    ord[i]=0;           
  } 
}
 
void affiche() //affiche les tableaux
{
  int i,j;
 
  for(i=0;i<N;++i)
    printf("%s",nom[i]);
  printf("\n");
 
  for(i=0;i<N;++i)
    printf("%d",absi[i]);
  printf("\n");
 
  for(i=0;i<N;++i)
    printf("%d",ord[i]);  
  printf("\n");       
}
 
int lire ()
{
  int compt=0;
  int i;
  char temp[5];
 
  for(;;)
  {
    printf("nom ?");
    scanf("%s",&temp);
 
    if(temp=="")
      break;
    else
      for(i=0;i<N;++i)
      {
        if(nom[i]=="0")
        {
          strcpy(nom[i],temp);
          printf("abs ?");
          scanf("%d",&absi[i]);
          printf("ord ?");
          scanf("%d",&absi[i]);
          compt++;
        }
      }           
    return compt;
  }       
}
 
 
int main()
{ 
  initTab();
 
  lire();
 
  affiche();
 
 
  system("pause");
 
  return EXIT_SUCCESS;
}

**Emmanuel Delahaye** · 01/11/2006, 18h43

Envoyé par tomasi

maintenant je cherche à le remplir avec des valeurs rentrées au clavier
tant que la valeur rentrée n'est pas vide il doit stocker cette valeur dans la première case libre (=0) et également stocker 2 autres valeurs dans 2 autres tableaux au même indice que dans le premier

il doit y avoir un souci dans les boucles car il y a une boucle infinie si la chaine est et il ne remplit pas le tableau dans le cas contraire

Ton code corrigé et commenté. Pose des questions si tu ne comprends pas.

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define N 10
 
char nom[N][5];
int absi[N];
int ord[N];
 
void initTab ()                 //initialise les tableaux à 0
{
   int i;
 
   for (i = 0; i < N; ++i)
      strcpy (nom[i], "0");
 
   for (i = 0; i < N; ++i)
   {
      absi[i] = 0;
      ord[i] = 0;
   }
}
 
void affiche ()                 //affiche les tableaux
{
   int i
   /* -ed- jamais utilisee...
   , j
   */
   ;
 
   for (i = 0; i < N; ++i)
      printf ("%s", nom[i]);
   printf ("\n");
 
   for (i = 0; i < N; ++i)
      printf ("%d", absi[i]);
   printf ("\n");
 
   for (i = 0; i < N; ++i)
      printf ("%d", ord[i]);
   printf ("\n");
}
 
int lire ()
{
   int compt = 0;
   int i;
   char temp[5];
 
   for (;;)
   {
      printf ("nom ?");
 
      /* -ed-
         scanf("%s",&temp);
 
         pas de & pour un tableau
       */
      scanf ("%s", temp);
 
      /* -ed-
         if(temp=="")
 
         ce n'est pas comme ca que l'on compare des chaines.
         Relire le cours sur les chaines.
 
       */
      if (strcmp (temp, "") == 0)
      {
         break;
      }
      else
      {
         for (i = 0; i < N; ++i)
         {
            /* -EDIT */
            if (strcmp (nom[i], "0") == 0)
            {
               strcpy (nom[i], temp);
               printf ("abs ?");
 
               scanf ("%d", &absi[i]);
 
               printf ("ord ?");
 
               /* -ed-
                  scanf("%d",&absi[i]);
 
                  pas le bon tableau.
                */
               scanf ("%d", &ord[i]);
               compt++;
            }
         }
      }
      /* -ed-
      return compt;
 
      probablement mal place (dans la boucle ? etrange...)
      deplace a la fin...
      */
   }
      return compt;
}
 
 
int main ()
{
   initTab ();
 
   lire ();
 
   affiche ();
 
   system ("pause");
 
   return EXIT_SUCCESS;
}

C'est bien d'utiliser des fonctions, mais tu devrais apprendre aussi à ne pas utiliser de variables globales.

D'autre part les données sont mal organisées. Tu peux sans doute un tableau de structures.

**tomasi** · 01/11/2006, 19h29

merci pour ces corrections

quel est le problème avec les variables globales ?

est-il préférable de tout passer en arguments ?

sinon à l'exécution du code j'ai toujours un souci, il ne me prend pas la valeur du nom et il ne passe pas non plus au remplissage des tableaux ord et absi

**Emmanuel Delahaye** · 01/11/2006, 19h46

Envoyé par tomasi

quel est le problème avec les variables globales ?

http://emmanuel-delahaye.developpez....s.htm#globales

est-il préférable de tout passer en arguments ?

Oui, c'est fait pour...

sinon à l'exécution du code j'ai toujours un souci, il ne me prend pas la valeur du nom et il ne passe pas non plus au remplissage des tableaux ord et absi

Avec quel code ?

**Gruik** · 01/11/2006, 20h18

Envoyé par Emmanuel Delahaye

http://emmanuel-delahaye.developpez....s.htm#globales

J'utilise souvent les globales pour des TAD, mais en static uniquement. Ca correspond un peu à des "variables de classe" privées.
Ex d'utilité : faire un traitement particulier à la premiere utilisation du module.

**Franck.H** · 01/11/2006, 20h20

En générale, si je ne peut faire autrement que d'utiliser une variable globale, je la met en privé et je créé un accesseur/modificateur mais cela est vraiment très rare que j'ai besoin de ce genre de variable.

C'est une manière plus propre je trouve

**Emmanuel Delahaye** · 01/11/2006, 20h32

Envoyé par Gruik

J'utilise souvent les globales pour des TAD, mais en static uniquement. Ca correspond un peu à des "variables de classe" privées.
Ex d'utilité : faire un traitement particulier à la premiere utilisation du module.

Argh ! A quoi ça peut bien servir ?

TADs et globales, sont antinomiques !

**Gruik** · 01/11/2006, 20h51

On peut imaginer que des modules aient besoin de "s'initialiser" (=> recuperer, preparer certaines resources, mais des resources internes au module), et de se "liberer". Cette initialisation pourrait se faire lorsque le premier objet du TAD est construit, et la liberation pourrait se faire à un appel explicite de liberation ( libereTesResources(); ) (ou avec atexit() ou bien un "attribut de fonction destructeur" à la gcc).

Je sais pas, on pourrait imaginer un transformateur de texte qui aurait besoin de lire un fichier de configuration lorsqu'il s'initialise, la construction d'un objet de ce TAD dependrait de ce fichier de configuration

D'un coté, on pourrait creer un autre TAD pour gerer ce "createur de resources"

Code :

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

1
2
3
4
5
6
7
8
9
 
struct resources_parser * rp = create_parser();
 
struct text * my_text = parse (rp, input_text);
 
/* ... */
 
free_text (my_text);
free_parser(rp);

Mais ça serait qd meme lourd de se trimballer cet objet rp partout, d'autant qu'il n'y a pas lieu d'avoir plusieurs instances de "resources_parser"

**Emmanuel Delahaye** · 01/11/2006, 21h22

Envoyé par Gruik

On peut imaginer que des modules aient besoin de "s'initialiser" (=> recuperer, preparer certaines resources, mais des resources internes au module), et de se "liberer". Cette initialisation pourrait se faire lorsque le premier objet du TAD est construit, et la liberation pourrait se faire à un appel explicite de liberation ( libereTesResources(); ) (ou avec atexit() ou bien un "attribut de fonction destructeur" à la gcc).

Ah, du code de goret, OK... Je ne mange pas de ce pain là... Pourquoi pas un Garbage Collector tant que tu y es... On est en C, là, pas en C pointu qui danse la java...

Je sais pas, on pourrait imaginer un transformateur de texte qui aurait besoin de lire un fichier de configuration lorsqu'il s'initialise, la construction d'un objet de ce TAD dependrait de ce fichier de configuration

D'un coté, on pourrait creer un autre TAD pour gerer ce "createur de resources"

Code :

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

1
2
3
4
5
6
7
8
9
 
struct resources_parser * rp = create_parser();
 
struct text * my_text = parse (rp, input_text);
 
/* ... */
 
free_text (my_text);
free_parser(rp);

Mais ça serait qd meme lourd de se trimballer cet objet rp partout, d'autant qu'il n'y a pas lieu d'avoir plusieurs instances de "resources_parser"

OK, dans le cas de gestion de ressources uniques, c'est acceptable. En lecture seule, OK ? (WORM)

**tomasi** · 02/11/2006, 00h03

Envoyé par Emmanuel Delahaye

Dans le code que j'ai posté, il y reste un bug (comparaisons de chaines avec == au lieu de strcmp()).

je ne vois pas à quel endroit...

le code que j'ai essayé est celui posté et corrigé

le souci que je rencontre est qu'il ne break pas quand la chaine rentrée est vide et que si elle n'est pas vide il redemande en boucle les valeurs de abs et ord au lieu de passer au point suivant (triplet nom, abs, ord)

**tomasi** · 04/11/2006, 00h37

personne n'a d'indications pour me sortir de ce mauvais pas ?

**Emmanuel Delahaye** · 04/11/2006, 10h02

Envoyé par tomasi

je ne vois pas à quel endroit...

J'ai corrigé le code et effacé ma remarque...

le code que j'ai essayé est celui posté et corrigé

le mien ?

le souci que je rencontre est qu'il ne break pas quand la chaine rentrée est vide et que si elle n'est pas vide il redemande en boucle les valeurs de abs et ord au lieu de passer au point suivant (triplet nom, abs, ord)

C'est possible. Tu ne sais pas corriger le code ? Ca fait un peu parti de ce métier non ? Faut se creuser la cervelle, ça va venir...

Par exemple : Comment peut-on savoir si une chaine est vide ?
Une fois qu'on le sait, il faut agir en conséquence...

[Débutant] Initialiser un tableau de chaine de caractères

C

Discussions similaires

Partager

Partager