les Astuces du C

**berg** · 07/05/2007, 02h02

Bonsoir,

J ai cree ce poste pour montrer des choses que j ai une chose il n y a pas tres longtemps :

Comment declarer rapidement un pointeur sur un entiers et de l initialiser a 0 par exemple?

Reponse :

Code :

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

int *p = (int[]){0};

Super pratique non ?
Tres franchement c est tellement logique que je n y avait jamais pense :p

Attention ceci reste une declaration static donc pas de free a la fin merci

Si vous avez d autre code comme ca je suis preneur.

**granquet** · 07/05/2007, 02h50

mais qu'est-ce que t'as contre:

Code :

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

int *p = NULL;

c'est pas plus lisible?

**kromartien** · 07/05/2007, 03h34

Utiliser la syntaxe de façon non standard est plus un moyen d'introduire la confusion dans l'esprit du relecteur si l'utilisation ne se justifie pas pour la compression du code, et encore même si àa se justifie, c'est difficile à relire à moins d'être entraîné à la relecture.
______________________________________________________
(Vous pouvez toujours aller faire un tour sur ce site si ça vous intéresse, il y a des chances que vous y trouviez votre bonheur.)

**Emmanuel Delahaye** · 07/05/2007, 08h15

Envoyé par berg

Code :

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

int *p = (int[]){0};

Super pratique non ?

L'art de compliquer les choses... De plus :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:4: warning: ISO C90 forbids compound literals
main.c:4: warning: unused variable `p'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 2 warnings

Je ne vois pas ce qui me ferais changer de

Code :

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

int *p = NULL;

**Franck.H** · 07/05/2007, 11h47

Envoyé par berg

Reponse :

Code :

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

int *p = (int[]){0};

Super pratique non ?

Pourquoi faire simple quand on peut faire compliqué

Si c'est ta devise y'a que toi que ca regarde

**kromartien** · 07/05/2007, 11h58

Envoyé par Franck.H

Pourquoi faire simple quand on peut faire compliqué

Si c'est ta devise y'a que toi que ca regarde

Mon lien est vraiment très bon si vous avez un esprit de compétiteur dans ce domaine.

**souviron34** · 07/05/2007, 13h09

en général la façon normale d'utiliser l'initialiseur global de C est pour les structures ou les tableaux ayant des valeurs constantes...

voir http://c.developpez.com/cours/bernar...gne/node80.php ou
http://c.developpez.com/cours/bernar...00000000000000

mais mêler une telle déclaration (déjà impossible réellement puisque dimension inconnue) à une initialisation relève au minimum de l'inconscience...

**berg** · 08/05/2007, 01h11

Envoyé par Emmanuel Delahaye

L'art de compliquer les choses... De plus :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:4: warning: ISO C90 forbids compound literals
main.c:4: warning: unused variable `p'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 2 warnings

Je ne vois pas ce qui me ferais changer de

Code :

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

int *p = NULL;

Cela ne met pas le pointeur a NULL ...
Cela sert tout simplement a reserver un morceau de memoire egale a 0 ou p pointe.

Code :

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

1
2
3
4
5
6
7
8
int main() {
        int *p = (int[]){0};
 
        if (p == NULL) printf("NULL\n");
        else printf("%d\n", *p);
 
        return 0;
}

cela me donne :

Code :

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

De plus je suis sous linux, je compile avec -Wall je n ai pas aucun warning.
Moi je trouvais que c etait une maniere elegante de creer un pointeur sur entier et de l initialiser a 0...

**koala01** · 08/05/2007, 03h26

Envoyé par berg

Cela ne met pas le pointeur a NULL ...
Cela sert tout simplement a reserver un morceau de memoire egale a 0 ou p pointe.

Code :

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

1
2
3
4
5
6
7
8
int main() {
        int *p = (int[]){0};
 
        if (p == NULL) printf("NULL\n");
        else printf("%d\n", *p);
 
        return 0;
}

cela me donne :

Code :

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

De plus je suis sous linux, je compile avec -Wall je n ai pas aucun warning.
Moi je trouvais que c etait une maniere elegante de creer un pointeur sur entier et de l initialiser a 0...

En fait, ce que tu fais, lors de la déclaration du pointeur *p,c'est de l'initialiser avec l'adresse à laquelle se trouve le premier élément du tableau ne contenant qu'un seul élément de type int...L'élément unique étant initialisé à 0...

