Discussion :

[akrlot]

salut;
je sais que le code ci dessous est erronné.je cherche des solutions pour corriger le probleme.

Code :

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

1
2
3
 
char* p;
*p='a';

si on veux faire malloc et que la chaine p est de longueur quelconque faudra t-il necessairement fixer une taille max
merci

[stephl]

salut;
je sais que le code ci dessous est erronné.je cherche des solutions pour corriger le probleme.

Code :

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

1
2
3
 
char* p;
*p='a';

Dans ce cas précis,

Code :

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

char c='a';

suffira.

si on veux faire malloc et que la chaine p est de longueur quelconque faudra t-il necessairement fixer une taille max
merci

Il faut bien connaître le nombre d'octets à allouer avant l'appel à malloc(). Il est néanmoins possible de changer la taille du buffer par la suite à l'aide de realloc() par exemple.

[akrlot]

Dans ce cas précis,

Code :

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

char c='a';

suffira.

Je sais que c'est equivalent a ce que tu as ecrit mais c'est pas ca la question.
au fait je veux un moyen pour pouvoir modifier *p alors que p est seulement declaré char *p .je pense que si j'alloue de la memoire pour p ca resoud le pb?

[stephl]

2 exemples:

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
/*Sans allocation dynamique*/
char c,*p;
 
c='b';
p=&c;
*p='a';
 
------------------------------------------
/*Avec allocation dynamique*/
char *p;
 
if ((p=(char *) malloc(sizeof (char)))==NULL)
 ...
else
 {
 *p='a';
 ...
 free(p);
 }

[Skyounet]

Ou sinon

Code :

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

const char * p = "a";

Mais bon on connait pas bien le contexte, explicite mieux ce que tu veux faire s'il te plait.

[koala01]

Salut,

2 exemples:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/*Avec allocation dynamique*/
char *p;
 
if ((p=(char *) malloc(sizeof (char)))==NULL)
 ...
else
 {
 *p='a';
 ...
 free(p);
 }

Deux choses sur cette partie du code:

• On le répète à longueur de temps, le cast des fonctions *alloc n'est pas nécessaire
• Il n'y a qu'une taille de type de base qui soit connue de manière précise et définitive: c'est justement la taille d'un char: il fait d'office 1

Bref, ton code malloc peut être modifié de manière plus que cohérente en

Code :

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

1
2
if ((p= malloc(1))==NULL)
/*...

Enfin, mais je suis sans doute l'exception qui confirme la règle sur ce point, je préfère ne faire qu'une chose par ligne, et, surtout, éviter de placer une assignation dans un test...

A titre tout à fait perso, j'aurais donc tendance à préférer écrire

Code :

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

1
2
3
4
 
p=malloc(1);
if(p==NULL)
/*...*/

qui revient de toutes façons au même au final, mais qui rend la (re)lecture plus facile

[gl]

Enfin, mais je suis sans doute l'exception qui confirme la règle sur ce point, je préfère ne faire qu'une chose par ligne, et, surtout, éviter de placer une assignation dans un test...

Non, tu n'es pas une exception.

[Trap D]

Enfin, mais je suis sans doute l'exception qui confirme la règle sur ce point, je préfère ne faire qu'une chose par ligne, et, surtout, éviter de placer une assignation dans un test...D

Pour beaucoup de gens qui ont appris le C par l'intermédiaire du K&R, ce n'est pas une énormité que de faire une assignation dans un test, puisque c'était ce qui était écrit dedans.