Question

I have a situation:

I nee to do something with a class.

What should be more efficiente, modify the method this way witf IFs or created methos for each action?

public value Value(int command)
        {
            if (command == 1)
            {
                DoSomething1();
            }

            if (command == 2)
            {
                DoSomething2();
            }
            else
            {
                return empty();
            }


        }

There are going to be like 50 o more of this commands.
Whats isbetter in terms of performance on execution and size of the exectuable?

Answer 1

At a high-level, it looks like you’re trying to implement some kind of dynamic-dispatch system? Or are you just wanting to perform a specified operation without any polymorphism? It’s hard to tell.

Anyway, based on the example you’ve given, switch block would be the most performant, as the JIT compiler converts it into an efficient hashtable lookup instead of a series of comparisons, so just do this:

enum Command { // observe how I use an enum instead "magic" integers
    DoSomethingX = 1,
    DoSomethingY = 2
}

public Value GetValue(Command command) {
    switch(command) {
        case Command.DoSomethingX: return DoSomethingX();
        case Command.DoSomethingY: return DoSomethingY();
        default: return GetEmpty();
    }
}

I also note that the switch block also means you get more compact code.