message d'avertissement à supprimer

Bonjour, j'ai le code suivant (je la fais simple) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 enum TypeTrace { TRACE1, TRACE2 }; enum TypeGraphe { GRAPHE1, GRAPHE2 }; TypeGraphe Trace2Graphe(TypeTrace type) { switch(type) { case TRACE1 : return GRAPHE1; case TRACE2 : return GRAPHE2; } }
ce code me renvoie le message suivant (Visual 2005)

Citation:

warning C4715: not all control paths return a value

je comprends le message, mais pourtant j'ai mis tous les types de mon enum dans ma fonction. Du coup, le default ne sert à rien.

=> que dois-je faire pour ne plus avoir ce message mais je voudrais que ce message apparaisse si j'oublie de mettre un enum dans ma fonction ?

Merci d'avance

Bonjour,

Je pense que cette petite modification peut supprimer le warning:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
enum TypeTrace
{
  TRACE1,
  TRACE2
};
 
enum TypeGraphe
{
   GRAPHE1,
   GRAPHE2
};
 
TypeGraphe Trace2Graphe(TypeTrace type)
{
  TypeGraphe retour = GRAPHE1;
  switch(type)
  {
    case TRACE1 : retour = GRAPHE1;break;
    case TRACE2 : retour = GRAPHE2;break;
  }
  return retour;
}

Salut

en D ça s'appelle final switch, mais en C/C++ je pense qu'il n'y a pas de solution à part un :
Code:

1 2 default : assert(0);
et le warning peut sauter avec un #pragma ignore

Cdt

Citation:
Envoyé par zenetcalme
Bonjour,

Je pense que cette petite modification peut supprimer le warning:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 enum TypeTrace { TRACE1, TRACE2 }; enum TypeGraphe { GRAPHE1, GRAPHE2 }; TypeGraphe Trace2Graphe(TypeTrace type) { TypeGraphe retour = GRAPHE1; switch(type) { case TRACE1 : retour = GRAPHE1;break; case TRACE2 : retour = GRAPHE2;break; } return retour; }
salut,
cela revient donc à faire une sorte de "default"...

Citation:

Envoyé par salseropom

salut,
cela revient donc à faire une sorte de "default"...

Oui c'est ça.
Dans ton message d'origine, si tu n'as ni TRACE1 ni TRACE2, tu retournes quoi ? (c'est le compilo qui le demande, pas moi) ;)

c'est peut être une sorte de default, mais bon, les return au milieu d'un switch, j'essaye d'éviter au maximum.

Après, c'est ton code, et tes règles de codage...