Stockage d'une valeur de fonction

Version imprimable

Bonjour a tous,

Je suis confronté a un petit problème :

J'ai une fonction :

Code:

1
2
3
4
5
6
char *options_mycat()
{
    char *os;
    os = "abcde";
    return os;
}

Et j'ai un main :

Code:

1
2
3
4
5
6
7
int main()
{
    char *opts;
    opts = options_mycat();
    puts(opts);
    return 0;
}

Puis j'affiche mon programme dans le terminal :

Code:

1
2
3
4
5
"
prompt$ ./a.out
 
prompt$
"

En clair : le résultat de ma fonction ne se stocke pas (du moins pas entièrement) dans ma variable "opts"... Ai-je oublié quelque-chose ?

Rhoblik.

PS : Lorsque je fais un "puts(os);" dans ma fonction tout va bien, "abcde" s'affiche correctement.
PPS : Lorsque je fais un "putchar(opts[0]);" dans mon main, il n'y a d'affiché que le "a" (normal puisque c'est le caractère qui est q l'emplacement 0 de mon char*, mais pourquoi le resultat de la fonction n'est pas stocké entièrement?).
PPPS : Pour ceux qui veulent avoir le vrai code (write_out(p,p), analyse_param(p), prompt(), et putstr(p) sont des fonction perso qui fonctionnent ^^ ) :

Code:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
 
char *options_mycat(int nb, char **params)
{
	int i;
	int y;
	int x;
	int j;
	char opts[3];
	char *os;
	int nb_opt;
  	if (nb > 1)
    {
      	x = 0;
      	for (i = 1; (i < nb) && (params[i] != "--") && (params[i] != "-"); i++)
		{
			if (params[i][0] == '-')
			{
	      		nb_opt++;
	      		for (y = 1; y < strlen (params[i]); y++)
	      		{
	      			if (params[i][y] == 'b' || params[i][y] == 'e' || params[i][y] == 'n')
	      			{
						if (option_cat (opts, params[i][y]) == 'N')
		  				{
		    				opts[x] = params[i][y];
		    				x++;
		  				}
		  			}
		  			else
		  			{
		  				opts[0] = 'i';
		  				opts[1] = params[i][y];
		  				os = opts;
		  				return os;
		  			}
		  		}
	    	}
		}
      	if (nb_opt == 0)
			return "N";
     	os = opts;
      	return os;
    }
  	else
    	return "N";
}
 
int	main (int ac, char **av)
{
	int i;
	char *opts;
	opts = options_mycat(ac, av);
	if (opts[0] == 'i')
	{
		putstr("cat: illegal option -- ");
		putchar(opts[1]);
		puts("");
		puts("usage: cat [-ben] [file ...]");
	}
	else
	{
		putchar(opts[0]);
		putchar(opts[1]);
		putchar(opts[2]);
		putchar('\0');
		if (ac > 1)
			for (i = 1; i < ac; i++)
			{
				write_out(analyse_param(av[i]), opts);
			}
		else
			prompt();
	}
	return 0;
}

25/01/2013, 06h30
Neckara

Bonjour,

Quand on retourne un pointeur, il faut bien faire attention à l'objet pointé.
En effet, si l'objet pointé est déclaré comme étant une variable locale, il sera détruit lors du return.
Ainsi le pointeur pointera sur une zone mémoire non-allouée et générera une segfault.

Mais "maChaine" n'est pas une variable locale, c'est une constante présente dans le "data segment" du programme qui existe du début à la fin du programme donc je ne pense pas que cela devrait poser problème (?).

Par contre même si le C permet d'écrire char * chaine = "maChaine", pour éviter les erreurs, je recommanderais plutôt d'écrire const char * chaine = "maChaine".

Une erreur qui peut arriver c'est de ne pas compiler le bon exécutable, tu peux essayer de rajouter un petit puts("toto");, de faire un clean puis de recompiller pour voir ce que cela donne.

Sinon l'utilisation d'un débogueur me semble plus qu'approprier pour voir ce qu'il se passe pas à pas.
25/01/2013, 10h12
diogene
L'exemple marche parfaitement et ne reflète pas le 'vrai' code
Code:

1 2 3 4 5 6 7 8 9 char *options_mycat(int nb, char **params) { ... char opts[3]; char *os; ... os = opts; return os; ....
Ici tu retournes l'adresse d'une variable locale (&opts[0]) qui est détruite en sortie de la fonction. C'est illégal.
25/01/2013, 20h37
rhoblik

Merci pour vos réponses constructives :)

Neckara,
J'ai déjà fait plein de tests de débogage : mon code fonctionne mais pas comme je veux ^^'
Quoi qu'il en soit merci pour ta recommandation de déclarer des constantes plutôt que des variables seules.

diogene,
Comment puis-je faire, donc, pour retourner tout mon tableau de char (opts[]), étant donné que ma fonction renvoi un char * ? (J'ai chercher le type de retour d'une fonction pour renvoyer un tableau, mais en vain...)
Je n'ai évidement pas le droit d'utiliser la fonction "malloc()", qui me permettrai de traiter directement un char *. C'est pour cette raison que j'utilise un tableau de char d'ailleurs ^^

Ça ne ferai pas 'propre' si je déclare ma variable 'opts' en globale ? :/
Parce que ce serai une solution, non ?

Rhoblik.
25/01/2013, 21h29
Bktero

Pour ne peux-tu pas utiliser malloc() : contrainte d'un enseignant ou de l'environnement cible ?

Une solution potentielle mais bien sale : déclarer le tableau de manière statique et renvoyer l'adresse de son premier élément. Je ne sais même pas quel comportement ça peut avoir, mais a priori, ça pourrait fonctionner.
25/01/2013, 23h13
rhoblik

Encore merci pour vos réponses.

Bktero,
La contrainte concernant le "malloc()" vient bien d'un enseignant ^^

Je vais donc tester cette solution, et je vous tiens au courant.

Rhoblik.

Citation:

Comment puis-je faire, donc, pour retourner tout mon tableau de char (opts[]), étant donné que ma fonction renvoi un char * ?

Si j'en juge par la fonction main(), car la solution à cette question dépend de l'utilisation du tableau par la fonction appelante, la solution naturelle serait :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 char *options_mycat(char * opts, int nb, char **params) { int i; int y; int x; int j; // char *os; ne sert probablement à rien ... return opts; ... return NULL; // ? peut être au lieu de return "N" int main (int ac, char **av) { int i; char opts[3]; char *os = options_mycat(opts, ac, av); ....
Citation:

(J'ai chercher le type de retour d'une fonction pour renvoyer un tableau, mais en vain...)

Tu ne peux pas ! Tu peux renvoyer l'adresse de début d'un tableau alloué en statique ou dynamique mais pas en automatique (variable locale). (A noter que si on enfouit le tableau dans une structure, on peut renvoyer une copie de la structure donc du tableau enfoui)

Citation:

Ça ne ferai pas 'propre' si je déclare ma variable 'opts' en globale ? :/
Parce que ce serai une solution, non ?

Il n'y a pour l'instant aucune raison d'en faire une globale.

27/01/2013, 20h29
rhoblik

Encore merci a vous pour vos réponses.

diogene,
Je me rends compte de ma bêtise en regardant ta solution : Pourquoi je n'y ai pas pense ?? C'est pourtant si 'naturel' comme tu le dis...!

Enfin... quoi qu'il en soit, j'ai testé la méthode de Bktero et celle de diogene : Elle fonctionnent toutes les deux ^^'
Cela dit, j'ai tout de même conservé celle de diogene, a savoir mettre le tableau en argument.

Rhoblik.

PS : Le problème est donc résolu.