C Problème de passage par référence

Recherche :

Dernière réponse
Sujet : C Problème de passage par référence
wpk	int fect(char user, int size); ou bien ds un esprit plus "objet" typedef struct { char user; int size; } UserString; int fect(UserString * user); ...

Votre réponse

Nom d'utilisateur

Pour poster, vous devez être inscrit sur ce forum .... si ce n'est pas le cas, cliquez ici !

Le ton de votre message

Votre réponse

Smilies

Liste des smilies perso
Wiki smilies
Chercher un smiley

Options

Activer votre signature
Désactiver les smilies
Activer la notification par email du sujet

Aperçu

Vous avez perdu votre mot de passe ?

Vue Rapide de la discussion

wpk	int fect(char user, int size); ou bien ds un esprit plus "objet" typedef struct { char user; int size; } UserString; int fect(UserString * user); ...

Jar Jar

wpk a écrit a écrit :

je dirais plutot que renvoyer un char * alloué ds la fct fect c'est MAL. Y'a rien de mieux pour creer des belles fuites de memoire ou des core à repetition. Pour moi, l'entité qui fait l'allocation, fait egalement la desallocation. Ici, c'est plutot au main d'allouer & desalouer.

Je ne suis pas d'accord. Si tu renvoies un char * alloué, il suffit de le spécifier, et ça se libère à chaque fois. Par contre, si tu l'alloues avant, tu ne peux pas connaître la longueur de la chaîne avant de l'allouer. Conséquences : utilisation inutile de mémoire, limitation arbitraire de la longueur des chaînes, et risques de débordement de buffer. Entre un risque de fuites de mémoire et un risque de débordement, je n'hésite pas longtemps...

wpk

Jar Jar a écrit a écrit :

Tu peux faire comme ça, mais c'est MAL(tm) de modifier des char * dans une procédure.

Code :

int fect(char *user);
int main(int argc, char **argv)
{
int c;
char *user;
user=malloc(20);
c=fect(user);
printf(user);
free(user);
return 0;
}
int fect(char *user)
{
snprintf(user,20,"%s","TEST" );
printf(user);
return 0;
}

Tu comprends ton erreur ? Si tu affectes une valeur à user (ie si tu écris user=quelquechose), tu modifies le pointeur, pas ce vers quoi il pointe.

je dirais plutot que renvoyer un char * alloué ds la fct fect c'est MAL. Y'a rien de mieux pour creer des belles fuites de memoire ou des core à repetition. Pour moi, l'entité qui fait l'allocation, fait egalement la desallocation. Ici, c'est plutot au main d'allouer & desalouer.

Jar Jar

Tu peux faire comme ça, mais c'est MAL(tm) de modifier des char * dans une procédure.

Code :

int fect(char *user);
int main(int argc, char **argv)
{
int c;
char *user;
user=malloc(20);
c=fect(user);
printf(user);
free(user);
return 0;
}
int fect(char *user)
{
snprintf(user,20,"%s","TEST" );
printf(user);
return 0;
}

Tu comprends ton erreur ? Si tu affectes une valeur à user (ie si tu écris user=quelquechose), tu modifies le pointeur, pas ce vers quoi il pointe.

belgique

je veux pas renvoyer de char. je veux passer un vecteur de char par ref. Merci pour la réponse. pas d'autres idées?

Jar Jar

Ouh là, t'as du mal avec les pointeurs, toi... Ta fonction fect n'a modifié à aucun moment ce qui est pointé par user, car tu commences par affecter une nouvelle valeur à ce pointeur.

Un truc mieux :

Code :

char *fect();
int main(int argc, char **argv)
{
char *user;
user=fect();
printf(user);
free(user); /* Ne JAMAIS oublier les free !!!!! */
return 0;
}
char *fect()
{
char *user;
user = malloc(20);
snprintf(user,20,"%s",TEST" );
return user;
}

Tu peux aussi remplacer le malloc et le snprintf par un appel à asprintf(&user,"%s","TEST" ); qui fait l'allocation tout seul, sans limite de taille (c'est mieux).

belgique

Voilà un code très simple mais qui ne merche pas correctement.
Le printf de la fonction fect affiche bien test mais celui du main non. PQ?.... Je désespère.

int fect(char *user);
int main()
{

char *user;
c=fect(user);
printf(user);
return 0;

}
int fect(char *user)
{
user=(char*)malloc(sizeof(char)*20);
user="TEST";
printf(user);
return 0;
}