Discussion :

[yohannc]

Bon voila, je suis un peu noob avec les pointeurs et j'ai un bout de code que je n'arrive pas à résoudre, si quelqu'un pourrait m'expliquer pourquoi ça bug (et non me referaire tout le code pour avoir la même fonctionnalité).

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
20
#include <stdio.h>
char* retournerJour(int day);
 
 
 
int main(void){
	int jourChoisit;
	char* ptr;
	scanf("%d",&jourChoisit);
	ptr = retournerJour(jourChoisit);
	printf("\n 1: %s  \n",ptr);
	printf("\n 2: %s  \n",ptr);
	system("pause");
}
 
char* retournerJour(int day){
	char jours2[7][9] ={"Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi","Dimanche"};
	printf("\n----->%s\n",jours2[day-1]);
	return jours2[day-1];
}

Le but du prog c'est de choisir au départ le numéro du jour de la semaine.
La fonction devrait renvoyer l'adresse de la dimension correspondant au jour.

Merci.

[homeostasie]

Le fonction:

Code :

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

1
2
3
4
5
char* retournerJour(int day){
	char jours2[7][9] ={"Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi","Dimanche"};
	printf("\n----->%s\n",jours2[day-1]);
	return jours2[day-1];
}

retourne un pointeur sur une variable locale, donc une fois que cette fonction se termine, cette valeur n'est plus valide (valeur stockée dans la pile).

Une solution serait de passer ton pointeur "ptr" à la fonction retournerJour(), en omettant pas d'allouer de la mémoire pour celui-ci.

[ram-0000]

Ce qui peut se faire aussi, c'est de retrounber un pointeur sur une variable statique. Cela ne pose pas de problème car c'est une variable qui n'est jamais modifiée.

Code :

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

1
2
3
4
5
6
7
8
9
 
const char * retournerJour(int day)
{
static const char * jours[] =
{ "Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi", "Dimanche" };
 
   printf("\n----->%s\n", jours[day - 1]);
   return jours[day - 1];
}

Par contre, il manque des test pour garantir que le parametre day de ta fonction sera OBLIGATOIREMENT entre 1 et 7 sinon, cela part en cacahuetes.

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
 
char* retournerJour(int day){
	char jours2[7][9] ={"Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi","Dimanche"};
	printf("\n----->%s\n",jours2[day-1]);
	return jours2[day-1];
}

Le but du prog c'est de choisir au départ le numéro du jour de la semaine.
La fonction devrait renvoyer l'adresse de la dimension correspondant au jour.

Pour ça, il faut retourner l'adresse d'une chaine et non d'un tableau de char local (comportement indéfini).

Ceci fonctionne. Attention, en cas d'erreur de numéro de jour (<1 ou >7), le comportement est indéfini.

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
20
21
22
23
24
25
26
27
28
 
char const *retournerJour (int day)
{
   static char const *jours[] =
      { "Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi",
      "Dimanche"
   };
   return jours[day - 1];
}
 
#ifdef TEST
#include <stdio.h>
#include <assert.h>
 
int main (void)
{
   assert (strcmp (retournerJour (1), "Lundi") == 0);
   assert (strcmp (retournerJour (2), "Mardi") == 0);
   assert (strcmp (retournerJour (3), "Mercredi") == 0);
   assert (strcmp (retournerJour (4), "Jeudi") == 0);
   assert (strcmp (retournerJour (5), "Vendredi") == 0);
   assert (strcmp (retournerJour (6), "Samedi") == 0);
   assert (strcmp (retournerJour (7), "Dimanche") == 0);
 
   puts ("P A S S E D");
   return 0;
}
#endif

[yohannc]

Merci bien de vos réponses rapides .
J'ai bien compris l'erreur, fallait y penser. Par contre visual studio gère ça bizarement. Quand je pointais la variable ptr (après le passage de la fonction) il voyait toujours "Vendredi" quand je mettais 5.

[Emmanuel Delahaye]

Quand je pointais la variable ptr (après le passage de la fonction) il voyait toujours "Vendredi" quand je mettais 5.

Ne pas perdre son temps à essayer d'expliquer un comportement indéfini. Tout peut arriver...