Taille tableau par utilisateur

**nicodn02** · 26/04/2007, 13h32

j'ai un petit soucis lors de la compilation. Voici l'erreur :

"subscripted value is neither array nor pointer "

et voici la fonction qui "beug"

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
 
void draw_grille(char *grille, int nb_lig, int nb_col){
     int i, j;
 
     for(i=0; i<nb_lig; i++){
 
         for(j=0; j<nb_col; j++){
             printf("%c", grille[i][j]);
         }
 
         printf("\n");  
     }
 
     printf("\n");
}

il me dit que l'erreur vient du printf :

Code :

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

1
2
 
printf("%c", grille[i][j]);

Voici le reste de tout 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
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
 
#include <stdio.h>
#include <stdlib.h>
 
void draw_grille(char *grille, int nb_lig, int nb_col);
void initialisation_grille(char *grille, int nb_lig, int nb_col);
 
 
int main (void){
    int col, lig;
    char *grille=NULL;
 
  scanf("%d", &col);
  scanf("%d", &lig);
 
   grille = malloc ( col*lig );
   if ( grille == NULL )
     {
         fprintf ( stderr, "Erreur d'allocation\n");
         return EXIT_FAILURE ;
     }
 
  initialisation_grille(grille, lig, col);
  draw_grille(grille, lig, col);
 
  free(grille);
 
  system("PAUSE");	
  return EXIT_SUCCESS;
}
 
 
void initialisation_grille(char *grille, int nb_lig, int nb_col)
{
    int i, j;
 
    for (i=0; i<nb_lig; i++)
    {
        for (j=0 ; j<nb_col; j++)
        {
            grille[i*nb_col + j] = '.';
        }
 
     }
}
 
 
void draw_grille(char *grille, int nb_lig, int nb_col){
     int i, j;
 
     for(i=0; i<nb_lig; i++){
 
         for(j=0; j<nb_col; j++){
             printf("%c", grille[i][j]);
         }
 
         printf("\n");  
     }
 
     printf("\n");
}

**Médinoc** · 26/04/2007, 13h34

Tu dois utiliser grille[i*nb_col + j] au lieu de grille[i][j]

**nicodn02** · 26/04/2007, 14h21

Cela focntionne, merci bien
Par contre, encore une question, toujours

, j'avais cru comprendre que l'utilisation ce faisait aussi par l'intermédiaire du sizeof auquel on rajoute le type de bloc memoire a allouer. Pourquoi dans le code on ne le fait pas (cf ci dessus) ?

et pourquoi ?

Code :

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

1
2
 
grille = malloc (col*lig);

Je pense que sa va liébrer un espace, un "carré", d'aire col*lig, non ?

Aussi, sans l'utilisation du sizeof, le code reste-t-il propre et correcte, ou l'utilisation du sizeof est mieux?

Puis-je rajouter une structure a mon projet ? Dans quel but ? ( je m'explique, car cela ferait très bien face aux profs, sur une notion que l'on vient de commencer

et que tout le monde n'utiliserais pas)

**Médinoc** · 26/04/2007, 14h34

Le sizeof() a été omis ici car sizeof(char) est toujours égal à 1.
Je considère ceci comme une mauvaise pratique, car si l'on change le type de grille, ça foire. Je considère de rajouter sizeof(*grille) dans le malloc().

Note: grille n'a pas besoin d'être initialisé pour que ça marche, car sizeof(*unPointeur) ne déréférence pas le pointeur en question.

Edit: Souviron34, TU SORS!
On ne met pas sizeof(unType) quand on peut faire autrement.

**souviron34** · 26/04/2007, 14h35

ton tableau est un tableau de char.

En C par définition la taille d'un char est 1.

Donc si tu veux tu peux mettre :

Code :

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

1
2
 
malloc ( col*lig*sizeof(char) )

mais sizeof(char) sera toujours 1.

Si par contre tu avais des int, des floats, des doubless des structs, là oui il te faudrait mettre sizeof().

**nicodn02** · 26/04/2007, 14h37

Merci beaucoup

A venir : mon explication de tout cela pour voir si je peut bien expliquer devant les profs
Et en ce qui concerne la structure, personne n'a une idée ? ( par exemple de passer le tableau avec les clo et lig dans une structure, mais après l'interet?)

