WebDriver factory без IF-ELSE

В этой заметке покажу, как можно написать WebDriver factory без применения IF-ELSE. Зачастую во фреймворках в случае кросс-браузерного тестирования приходится писать драйвер фэктори. Большинство реализаций, которые мне приходилось видеть, выглядели так:

BrowserFactory.class

class BrowserFactory{
    public static final WebDriver DEFAULT_FF = new FirefoxDriver();

 public static WebDriver getDriver(String name){
     WebDriver driver = null;
     if("FIREFOX".equalsIgnoreCase(name)){
        driver= new FirefoxDriver();
     }else if("CHROME".equalsIgnoreCase(name)){
         driver = new ChromeDriver();
     }else if("IE".equalsIgnoreCase(name)){
         driver = new InternetExplorerDriver();
     }else {
         driver = DEFAULT_FF;
     }
     return driver;
 }

Но такая реализация, скажем так, не самая хорошая: читабельность кода страдает, человеку приходится гадать, как же правильно задать имя браузера, ну и самое неприятное - куча if-else. Давайте посмотрим, как можно улучшить ситуацию. Объектно-ориентированные языки программирования обладают мощнейщим принципом - полиморфизмом. Его мы и будем использовать, чтобы удалить if-else в коде. В результате читабельность кода значительно повысится.

Рефлексия

Первое "топорное" решение (я назвал бы его "решением для реальных пацанов"):

public class Browsers{
    public static WebDriver newBrowser(String name){
        return Class.forName(name).newInstance();
    }
}

Смотрится очень просто, НО сложность заключается в том, что человек должен помнить полное имя класса. К тому же, использование рефлексии повышает степень брутальности в пару раз. Поэтому таких решений следует избегать.

Map

Второе решение более приемлемое - использование Map.

public class Browsers{
    private static Map<String,WebDriver> drivers = new HashMap<>(){
        put("CHROME",new ChromeDriver());
        put("FIREFOX",new FirefoxDriver());
        put("IE",new InternetExplorerDriver());
    }

    public static WebDriver newDriver(String name){
        return drivers.get(name);
    }
}

Это решение обладает недостатком: нужно знать правильное имя драйвера, которое задано в Map.

Switch

switch (name) {
    case "FIREFOX": return new FirefoxDriver();
    case "CHROME": return new ChromeDriver();
    case "IE": return new InternetExplorerDriver();
    default: return DEFAULT_FF;
}

Enum

Самое элегантное решение:

public enum Browsers {
    CHROME {
        public WebDriver create(){
            return new ChromeDriver();
        }
    },
    IE {
        public WebDriver create(){
            return new InternetExplorerDriver();
        }
    },
    FIREFOX {
        public WebDriver create() {
            return new FirefoxDriver();
        }
    };

    public WebDriver create(){
        return null;
    }
}

class Test {
    public static void main(String[] args) {
       WebDriver driver = Browsers.FIREFOX.create();
    }
}

Классно, не правда ли? Нету надоедливых if-else, нету строк, которые нужно помнить - просто вызываем нужный нам тип драйвера и все. Лично я предпочитаю этот метод.

P.S. Конструкция if-else делает код сложным для чтения, поэтому нужно стараться избегать их использования.

Частные консультации и обучение