Как нормально пробросить данные в дочерние классы, используя ООП?

Question

[ColdSpirit]

Доброе время суток! Пытаюсь сделать ИИ по совести, чтобы и ООП был, и универсальный, но не получается. Тупо C# не позволяет мне сделать это даже с генериками. Цель - пробросить в реализацию интерфейса IBotBehaviour кастомную информацию, нужную лишь для данного поведения. Хотелось бы пробросить ее через конструктор или метод. Но так, чтобы ее было обязательно пробрасывать.

Вот пример интерфейсов и реализаций:
IBotBehaviour - интерфейс поведения, который должен обязывать передавать ему информацию.
IBot - использует IBotBehaviour.
IdleBehaviour - ничего не делает, следовательно инфа ему никакая не нужна.
FollowBehaviour - требует 2 аргумента типа Positionу
BotTest - использует попеременно разные поведения и пробрасывает в них совершенно разные параметры.

Пример кода

public interface IBotBehaviour
{
    void Process();
    void Setup(); // ?
}

public interface IBot
{
    void SetBehaviour(IBotBehaviour behaviour);
}

public class IdleBehaviour : IBotBehaviour
{
    public void Process()
    {
        // wait 1s
    }

    public void Setup()
    {
    }
}

public class FollowBehaviour : IBotBehaviour
{
    Position myPosition;
    Position targetPosition;

    public void Process()
    {
        // move my pos to target pos
    }

    public void Setup(Position myPosition, Position targetPosition)
    {
        this.myPosition = myPosition;
        this.targetPosition = targetPosition;
    }
}

public class BotTest : IBot
{
    protected IBotBehaviour currentBehaviour;

    public void SetBehaviour(IBotBehaviour behaviour)
    {
        currentBehaviour = behaviour;
    }

    // call currentBehaviour.Process() in game cycle
}

Comments

[Андрей]

Суть проблемы не очень понятна.

Цель - пробросить в реализацию интерфейса IBotBehaviour кастомную информацию, нужную лишь для данного поведения.

Если generic-методов в интерфейсах вам недостаточно, используйте абстрактные классы.
Так же посмотрите паттерн декоратор.

[BoShurik]

А когда вызывается currentBehaviour.Call()?

[ColdSpirit]

BoShurik, код работает в юнити, вызывается в Update(). Я просто вынес самое важное, чтобы отвязать от юнити.

[ColdSpirit]

Андрей, я пытался использовать абстрактные классы, чтобы пробросить инфу, но безуспешно. Насчет декораторов не думал, спасибо, попробую.

Answer 1

[Griboks]

Интерфейс позволяет задать сигнатуру методу. Следовательно, аргументы можно прописать в самом интерфейсе.

Comments

[ColdSpirit]

Вопрос не о том, можно ли это сделать, а как это сделать лучше всего.

[Griboks]

ColdSpirit, лучше всего в методе, которому нужны аргументы, указать эти аргументы. Например:

public interface IBotBehaviour
{
    void Process(int pointer); //поведение, которое обязывает передавать ему информацию
    void Setup();
}

[ColdSpirit]

Griboks, в этом случае теряется смысл в интерфейсах, так как данные в разных поведениях разные.

[Griboks]

ColdSpirit, если данные разные, то и поведения разные (если мы говорим о методах). А если поведения разные, то их нельзя объединить в интерфейсе, т. к. эта абстракция объединяет одинаковое поведение.

Либо все животные лают, либо только собаки. Нельзя сказать, что кошка лает по-кошачьи.

[Станислав Силин]

Griboks, интерфес не объединяет поведение. Он определяет контракт который должен реализовать класс. А как он его реализовал это уже совсем другая история.

[Griboks]

Станислав Силин, если вашими терминами, то нельзя записать разные контракты под одним именем.

Answer 2

[Станислав Силин]

Простой, но немного измененый вариант

public interface IBotBehaviour
{
    void Process();
}

public interface IBot
{
    void SetBehaviour(IBotBehaviour behaviour);
}

public class IdleBehaviour : IBotBehaviour
{
    public void Process()
    {
        // wait 1s
    }
}

public class FollowBehaviour : IBotBehaviour
{

    public FollowBehaviour(Position myPosition, Position targetPosition)
    {

    }


    public void Process()
    {
        // move my pos to target pos
    }
}


// new BotTest().SetBehaviour(new FollowBehaviour(myPosition, targetPosition))
public class BotTest : IBot
{
    protected IBotBehaviour currentBehaviour;

