Control cannot fall through from one case label

You missed some breaks there:

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

Without them, the compiler thinks you’re trying to execute the lines below case "SearchAuthors": immediately after the lines under case "SearchBooks": have been executed, which isn’t allowed in C#.

By adding the break statements at the end of each case, the program exits each case after it’s done, for whichever value of searchType.

Leave a Comment