0

Возьмем за основу, что преобразовать из возвращаемого типа ф-ции в другие можно только зная тип класса(в которой находится эта ф-ция), ибо возвращаемый тип ф-ции напрямую зависит от вычислений, производимых в процессе парса.
В голову сразу же приходит следующее решение: создать класс-парсер, в котором много ф-ций parse, которые преобразовывают возвращаемый тип ф-ции в другие типы. Но тогда обозначить, что это парсер, можно только через интерфейс без методов. И тогда вся абстракция рушится - мы начинаем зависеть от конкретной реализации. Как поступить? Может, я изначально неправильно систему построил, и эта проблема неискоренима?

Если нужно, вот текст программы:

public interface Crashable<OutputType, ToBreakDeterminator> {
    boolean hasCrushed();
    default boolean canBreakDown(CrasherUI<OutputType, ToBreakDeterminator> crasher) { // проверять должен CrasherUI.canCrash()!
        return crasher.canCrash(this);
    }
    OutputType breakDown(CrasherUI<OutputType, ToBreakDeterminator> crasherUI) throws IllegalStateException;
    ToBreakDeterminator getToBreakDeterminator();   
}

public class Thing implements Crashable<String, Integer>{
    private Integer rigidity;
    private String name;

    private CrasherUI crushedBy;

    public final static Integer DEFAULT_RIGIDITY = 50;
    public final static String DEFAULT_NAME = "unnamed";

    {
        crushedBy = null;
    }

    //...

    @Override
    public String breakDown(CrasherUI<String, Integer> crasher) throws IllegalStateException {
        if(hasCrushed()) {
            throw new IllegalStateException("It is already crushed by " + crushedBy);
        }
        crushedBy = crasher;
        return "u broke " + name + " with " + rigidity + " by " + crasher + ", dude"; 
        // это возврат ф-ции. По-идее должен возвращать уже OutputType(см абстрактный класс)
    }

    //...
}

А вот сам абстрактный класс:

public abstract class AbstractApartaments<OutputType, InputType, ToBreakDeterminator> implements ApartamentUI{
    private Stack<? extends Crashable<InputType, ToBreakDeterminator>> things;

    private CrasherUI<InputType, ToBreakDeterminator> crasher;

    private Function<InputType, OutputType> inputToOutput; // один тип(Crashable) только обрабатывать можем. 
    private ConcurrentLinkedQueue<OutputType> toUserOutputs;
    private ToUserOutputUI<OutputType> out;

    {
        things = new Stack();
        toUserOutputs = new ConcurrentLinkedQueue();
    }

    public AbstractApartaments(Collection<? extends Crashable<InputType, ToBreakDeterminator>> toCrush, CrasherUI<InputType, ToBreakDeterminator> crasher, Function<InputType, OutputType> inputToOutput, ToUserOutputUI<OutputType> out){
        things.addAll(toCrush); // Ошибка!
        this.crasher = crasher;
        this.inputToOutput = inputToOutput;
        this.out = out;
    }


    @Override
    public boolean hasThingsToCrash() {
        return things.isEmpty();
    }

    @Override
    public boolean crashNext() {
        if(!things.isEmpty()) {
            Crashable<InputType, ToBreakDeterminator> toCrash = things.pop();
            if(toCrash.canBreakDown(crasher)) {
                toUserOutputs.add(inputToOutput.apply(toCrash.breakDown(crasher)));
                return true;
            } else {
                return false;
            }
        } else {
            throw new NoSuchElementException();
        }
    }

