开关语句的替代方法

Question

null

代码如下所示：

    public class SomeType
    {
        // Some properties.
    }

    public enum SomeTrigger
    {
        Loaded,
        Initial,
        Timer,
        Final
    }


    public class SomeBaseObject
    {
        // 
        // I am not allowed to change this class. It is not mine.
        //

        protected Dictionary<string, SomeType> Input;
        protected Dictionary<string, SomeType> Output;


        // Timer is evaluated every 2 sec.
        public virtual void Execute(SomeTrigger trigger, string value)
        {
            switch (trigger)
            {
                case SomeTrigger.Loaded:
                    break;
                case SomeTrigger.Initial:
                    break;
                case SomeTrigger.Timer:
                    break;
                case SomeTrigger.Final:
                    break;
                default:
                    break;
            }
        }

        // The rest of code...
    }

和我想提高的班级：

public class SomeSpecificObject : SomeBaseObject
    {
        private bool isInitializationCompleted;
        private string connection;

        public override void Execute(SomeTrigger trigger, string value)
        {
            switch (trigger)
            {
                case SomeTrigger.Loaded:
                    this.OnLoaded();
                    break;
                case SomeTrigger.Initial:
                    this.OnInitial();
                    break;
                case SomeTrigger.Timer:
                    this.OnTimer();
                    break;
                case SomeTrigger.Final:
                    this.OnFinal(value);
                    break;
                default:
                    break;
            }
        }

        private void OnLoaded()
        {
            // Read Input and Output collection.
        }

        private void OnInitial()
        {
            // Initialization (connect to the server).
            // Bla bla bla
            this.connection = //Value from the plug-in;
            this.isInitializationCompleted = true;

        }

        private void OnTimer()
        {
            if (isInitializationCompleted)
            {
                // Do something
                // Connection is using here.
                // Calculate values on a Input collection, etc.
            }
        }

        private void OnFinal(string value)
        {
            if (isInitializationCompleted)
            {
                // something with "value"
                // Connection is using here.
                // Clear state.
            }
            else
            {
                // Connection is using here.
                // Cancel inistialization
            }
        }
    }

我该怎么办？每个字段都在使用，但每个触发器都在使用。另外，这种情况有点特殊（OnFinalMethod需要参数）。基于上面的文章，我尝试重构这段代码，但没有成功。

My attempts to apply some tips: 

    public interface ITrigger
    {
        void Execute();
    }

    public class LoadedTrigger : ITrigger
    {
        public void Execute()
        {
            throw new NotImplementedException();
        }
    }

    //
    // Exactly the same class for the rest cases.
    //
    public class TriggerHandler
    {
        private Dictionary<SomeTrigger, ITrigger> triggerDictionary;

        public TriggerHandler()
        {

            triggerDictionary.Add(SomeTrigger.Loaded, new InitialTrigger());
    // and for the rest

        }

        public void HandleTrigger(SomeTrigger trigger)
        {
            triggerDictionary[trigger].Execute();
        }
    }

你知道吗？：）

Answer 1

一开始，switch语句总体上没有什么问题。仅仅在代码中使用它就会产生一些隐含的结构限制。如果这些限制中的一个与代码的发展方式相矛盾，换句话说，如果需求以开关逻辑变得过时的方式发生变化，那么继续在代码中保留它将导致技术债务。

从上面的代码中，并不清楚对它的要求是什么。例如，为什么首先需要有Execute方法？可以直接用just loadd()、Initial()和其他方法替换它吗？这显然会消除开关。

另一种方法是使用事件对象而不是枚举，并使用方法重载：Execute（LoadedTrigger...）,...,Execute（FinalTrigger...,...）。

如果您想在运行时添加处理程序，使用dictionary的方法是很好的（是吗？），但缺少的部分是检查触发器类型和调用不同的execute方法：

var t = triggerDictionary[trigger];
if (t is SimpleTrigger st) st.Execute();
else if (t is AdvancedTrigger at) at.Execute(value);

或者，向触发器的execute方法添加值。但如果没有这样的要求，又何必费心呢？

例如。TimerTrigger对象需要知道初始化是成功的。

您可能需要一个状态机。开关是为简单状态机编写转换代码的一种常用方法。