有什么重构建议吗

Question

我有一个很长的switch语句代码（大约8个案例），它决定了在浏览器中使用什么搜索来查找元素。有没有关于如何重构这段代码的建议？

WebElement CurrentObject=null；切换（SearchBy）{case“className”：尝试{CurrentObject=new WebDriverWait（driver，ConstantValues.LONGWAIT）。直到（ExpectedConditions.presenceOfElementLocated（By.className（SearchPar））；}catch（异常e）{System.out.println（“未找到元素：”e）；}打破

    case "cssSelector":
        try {
            CurrentObject = new WebDriverWait(driver, ConstantValues.LONGWAIT)
                    .until(ExpectedConditions.presenceOfElementLocated(By.cssSelector(SearchPar)));
        } catch (Exception e) {
            System.out.println("Element not found: " + e);
        }
        break;

    case "id":
        try {
            CurrentObject = new WebDriverWait(driver, ConstantValues.LONGWAIT)
                    .until(ExpectedConditions.presenceOfElementLocated(By.id(SearchPar)));
        } catch (Exception e) {
            System.out.println("Element not found: " + e);
        }
        break;

    case "linkText":
        try {
            CurrentObject = new WebDriverWait(driver, ConstantValues.LONGWAIT)
                    .until(ExpectedConditions.presenceOfElementLocated(By.linkText(SearchPar)));
        } catch (Exception e) {
            System.out.println("Element not found: " + e);
        }
        break;

    case "name":
        try {
            CurrentObject = new WebDriverWait(driver, ConstantValues.LONGWAIT)
                    .until(ExpectedConditions.presenceOfElementLocated(By.name(SearchPar)));
        } catch (Exception e) {
            System.out.println("Element not found: " + e);
        }
        break;
   default:
        System.out.println(">>> SEARCH BY KEYWORD IS NOT VALID! <<<");
    }

Answer 1

如果searchBy总是匹配一个方法名，那么我认为反射可能是解决方案。

Method searchMethod = By.class.getMethod(searchBy, returnClass.class);
CurrentObject = new WebDriverWait(driver, ConstantValues.LONGWAIT).until(ExpectedConditions.presenceOfElementLocated(searchMethod.invoke(null,searchPar);

您必须捕获一些可能的异常。

Answer 2

我强烈建议不要以这种方式编写代码，而是创建一个框架。

创建一个SeleniumMutility类（或您想要的任何名称）并编写如下方法：示例（您可以删除WebDriver）

/**
 * Method returns WebElement by Xpath.
 * 
 * @param String xpathExpression
 * @param WebDriver driver
 * @return WebElement
 */
public WebElement getElementByXpath(String xpathExpression, WebDriver driver){
    return driver.findElement(By.xpath(xpathExpression));
}

/**
 * Method returns WebElement by ID.
 * 
 * @param String id
 * @param WebDriver driver
 * @return WebElement
 */
public WebElement getElementByID(String id, WebDriver driver){
    return driver.findElement(By.id(id));
}

或者像这样：

/**
 * Method returns By.
 * 
 * @param String identifier Example: xpath,id,name etc
 * @param String expression Example: //*[@class='text']
 * @return By
 */
public By getBy(String identifier, String expression){
    switch (identifier.toLowerCase()) {
    case "xpath":
        return By.xpath(expression);
    case "id":
        return By.id(expression);
    case "name":
        return By.name(expression);
    case "classname":
        return By.className(expression);
    case "cssselector":
        return By.cssSelector(expression);
    case "linktext":
        return By.linkText(expression);
    case "partiallinktext":
        return By.partialLinkText(expression);
    case "tagname":
        return By.tagName(expression);
    default:
        throw new RuntimeException("Invalid identifier passed: " + identifier);
    }
}

写显式等待和流利等待在不同的类中，并反复使用它。

现在，您的整个代码如下所示：

        try {
            CurrentObject = waitTillElementLocated(getBy("cssselector","SearchPar"));
        } catch (Exception e) {
            System.out.println("Element not found: " + e);
        }

或者

        try {
            CurrentObject = waitTillElementLocated(getBy("id","SearchPar"));
        } catch (Exception e) {
            System.out.println("Element not found: " + e);
        }

Answer 3

Switch-case是许多编程语言中的反模式。为了避免它们，您可以使用一些技术，如在Java中用多态替换条件。我建议将它们与Reflection一起使用。这是Java的一个特点。