Le pire de l'histoire, c'est qu'il est fort vraissemblable que tant p[0](car cela revient en fait à cela) que p en tant que pointeur se trouvent dans la mémoire "read-only" avec les différentes limites que cela impose

Bref, ce n'est ni élégant, ni facile à relire, et très limité d'utilisation...

Autant dire tout de suite que je préférerais encore

Code :

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

1
2
3
 
int entier=0;
int *p=&entier;

qui aura au moins l'avantage de rester cohérent avec ce que l'on veut réellement

**Emmanuel Delahaye** · 08/05/2007, 09h58

Envoyé par berg

Cela ne met pas le pointeur a NULL ...
Cela sert tout simplement a reserver un morceau de memoire egale a 0 ou p pointe.

Code :

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

1
2
3
4
5
6
7
8
int main() {
        int *p = (int[]){0};
 
        if (p == NULL) printf("NULL\n");
        else printf("%d\n", *p);
 
        return 0;
}

cela me donne :

Code :

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

Je ne suis pas sûr que ce code soit correct. Je pense que c'est plutôt une extension de gcc.

De plus je suis sous linux, je compile avec -Wall je n ai pas aucun warning.

Insuffisant :

Code :

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

-Wall -Wextra -ansi -pedantic

Moi je trouvais que c etait une maniere elegante de creer un pointeur sur entier et de l initialiser a 0...

Code :

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

1
2
3
4
5
 
{
   int x = 0;
   char *p = &x;
}

Je ne sais pas si c'est élégant, mais c'est au moins correct. Ce que tu veux faire est une sorte d'extension du mécanisme de définition de pointeurs sur chaines, via un tableau anonyme.

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
 
#include <stdio.h>
 
int main (void)
{
   int *p = (int[]) { 123, 456 };
 
   if (p == NULL)
      printf ("NULL\n");
   else
   {
      printf ("%d\n", p[0]);
      printf ("%d\n", p[1]);
   }
   return 0;
}

L'idée est intéressante, mais à ma connaissance, ce n'est pas supporté par le langage C.

**josse95** · 08/05/2007, 11h56

Ce qui est valide:

Code :

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

1
2
int p0[] = {123,456};
int *p = p0;

C'est moins concis que la forme proposée par berg mais même si on comprend bien ce que cette dernière signifie, elle n'est pas acceptée par tous les compilateurs.

**berg** · 08/05/2007, 13h10

Envoyé par Emmanuel Delahaye

...
Insuffisant :

Code :

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

-Wall -Wextra -ansi -pedantic

....

effectivement cela me donne :

Code :

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

1
2
test.c: In function ‘main’:
test.c:4: warning: ISO C90 forbids compound literals

Donc ok a proscrire ... Dommage ^^ .

**Franck.H** · 08/05/2007, 13h22

Envoyé par berg

effectivement cela me donne :

Code :

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

1
2
test.c: In function ‘main’:
test.c:4: warning: ISO C90 forbids compound literals

Donc ok a proscrire ... Dommage ^^ .

Et puis de toute manière NULL c'est quoi, c'est simplement:

Code :

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

#define NULL ((void *) 0)

Du moins sur Linux, cela peut dépendre de l'implémentation !

**Emmanuel Delahaye** · 08/05/2007, 16h54

Envoyé par Franck.H

Et puis de toute manière NULL c'est quoi, c'est simplement:

Code :

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

#define NULL ((void *) 0)

Du moins sur Linux, cela peut dépendre de l'implémentation !

Non, relis le post. Malgré les apparences, il ne s'agit pas de définir un pointeur NULL comme on a été nombreux à le croire au départ, mais de définir un pointeur sur un tableau anonyme initialisé, comme ça a été illustré par la suite. Il s'agit probablement d'une astuce GNUC, mais pas C.

**Franck.H** · 08/05/2007, 17h23

Envoyé par Emmanuel Delahaye

Non, relis le post. Malgré les apparences, il ne s'agit pas de définir un pointeur NULL comme on a été nombreux à le croire au départ, mais de définir un pointeur sur un tableau anonyme initialisé, comme ça a été illustré par la suite. Il s'agit probablement d'une astuce GNUC, mais pas C.

Arf, en relisant effectivement

**granquet** · 08/05/2007, 17h36

Envoyé par Franck.H

Arf, en relisant effectivement

comme quoi, en plus d'etre seulement GNUC (d'ailleurs ce n'est qu'une supposition?), c'est donc illisible pour le commun des mortels.

et donc a proscrire totalement