Le contrôle ne peut pas tomber à cause d'une étiquette de cas

Question

J'essaie d'écrire une instruction de commutation qui saisit le terme recherché dans le champ de recherche en fonction de la zone de texte de recherche présente. J'ai le code suivant. Mais je reçois une erreur "Control cannot fall through from one case label".

switch (searchType)
{
    case "SearchBooks":
        Selenium.Type("//*[@id='SearchBooks_TextInput']", searchText);
        Selenium.Click("//*[@id='SearchBooks_SearchBtn']");

    case "SearchAuthors":
        Selenium.Type("//*[@id='SearchAuthors_TextInput']", searchText);
        Selenium.Click("//*[@id='SearchAuthors_SearchBtn']");
}

Le contrôle ne peut pas tomber d'une étiquette de cas ( case "SearchBooks": ) à un autre

Le contrôle ne peut pas tomber d'une étiquette de cas ( case "SearchAuthors": ) à un autre

Answer 1

Tu as raté des pauses là :

switch (searchType)
{
    case "SearchBooks":
        Selenium.Type("//*[@id='SearchBooks_TextInput']", searchText);
        Selenium.Click("//*[@id='SearchBooks_SearchBtn']");
        break;

    case "SearchAuthors":
        Selenium.Type("//*[@id='SearchAuthors_TextInput']", searchText);
        Selenium.Click("//*[@id='SearchAuthors_SearchBtn']");
        break;
}

Sans eux, le compilateur pense que vous essayez d'exécuter les lignes ci-dessous case "SearchAuthors": immédiatement après les lignes sous case "SearchBooks": ont été exécutées, ce qui n'est pas autorisé en C#.

En ajoutant le break à la fin de chaque cas, le programme quitte chaque cas après qu'il ait terminé, pour toute valeur de searchType .

Answer 2

Vous devez break; , throw , goto o return de chacune de vos étiquettes de cas. Dans une boucle, vous pouvez également continue .

        switch (searchType)
        {
            case "SearchBooks":
                Selenium.Type("//*[@id='SearchBooks_TextInput']", searchText);
                Selenium.Click("//*[@id='SearchBooks_SearchBtn']");
                break;

            case "SearchAuthors":
                Selenium.Type("//*[@id='SearchAuthors_TextInput']", searchText);
                Selenium.Click("//*[@id='SearchAuthors_SearchBtn']");
                break;
        }

La seule fois où ce n'est pas vrai, c'est lorsque les étiquettes des caisses sont empilées comme ceci :

 case "SearchBooks": // no code inbetween case labels.
 case "SearchAuthors":
    // handle both of these cases the same way.
    break;

Answer 3

En C#, vous pouvez faire plus qu'un simple passage, mais vous devez utiliser la "redoutable" instruction goto. Par exemple :

switch (whatever)
{
  case 2:
    Result.Write( "Subscribe" );
    break;
  case 1:
    Result.Write( "Un" );
    goto case 2;
}

Answer 4

Vous devez ajouter une déclaration de rupture :

switch (searchType)
{
case "SearchBooks":
    Selenium.Type("//*[@id='SearchBooks_TextInput']", searchText);
    Selenium.Click("//*[@id='SearchBooks_SearchBtn']");
    break;
case "SearchAuthors":
    Selenium.Type("//*[@id='SearchAuthors_TextInput']", searchText);
    Selenium.Click("//*[@id='SearchAuthors_SearchBtn']");
    break;
}

Cela suppose que vous souhaitiez soit traiter le message SearchBooks cas ou le site SearchAuthors - comme vous l'aviez écrit, dans une instruction de commutation traditionnelle de type C, le flux de contrôle serait "passé" d'une instruction de cas à la suivante, ce qui signifie que les 4 lignes de code sont exécutées dans le cas où searchType == "SearchBooks" .

L'erreur de compilation que vous voyez a été introduite (du moins en partie) pour avertir le programmeur de cette erreur potentielle.

Vous auriez également pu lancer une erreur ou retourner une méthode.

Answer 5

À la fin de chaque switch case, il suffit d'ajouter l'élément break -pour résoudre ce problème

switch (manu)
{
    case manufacturers.Nokia:
        _phanefact = new NokiaFactory();
        break;

    case manufacturers.Samsung:
        _phanefact = new SamsungFactory();
        break;  
}