**nicodn02** · 26/04/2007, 14h41

En revenant sur tes paroles Médinoc :
"Note: grille n'a pas besoin d'être initialisé pour que ça marche, car sizeof(*unPointeur) ne déréférence pas le pointeur en question."
Tu parle de ma fonction initialisation_tableau ou tu parle d'un autre type de initialisation?

et en ce qui concerne le sizeof(*grille) je n'y avait pas pensé, comme sa, si 'lon change le type de grille, et ma le sizeof fonctionnera toujours car *grille est de type int, est-ce cela?

**Thierry Chappuis** · 26/04/2007, 14h45

Envoyé par nicodn02

En revenant sur tes paroles Médinoc :
"Note: grille n'a pas besoin d'être initialisé pour que ça marche, car sizeof(*unPointeur) ne déréférence pas le pointeur en question."
Tu parle de ma fonction initialisation_tableau ou tu parle d'un autre type de initialisation?

et en ce qui concerne le sizeof(*grille) je n'y avait pas pensé, comme sa, si 'lon change le type de grille, et ma le sizeof fonctionnera toujours car *grille est de type int, est-ce cela?

oui.

Thierry

**nicodn02** · 26/04/2007, 14h50

Mais pourquoi cela :

Note: grille n'a pas besoin d'être initialisé pour que ça marche, car sizeof(*unPointeur) ne déréférence pas le pointeur en question

quelle initialisation?

**Médinoc** · 26/04/2007, 14h58

Si je fais:

Code :

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

1
2
3
4
int * grille;
size_t taille = lig*col* sizeof(*grille);
 
grille = malloc(taille);

Là, grille n'est pas initialisé quand je fais le sizeof. Le pointeur peut pointer n'importe où, et souvent à une adresse invalide.
Mais le sizeof() s'en moque, donc ce n'est pas un problème.

grille sera initialisé après la dernière ligne, où on lui affecte l'adresse retournée par malloc().
Attention : Le pointeur grille sera initialisé, mais pas les données pointées.

**nicodn02** · 26/04/2007, 15h02

Si j'ai bien tout compris, pour allouer dynamiquement un espace memoire sur la machine, on fait :

declaration d'un poiteur sur un tableau(pour mon exemple de projet) car lallocation, comme elle est dynamique, le pointeur permettera de se diriger vers cette allocation
l'utilsiation de la fonctione malloc permet cette allocation dynamique
En "parametre" de cette fonction, l'on doit indique la taille a allouer avec le type de donnée qui vont remplir cette allocation.. de la forme

grille(le pointeur sur l'espace)=malloc(nb_col * nb_lig * sizeof(*grille))
*grille etant le type

Est-ce bien tout cela ou ai-je oublier quelque choses ?
Rectifier moi s'il vous plait, sur n'importe quel point ou terme que j'utilise, comme c'est pour une présentation a l'oral, merci

**Thierry Chappuis** · 26/04/2007, 15h06

Envoyé par nicodn02

Merci beaucoup

A venir : mon explication de tout cela pour voir si je peut bien expliquer devant les profs
Et en ce qui concerne la structure, personne n'a une idée ? ( par exemple de passer le tableau avec les clo et lig dans une structure, mais après l'interet?)

L'intérêt de mettre le tableau et ses dimensions dans une structure est que tu peux manipuler cela comme une seule entité, un seul "object". On parle de type abstrait de donnée lorsque tu crées un nouveau type comme par exemple:

Code :

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

1
2
3
4
5
6
typedef Tableau2D_ Tableau2D;
struct Tableau2D_ {
    int lignes;
    int colonnes;
    int *tableau;
};