    public void SetBehaviour(IBotBehaviour behaviour)
    {
        this.currentBehaviour = behaviour;
    }

    // call currentBehaviour.Process() in game cycle
}

Пример использования

new BotTest().SetBehaviour(new FollowBehaviour(myPosition, targetPosition))

Более сложный, но храняем ваши интерфейсы

public interface IBotState
{

}

public interface IBotBehaviour
{
    void Process();
    void Setup(IBotState state);
}

public interface IBotBehaviour<TState>
    where TState : IBotState
{
    void Setup(TState state);
}

public interface IBot
{
    void SetBehaviour(IBotBehaviour behaviour);
}

public class IdleBehaviour : IBotBehaviour
{
    public void Process()
    {
        // wait 1s
    }

    public void Setup(IBotState state)
    {

    }
}

public class FollowState : IBotState
{
    public Position myPosition;
    public Position targetPosition;
}

public class FollowBehaviour : IBotBehaviour<FollowState>
{
    Position myPosition;
    Position targetPosition;

    public void Process()
    {
        // move my pos to target pos
    }

    public void Setup(IBotState state)
    {
        this.Setup((FollowState)state);
    }

    public void Setup(FollowState state)
    {
        this.myPosition = state.myPosition;
        this.targetPosition = state.targetPosition;
    }
}

public class BotTest : IBot
{
    protected IBotBehaviour currentBehaviour;

    public void SetBehaviour(IBotBehaviour behaviour)
    {
        currentBehaviour = behaviour;
    }

    // call currentBehaviour.Process() in game cycle
}

Comments

[ColdSpirit]

Спасибо! 3 вариант близок к решению, но хотелось бы не допустить пробрасывания IBotState в FollowBehaviour, чтобы ненароком никто не кинул туда данные для IdleBehaviour или других поведений. Такое вообще реально сделать?

Интерфейсы не особо важны, главное лишь бы был один абстрактный родитель. Через конструктор пробрасывать данные - не оч подходит, т.к. нужен метод Setup, чтобы обновлять данные, иначе придется плодить новые одинаковые поведения каждый раз.

[Станислав Силин]

ColdSpirit, вся суть интерфеса в том чтобы выделить общее. Вы же пытаетесь выделить общее и при этом учитывать конкретную реализацию, что уже противоречит само себе.

Как вариант можно сделать так

public interface IBotState
{

}

public interface IBotBehaviour
{
    void Process();
    bool TrySetup(IBotState state);
}

public interface IBotBehaviour<TState>
    where TState : IBotState
{
    bool TrySetup(TState state);
}

public interface IBot
{
    void SetBehaviour(IBotBehaviour behaviour);
}

public class IdleBehaviour : IBotBehaviour
{
    public void Process()
    {
        // wait 1s
    }

    public void Setup(IBotState state)
    {

    }
}

public class FollowState : IBotState
{
    public Position myPosition;
    public Position targetPosition;
}

public class FollowBehaviour : IBotBehaviour<FollowState>
{
    private Position myPosition;
    private Position targetPosition;

    public void Process()
    {
        // move my pos to target pos
    }

    public bool TrySetup(IBotState state)
    {
        var followState = state as FollowState;
        if (followState == null)
        {
            return false;
        }

        return this.TrySetup(followState);
    }

    public bool TrySetup(FollowState state)
    {
        this.myPosition = state.myPosition;
        this.targetPosition = state.targetPosition;

        return true;
    }
}

public class BotTest : IBot
{
    protected IBotBehaviour currentBehaviour;

    public void SetBehaviour(IBotBehaviour behaviour)
    {
        this.currentBehaviour = behaviour;
    }

    // call currentBehaviour.Process() in game cycle
}

Это не решает проблему на этапе компиляции, но позволяет ничего не сломать

[Станислав Силин]

ColdSpirit,

чтобы обновлять данные, иначе придется плодить новые одинаковые поведения каждый раз

Не вижу ничего плохого в том чтобы и поплодить, если это решает проблему. Если вы переживаете за GC, то в данном случаи это не страшно, поскольку вы же не будуте создвавать их каждый фрейм, а только тогда когда нужно

Answer 3

[ColdSpirit]

Сделал в итоге по такому принципу: все нужные данные для поведений содержит бот в открытом доступе, а поведения уже забирают их, кому что нужно:

public interface IBotBehaviour
{
    void Setup(BotBase bot);
    void Process();
}