Discussion :

[salseropom]

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

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
 
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)

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

[zenetcalme]

Bonjour,

Je pense que cette petite modification peut supprimer le warning:

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
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;
}

[kessoufi]

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 :

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

1
2
 
default : assert(0);

et le warning peut sauter avec un #pragma ignore

Cdt

[salseropom]

Bonjour,

Je pense que cette petite modification peut supprimer le warning:

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
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"...

[fregolo52]

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)

[zenetcalme]

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...