    @Override
    public void pullToUserOutputs() {
        while(!toUserOutputs.isEmpty()) {
            out.toOutput(toUserOutputs.poll());
        }
    }

}
15
  • Дженерики не подходят? 26 янв 2020 в 6:15
  • @SuvitrufsaysReinstateMonica Хмм, жестко ограничить список типов, в которые возможен парс? А чтобы не иметь жесткое ограничение достаточно сделать скелет из интерфейсов... Но тогда возникает проблема - кое-где в абстрактном классе я использую эту ф-цию преобразования. И мне нужен конкретный тип на выходе(результат одной из ф-ций parse), который указан в дженерике абстрактного класса. Вот тогда-то и можно опереться на наш скелет, в котором нижние элементы представляют собой Parser<T, V>{V parse(T t);}. То есть можно просто подставлять нужный парсер на место интерфейса.
    – Miron
    26 янв 2020 в 6:25
  • @SuvitrufsaysReinstateMonica А можно как-то регулировать кол-во одинаковых наследуемых интерфейсов? Как-то не чисто выглядит Paser2<From, A, B>, Parser3<From, A, B, C>, Parser4<From, A, B, C, D> и тд.
    – Miron
    26 янв 2020 в 6:29
  • @Miron добавьте, пожалуйста пример кода в вопрос, по тексту очень тяжело воспринимается.
    – Z.John
    26 янв 2020 в 6:40
  • 1
    @StrangerintheQ Добавил код.
    – Miron
    26 янв 2020 в 6:55

1 ответ 1

1

Пометьте что ваша очередь может принимать любой тип наследованный от Crashable

public abstract class AbstractApartaments<InputType, ToBreakDeterminator> implements ApartamentUI{
private Stack<Crashable> things;
private CrasherUI crasher;
private ConcurrentLinkedQueue<Crashable> toUserOutputs;
private ToUserOutputUI<OutputType> out;

{
    things = new Stack<>();
    toUserOutputs = new ConcurrentLinkedQueue<>();
}

public AbstractApartaments( Collection<? extends Crashable<InputType, ToBreakDeterminator>> toCrush,
                            CrasherUI<InputType, ToBreakDeterminator> crasher,
                            ToUserOutputUI<OutputType> out){

    this.things.addAll(toCrush);
    this.crasher = crasher;
    this.out = out;
}


@Override
public boolean hasThingsToCrash() {
    return things.isEmpty();
}

@Override
public boolean crashNext() {
    if(!things.isEmpty()) {
        Crashable toCrash = things.pop();
        if(toCrash.canBreakDown(crasher)) {
            toUserOutputs.add(toCrash.breakDown(crasher));
            return true;
        } else {
            return false;
        }
    } else {
        throw new NoSuchElementException();
    }
}

@Override
public void pullToUserOutputs() {
    while(!toUserOutputs.isEmpty()) {
        out.toOutput(toUserOutputs.poll());
    }
}

}
10
  • @Miron если я вас правильно понял, примерно так, но не уверен что я вас правильно понял. Дженерики и правда мощная вещь, но они далеко не всегда нужны для создании гибкой архитектуры. В вашем примере абсолютно точно не правильная архитектура. Если есть желание, выложите проект на github или bitbucket - могу посмотреть, подсказать по архитектуре.
    – Z.John
    26 янв 2020 в 8:01
  • Не дает передать ссылку объекта "Collection<? extends Crashable<InputType, ToBreakDeterminator>>" в "Stack<? extends Crashable<InputType, ToBreakDeterminator>>". Выдает следующую ошибку: "The method addAll(Collection<? extends capture#2-of ? extends Crashable<InputType,ToBreakDeterminator>>) in the type Vector<capture#2-of ? extends Crashable<InputType,ToBreakDeterminator>> is not applicable for the arguments (Collection<capture#3-of ? extends Crashable<InputType,ToBreakDeterminator>>)"
    – Miron
    26 янв 2020 в 8:04
  • @Miron да, надо передать ему дженерики которые вы используете в интерфейсе. И вы не инициализируете crasher.
    – Z.John
    26 янв 2020 в 8:15
  • И там, и там же указано "? extends Crashable<InputType, ToBreakDeterminator>". Чем эти записи не идентичны? crasher указывается в конструкторе, я это опустил.
    – Miron
    26 янв 2020 в 8:19
  • надо смотреть, так ничего не подскажу
    – Z.John
    26 янв 2020 в 8:21

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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