Как вызвать метод, в засисимости от значения переменной?

Question

[BitNeBolt]

Есть объект, в конструкторе которого указывается параметр. У объекта необходимо вызывать метод do(), который в зависимости от параметра должен выполнять другой метод. Как это реализовать без switch именно в c#?

Answer 1

[ayazer]

судя по всему вы пытаетесь получить такое поведение. Но да, выглядит как не лучшее решение. Только я не понимаю откуда такая категоричность про использование switch

public class Class1
    {
        private readonly Action _action;

        public Class1(string actionName)
        {
            switch (actionName)
            {
                case "1":
                    _action = Action1;
                    break;
                case "2":
                    _action = Action2;
                    break;
                default:
                    throw new Exception($"Unexpected action name {actionName}");
            }
        }

        public void Act()
        {
            _action.Invoke();
        }

        private void Action1()
        {
            // ...
        }

        private void Action2()
        {
            // ...
        }
    }

Answer 2

[Владимир Коротенко]

if(arg == 'one') this.doOne();

Comments

[BitNeBolt]

Спасибо, подходит. Но если проверок много, как и объектов, то будет ли данный способ самым эффективным? На джаве я реализовывал интерфейс в зависимости от параметра, и в методе do() указывал нужный для вызова метод только при инициализации объекта. Таким образом я делал проверку параметра только 1 раз, в конструкторе. В c# у меня такое не получилось. Возможно ли это в c#?

[Qualiant]

NeMalenKihren, скорее всего нельзя. Но такая реализация - антипаттерн и нарушение KISS. Вызываемые методы должны быть строго ведомыми.

[Владимир Коротенко]

NeMalenKihren, Не совсем понял что хотите, покажите код на яве. А лучше расскажите зачем вы это хотите сделать

[freeExec]

NeMalenKihren,

void Do1() {};
void Do2() {};
Action myDo;

if (a)
    myDo = Do1;
else
    myDo = Do2;

myDo();

[BitNeBolt]

Владимир Коротенко, Записать можно короче и лучше, но вот что делаю я:

public class Object
{
    public interface IObject
    {
        void doSmth();
    }

    public static IObject build(String param)
    {
        IObject obj;

        switch (param)
        {
            case "s":
                obj = new IObject()
                {
                    @Override
                    public void doSmth()
                    {
                        //doing
                    }
                };

                break;

            case "f":
                obj = new IObject()
                {
                    @Override
                    public void doSmth()
                    {
                        //doing
                    }
                };

                break;

            default:
                obj = new IObject()
                {
                    @Override
                    public void doSmth()
                    {
                        //doing
                    }
                };

                break;
        }

        return obj;
    }
}

[Qualiant]

freeExec, а чем это отличается от
if(arg == 'one') this.doOne();
?

[BitNeBolt]

freeExec, спасибо, то что нужно.

[freeExec]

Qualiant, вопрос был в том чтобы проверка была один раз, а не при каждом вызове. Да, результат проверки можно как-то кешировать. Ну а так в общем случае ничем.

[Qualiant]

freeExec, могу ошибаться, но вроде бы это называется "преждевременная оптимизация".

[Владимир Коротенко]

NeMalenKihren, Насчет множества проверок. Реально множества https://docs.microsoft.com/en-us/archive/blogs/rue...

По сути если вы правильно написали у вас просто сравниваются адреса, а так как одинаковые строки это один объект, то это ультра быстрая операция.

https://habr.com/ru/post/172627/