Discussion :

[telodo]

Bonjour



J'ai fait une méthode comme ceci :

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
 
    protected string definitionQSE()
    {
        string act = Request.QueryString["act"];
        string DossierQSE;
 
        if (act == "M1")
        {
            DossierQSE = "/Prisma_QSE/Processus M1 Définir et déployer la stratégie/";
            return DossierQSE.ToString();
        }
        else if (act == "M2")
        {
            DossierQSE = "/Prisma_QSE/Processus M2 Gérer le SMQE";
            return DossierQSE.ToString();
        }

Pourquoi cela m'affiche une erreur ?

Sinon existe-t-il une autre façon de retourner une variable qui prends une valeure celon certains paramètres ?

Je souhaite ensuite récupérer la valeure de DossierQSE afin de l'utiliser dans d'autres méthodes.


En vous remerciant !

[nicolas.sitbon]

Quelle est l'erreur? donne le code complet qui permet de reproduire l'erreur et pas une partie de la méthode.
Cordialement.

[Nathanael Marchand]

Et si act=="M3" ton programme tire à pile ou face quoi faire?
Ton erreur est "Not all code path return a value" ce qui signifie que lorsque le compilateur essaie tous les chemins possible, il trouve au moins un chemin qui ne passe pas par un return, en l'occurence ici quand act vaut ni M1 ni M2. Même si c'est impossible d'après toi parceque tu as pris des précautions particulières avant, le compilateur ne le sait pas!

[nicolas.sitbon]

Et si act=="M3" ton programme tire à pile ou face quoi faire?
Ton erreur est "Not all code path return a value" ce qui signifie que lorsque le compilateur essaie tous les chemins possible, il trouve au moins un chemin qui ne passe pas par un return, en l'occurence ici quand act vaut ni M1 ni M2. Même si c'est impossible d'après toi parceque tu as pris des précautions particulières avant, le compilateur ne le sait pas!

C'est ce que j'ai supposé, mais n'ayant pas le code complet de la méthode, on ne sait pas ce qu'il fait après avec sa chaine...

[calagan99]

Et si act=="M3" ton programme tire à pile ou face quoi faire?
Ton erreur est "Not all code path return a value" ce qui signifie que lorsque le compilateur essaie tous les chemins possible, il trouve au moins un chemin qui ne passe pas par un return, en l'occurence ici quand act vaut ni M1 ni M2. Même si c'est impossible d'après toi parceque tu as pris des précautions particulières avant, le compilateur ne le sait pas!

Autrement dit rajoute :

Code :

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

1
2
else
return string.Empty;

[olibara]

Autre petit conseil

Code :

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

1
2
3
4
5
6
       if (act == "M1") // utilise plutot act.Equals("M1")
        {
            DossierQSE = "/Prisma_QSE/Processus M1 Définir et déployer la stratégie/";
            return DossierQSE.ToString();
               // ToString ne sert a rien ici, c'est DEJA un string !
        }

[Themacleod1980]

ou tout simplement instancie ton DossierQSE et return le a la fin :

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
 
 protected string definitionQSE()
    {
        string act = Request.QueryString["act"];
        string DossierQSE = ""; /*(ou string.Empty)*/
 
        if (act == "M1") //ou act.Equals("M1")
        {
            DossierQSE = "/Prisma_QSE/Processus M1 Définir et déployer la stratégie/";
            //return DossierQSE.ToString();
        }
        else if (act == "M2") //ou act.Equals("M2")
        {
            DossierQSE = "/Prisma_QSE/Processus M2 Gérer le SMQE";
            //return DossierQSE.ToString();
        }
 
 
return DossierQSE;
}

ou utilise un switch et met une valeur par défaut :

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
 
protected string definitionQSE()
{
   string act = Request.QueryString["act"];
   string DossierQSE; 
 
  switch(act)
  {
     case "M1":
            DossierQSE = "/Prisma_QSE/Processus M1 Définir et déployer la stratégie/";
     break;
     case "M2":
            DossierQSE = "/Prisma_QSE/Processus M2 Gérer le SMQE";
     break;
     Default:
            DossierQSE = String.Empty;
     break;
  }
 
  return DossierQSE;
}