Discussion :

[BB008]

Bonsoir,

J'ai un petit souci avec l'une de mes fonctions. Dès que je compile sous Visio, j'obtiens ce warning. Je n'arrive pas à comprendre où est le problème. Je suis coincée et je n'arrive pas à avancer. Si quelqu'un aurait l'amabilité de m'aider svp ce serait gentil.

Merci d'avance.

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
int Booking::Reserve(RoomReservation*reserv)
{
 
	int n = reserv->getArrival()->getDay();
	int k = reserv->getNB_nights();
 
	for(int i=0; i<N; i++)
	{
		for(int j = n; j<=((30 *(n-1)-1) + k); j++)
		{
			if(rooms[i][j]==0)
 
			{
				rooms[i][j] = reserv->getNB_reservation();
 
				ReservationRequest = reserv;
 
				return ReservationRequest->getNB_room();
			}
 
		else 
			return 0;
 
		}
	}
}

[ram-0000]

Dès que je compile sous Visio,

Ce ne serait pas Visual Studio plutôt ? Visio est un logiciel de dessin

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
int Booking::Reserve(RoomReservation*reserv)
{
 
	int n = reserv->getArrival()->getDay();
	int k = reserv->getNB_nights();
 
	for(int i=0; i<N; i++)
	{
		for(int j = n; j<=((30 *(n-1)-1) + k); j++)
		{
			if(rooms[i][j]==0)
 
			{
				rooms[i][j] = reserv->getNB_reservation();
 
				ReservationRequest = reserv;
 
				return ReservationRequest->getNB_room();
			}
 
		else 
			return 0;
 
		}
	}
}

Ton else n'est pas terriblement aligné, il devrait être aligné sur le if. Ceci dit, c'est déjà bien, il y a in minimum d'indentation.

Visiblement, tu as une variable globale qui s'appelle N (très mal nommée d'ailleurs). Il est possible que la boucle globale ne soit jamais parcourue si N est négatif et donc effectivement, que tu quittes ta fonction avec une valeur aléatoire ce qui te provoque ton warning.

[BB008]

Oui je suis un peu étourdie : Visual Studio effectivement.
Mais j'ai fait un #define N 10 dans mon .h.
Je ne vois comment N peut-être négatif.
Merci pour votre réponse et merci de m'avoir accordée un peu de votre temps.

Ce ne serait pas Visual Studio plutôt ? Visio est un logiciel de dessin


Ton else n'est pas terriblement aligné, il devrait être aligné sur le if. Ceci dit, c'est déjà bien, il y a in minimum d'indentation.

Visiblement, tu as une variable globale qui s'appelle N (très mal nommée d'ailleurs). Il est possible que la boucle globale ne soit jamais parcourue si N est négatif et donc effectivement, que tu quittes ta fonction avec une valeur aléatoire ce qui te provoque ton warning.

[ram-0000]

Un compilateur n'est pas omniscient. Il arrive à analyser un code mais lorsqu'il devient complexe (2 boucles for imbriquées), il peut avoir des faiblesses.

D'ailleurs en regardant ton code (et en le simplifiant un peu), je le trouve un peu bizarre :

Code :

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

1
2
3
4
5
6
7
8
9
10
for(int i=0; i<N; i++)
{
	for(int j = n; j<=((30 *(n-1)-1) + k); j++)
	{
		if(condition == 0)
			return quelque chose
		else
			return autre chose
	}
}

Tes 2 boucles for ne sont parcourues qu'une seule fois car condition est soit vrai soit fausse donc la fonction est quittée immédiatement sans plus parcourir tes boucles, bizarre !!

[BB008]

Oui je comprends ce que vous voulez dire. Je voulais traduire en code ces phrases : "La fonction Reserve reçoit en paramètre un pointeur sur de réservation. La fonction recherche les chambres disponibles à la période. Si une chambre est disponible, le numéro de réservation sera stocké à l'élément du tableau correspondant à ce numéro de chambre. Si la réservation est acceptée la fonction doit stocker l'objet de réservation dans ReservationRequest. La fonction doit retourner le numéro de chambre réservée ou 0 si la réservation est infructueuse."

Étant débutante en C++, j'ai essayé de traduire de cette manière. Donc si j'ai bien compris je dois modifier mon if.

[Steph_ng8]

Bonsoir.

Donc si j'ai bien compris je dois modifier mon if.

Je dirais que ton if ne devrait pas avoir de else.

Si j'ai bien compris, tu veux parcourir le tableau tant qu'une condition est fausse.
Autrement dit, si la condition est vraie, tu t'arrêtes ; sinon, tu continues le parcours.
Donc, pour résumer :

Code :

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

1
2
3
4
5
6
7
si <condition_vraie>
  alors
    <action>
    stop
  sinon
     <rien>
fin si

[BB008]

Bonsoir,

Merci pour votre réponse.
Le problème est que je dois absolument retourner deux valeurs : une valeur si la condition est vraie, sinon 0 dans le cas contraire.

[Steph_ng8]

Tu ne dois retourner 0 que si tu as parcouru tout ton tableau et que tu n'as pas pu faire la réservation.
Ce qui arrive lorsque les deux deux boucles for (enfin surtout la première, en fait) sont terminées, et que la condition n'a jamais été vraie.

Autrement dit, l'instruction return 0; ne doit pas apparaître à l'intérieur des boucles, mais après.
Ainsi, ça correspondra à ce que tu veux :

• si, durant le parcours, la condition est vraie, alors on enregistre et on s'arrête ;
• sinon, c'est que le parcours est terminé et que la condition n'a jamais été vraie ; alors on renvoie l'information comme quoi on n'a pas pu enregistrer la réservation.

[BB008]

Merci beaucoup pour votre explication. Je n'ai plus de warning. Je n'avais pas compris le problème des for. Encore merci de m'avoir éclairée.

[oodini]

Mais j'ai fait un #define N 10 dans mon .h.

Essaye de perdre l'habitude d'utiliser des

Code :

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

#define

.
Utilise plutôt in

Code :

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

const int.