1

Для автоматизации мобильных приложений использую шаблон проектирования Factory. Набор методов для iOS и Android одинаковый, а вот элементы объявляются по-разному. Как избежать дублирование кода?

Здесь упрощенный пример:

public interface FormHelper {
    void close();
}

public class AndroidForm implements FormHelper {
    private Button btnClose = new Button("//a[text()='Close']");
    @Override
    public void close() {
        btnClose.click();
    }
}

public class IOSForm implements FormHelper {
    private Button btnClose = new Button("//div[@id='close']");
    @Override
    public void close() {
        btnClose.click();
    }
}

public class FormFactory {
    public static FormHelper getForm(){
    if (x > 0) {
        return new AndroidForm();
    } else {
        return new IOSForm();
    }
}
1
  • Абстрактный класс?
    – Andrey M
    13 сен 2016 в 17:38

2 ответа 2

1

Ваш код можно вот так зарефакторить:

public abstract class FormHelper {
    private final Button button;

    protected FormHelper(String action) {
        button = new Button(action);
    }

    public void close(){
        button.click();
    }
}

public class AndroidForm extends FormHelper {
    private final static String action="//a[text()='Close']";

    protected AndroidForm() {
        super(action);
    }
}

public class IOSForm extends FormHelper {
    private final static String action = "//div[@id='close']";

    protected IOSForm() {
        super(action);
    }
}

public class FormFactory {
    public static FormHelper getForm(int x) {
        if (x > 0) {
            return new AndroidForm();
        } else {
            return new IOSForm();
        }
    }
}
5
  • К сожалению, для реального проекта это не подходит: там много методов и много переменных типа btnClose. Передавать их все в конструкторе не вариант
    – alex
    13 сен 2016 в 15:24
  • @alex вы бы тогда и привели в вопросе код, приближенный к реальному.
    – andreycha
    13 сен 2016 в 16:13
  • @andreycha да. надо было, не хотел нагромождать большие куски кода
    – alex
    13 сен 2016 в 16:50
  • @alex ну так отредактируйте вопрос.
    – andreycha
    13 сен 2016 в 18:23
  • @andreycha нашел решение, спасибо за помощь
    – alex
    14 сен 2016 в 14:58
0

Нужно добавить абстрактный класс, унаследовать от него классы android и ios, и в них добавить блок нестатической инициализации.

public interface FormHelper {
    void close();
}

public abstract class FormAbstract implements FormHelper{
    protected Button btnClose;
    @Override
    public void close() {
        btnClose.click();
    }
}

public class AndroidForm extends FormAbstract {
    {
        btnClose = new Button("//a[text()='Close']");
    }
}

public class IOSForm extends FormAbstract {
    {
        btnClose = new Button("//div[@id='close']");
    }
}

public class FormFactory {
    public static FormHelper getForm(){
    if (x > 0) {
        return new AndroidForm();
    } else {
        return new IOSForm();
    }
}

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.