Discussion :

[mkrzemin]

Salut à toutes et à tous,

Le titre semble un peu lourd, mais c'est exactement ce qui me pose problème à l'heure qu'il est. Il s'agit là plus d'un question de grammaire que de trouver une solution!
Je m'explique avec le code suivant:

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
 
#include <stdio.h>
#include <stdlib.h>
 
typedef struct
{
	double hauteur;
	double largeur;
} rectangle;
 
typedef struct
{
	unsigned int nbRectangles;
	rectangle *name;
} ensRect;
 
int main(void)
{
	register unsigned int count;
	ensRect *touslesRectangles;
 
	if ((touslesRectangles = malloc(5*sizeof(ensRect))) == NULL)
	{
		return 1;
	}
 
	for (count=0; count<5; count++)
	{
		touslesRectangles[count] = {0, NULL};
	}
 
	return 0;
}

Avec ce code, le compilateur (gcc) m'indique l'erreur suivante:
error: expected expression before ‘{’ token

En revanche, le programme est correctement compilé si je procède à l'initialisation de la façon suivante:

Code :

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

1
2
3
4
5
6
 
	for (count=0; count<5; count++)
	{
		touslesRectangles[count].nbRectangles = 0;
		touslesRectangles[count].name = NULL;
	}

Quelqu'un aurait-il une explication ?

Merci,
Mickaël

[Obsidian]

Bonjour,

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 <stdio.h>
#include <stdlib.h>
…
int main(void)
{
	…
	for (count=0; count<5; count++)
	{
		touslesRectangles[count] = {0, NULL};
	}
	…
 
}

Tu ne peux pas remplir un tableau de cette façon : cette notation n'est valable qu'à la compilation, pour indiquer au compilateur quel doit être l'état de la mémoire occupé par ton tableau au démarrage. À l'exécution, en revanche, ce bloc ne correspond à rien. Tu remarqueras en outre qu'il n'est pas non plus typé et, donc, ne correspond pas spécialement à un ensRect plus qu'à autre chose.

Par contre, tu peux tout-à-fait t'en servir pour déclarer une instance de ensRect et copier cette instance dans ton tableau de structures :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
int main(void)
{
	register unsigned int count;
	ensRect const zero = {0, NULL};
	ensRect *touslesRectangles;
…
 
	for (count=0; count<5; count++)
	{
		touslesRectangles[count] = zero;
	}
…}

Enfin, en C, « NULL » est censé être un marqueur mais en pratique, il est pour ainsi dire toujours égal à « 0 ». Si tu veux faire un programme portable et strictement conforme à la norme, tu procèdes comme ci-dessus, sinon tu initialises directement à zéro la mémoire allouée en utilisant calloc() plutôt que malloc() (à ne pas faire systématiquement, toutefois, car effacer à chaque fois de grandes plages de mémoire est inutilement coûteux).

[diogene]

Une alternative en C99 est de créer un objet ensRect non nommé et initialisé par tes valeurs :

Code :

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

1
2
3
4
	for (count=0; count<5; count++)
	{
		touslesRectangles[count] = (ensRect){0, NULL};
	}

[Kirilenko]

Bonjour,

On a aussi une autre solution en C89 :

Code :

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

1
2
3
4
5
6
7
8
static struct s initializer;
 
/* ... */
 
for (/* ... */)
{
    touslesRectangles[/* ... */] = initializer;
}

Le problème étant que NULL peut avoir une représentation qui n'est pas « tous-bits-0 ». Or, avec des fonctions comme memset ou calloc, on touche à du bas niveau, et le compilateur ne contrôle plus ce qu'il est en train de remplir (ce n'est qu'une grosse zone mémoire générique). Par contre, quand 0 est utilisé dans le cadre des pointeurs, le compilateur doit en théorie se charger de le convertir vers la réelle représentation qui est utilisée par la machine.

Code :

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

void *p = 0; /* Légal */

Le seul cas limite est l'appel de fonctions à nombre variable d'arguments. Dans ce cas-là, il faudra transtyper la constante, sans quoi le compilateur ne pourra pas savoir qu'il s'agit d'un pointeur. Exemple :

Code :

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

1
2
void f(const char *fmt, ...);
void g(void) { f (fmt, (char *)0); }

Bonne journée !

ÉDIT : 0 ne passe pas avec [ CODEINLINE ] ; ça donne ça : « 0 ».

[mkrzemin]

les gars, ça sent la maîtrise!

En effet, j'avais aussi remarqué, comme l'a fait remarqué diogene, qu'en typant mon initilisation, ça marchait. Je comprends maintenant son intérêt!

Sinon, j'ai une préférence pour l'utilisation de NULL en C. Il ne correspond pas tout à fait à 0, mais plutôt à une macro définie comme ((void *)0), que tu peux toujours redéfinir pour la portabilité de ton programme! Après, tout est histoire de goût!

Encore merci,
Mickaël

[ternel]

les gars, ça sent la maîtrise!

En effet, j'avais aussi remarqué, comme l'a fait remarqué diogene, qu'en typant mon initilisation, ça marchait. Je comprends maintenant son intérêt!

Sinon, j'ai une préférence pour l'utilisation de NULL en C. Il ne correspond pas tout à fait à 0, mais plutôt à une macro définie comme ((void *)0), que tu peux toujours redéfinir pour la portabilité de ton programme! Après, tout est histoire de goût!

Encore merci,
Mickaël

Redéfinir NULL? quelle horreur!

[mkrzemin]

Redéfinir NULL? quelle horreur!

Ce que je voulais dire, ce n'est pas RE-définir, mais plutôt définir, tout simplement, dans l'hypothèse où il ne l'est pas!

Mickaël

[ternel]

<stdlib.h> est là pour ca.
Il définit NULL, et il ne faudrait surtout pas que ton NULL soit différent de celui-là, ca ferait désordre