ansi que les fonctions qui permettent de le manipuler. L'exemple donnée par millie, par exemple, est instructif de ce point de vue. Tout ce que tu as à faire, c'est de passer ton objet de type Tableau2D aux différentes fonctions, et tu n'as plus à t'occuper des dimensions, ni de la façon dont est implanté le tableau en interne. Les fonctions représentes les différentes actions possibles avec l'objet considéré, et tant que tu utilises cette interface, tu n'as pas besoin de savoir si on accède aux cases du tableau en faisant tableau[i*colonnes + j] ou tableau[i][j]. Tu te contente de faire:

Code :

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

1
2
3
4
Tableau2D tab;
tableauInit(&tab, 3, 3);
tableauFixerValeur(&tab, 0, 0, 100);
tableauDetruire(&tab);

On appelle cela l'encapsulation des données, et c'est un concept central en programmation orienté-objets.

Thierry

**nicodn02** · 26/04/2007, 15h12

Tu penses que cela ajouterais une autre facon de voir le projet, c'est à dire un projet plus propre, plus evolué et plus facile de maintient?
Qu'apporterait de plus une structure dans mon projet?

(Et en ce qui concerne mon "micro discours" sur l'explication du malloc ?)

**millie** · 26/04/2007, 17h06

Dès que tu commences à faire de gros projet, on essate de s'abstraire au maximum de l'implémentation (le code) du programme. On cherche à déterminer des "objets" (l'exemple du tableau est un bon exemple) et les méthodes (ou les opérations) que tu souhaites faire dessus.

De manière conceptuel, tu définis un tableau (totalement indépendant de son implémentation) dont les opérations dessus sont :

Code :

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

1
2
3
4
5
6
7
 
void tableauInit(Tableau2D* tableau, size_t largeur, size_t hauteur)
int tableauGetValue(Tableau2D* tableau, unsigned int posx, unsigned int posy)
void tableauSetValue(Tableau2D* tableau, unsigned int posx, unsigned int posy, int value)
void tableauDelete(Tableau2D* tableau)
size_t tableauGetHeight(Tableau2D* tableau)
size_t tableauGetWidth(Tableau2D* tableau)

Ca permet de s'abstraire totalement du fonctionnement du tableau. En effet, on t'a donné plusieurs implémentations possibles : une avec malloc, une sans, une avec plein de malloc. Mais ceci n'est finalement pas important pour l'utilisateur de ton "objet". Ce qu'ils souhaient uniquement savoir, ce sont les opérations minimals qui sont dessus.

C'est un concept essentiel (concept objet en C) à prendre en compte dès que tu développes de gros applications et que tu travailles en équipe.
Ca permet également d'avoir une meilleure compréhension du code (les choses ont une bonne sémantique et on comprend tout de suite quoi fait quoi)

**nicodn02** · 26/04/2007, 17h14

En ce qui concerne les autres fonctions a part le main(), c'est OK pour cela :
initialisation_grille
draw_grille
place_pion_vertical
test_puissance4
menu...

Mais ce que je voudrais savoir est: est-ce que l'apport d'une structure, comprenant par exemple le tableau (plutôt le pointeur, char), et le nb_col(int) et nb_lig(int), m'avancerais à quelques choses? un gain?

**Thierry Chappuis** · 26/04/2007, 17h23

Envoyé par nicodn02

En ce qui concerne les autres fonctions a part le main(), c'est OK pour cela :
initialisation_grille
draw_grille
place_pion_vertical
test_puissance4
menu...

Mais ce que je voudrais savoir est: est-ce que l'apport d'une structure, comprenant par exemple le tableau (plutôt le pointeur, char), et le nb_col(int) et nb_lig(int), m'avancerais à quelques choses? un gain?

Oui, millie vient de t'expliquer cela très bien.

Thierry

**nicodn02** · 26/04/2007, 17h28

Merci de votre aide a tous, maintenant, je vais essayer de me débrouiller pour l'utilisation de ma structure au sein de mon projet..

Taille tableau par utilisateur

C

Discussions similaires

Partager

Partager