les Astuces du C

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:

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.

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

Code:

int *p = NULL;

c'est pas plus lisible? :aie:

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.)

Citation:

Envoyé par berg

Code:

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

Super pratique non ?

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

Code:

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:

int *p = NULL;

Citation:

Envoyé par berg

Reponse :

Code:

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

Super pratique non ?

Pourquoi faire simple quand on peut faire compliqué :aie: Si c'est ta devise y'a que toi que ca regarde :mouarf:

Citation:

Envoyé par Franck.H

Pourquoi faire simple quand on peut faire compliqué :aie: Si c'est ta devise y'a que toi que ca regarde :mouarf:

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

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...

Citation:
Envoyé par Emmanuel Delahaye
L'art de compliquer les choses... De plus :
Code:

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:

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:

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:

0

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...

Citation:
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:

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:

0

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 :P

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:

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

Citation:
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:

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:

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

Citation:

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

Insuffisant :

Code:

-Wall -Wextra -ansi -pedantic

Citation:

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

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:

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.

Ce qui est valide:
Code:

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.

Citation:

Envoyé par Emmanuel Delahaye

...
Insuffisant :

Code:

-Wall -Wextra -ansi -pedantic

....

effectivement cela me donne :
Code:

1 2 test.c: In function ‘main’: test.c:4: warning: ISO C90 forbids compound literals
Donc ok a proscrire ... Dommage ^^ .

Citation:
Envoyé par berg
effectivement cela me donne :
Code:

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:

#define NULL ((void *) 0)

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

Citation:

Envoyé par Franck.H

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

Code:

#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.

Citation:

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 :aie:

Citation:

Envoyé par Franck.H

Arf, en relisant effectivement :aie:

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 :aie:

Citation:

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.
....

Pourtant je l avais deja dit, comme quoi ca ne doit pas etre vraiement facile a lire pour le commun des mortels :king:

Je suis en train d'étudier Blender, me gave de tutos depuis ce matin 10h donc on va mettre ca sur le compte de l'inattention :aie: :mouarf: