当前位置:  开发笔记 > 编程语言 > 正文

重构我的C#代码 - 切换语句

如何解决《重构我的C#代码-切换语句》经验,为你挑选了1个好方法。

我有以下代码,我目前正在使用....基本上,此方法为每个任务分配正确的布尔标志(TRUE/FALSE).随着越来越多的任务需要添加..我可以看到switch语句必须增长以满足每个任务.

必须有一种更简单的方法......保持方法小.

代码:(忘记命名约定,它已被更改为发布)

public ClassStructure.User AssignTaskStatusToUser(ClassStructure.User,
                                                  List TaskStatus)
{
    foreach (ClassStructure.Tasks data in TaskStatus)
    {
        string Task_CallID = data.Task_Call_ID;

        switch (Task_CallID)
        {
            case ClassStructure.Tasks_CallIDs_Strings.TASK1:
                User.TASK1 = data.Task_Flag;
                break;

            case ClassStructure.Tasks_CallIDs_Strings.TASK2:
                User.TASK2 = data.Task_Flag;
                break;

            case ClassStructure.Tasks_CallIDs_Strings.TASK3:
                User.TASK3 = data.Task_Flag;
                break;
        }
    }

    return User;
}

ClassStructure.Tasks_CallIDs_Strings =字符串表示任务

data.Task_Flag =布尔值

User.TASKX =布尔值

欢迎任何反馈.我相信有一个简单的解决方案.



1> Elroy..:

对于像这样的很多值,我会使用这样的地图:

Dictionary

并通过映射CallIDs字符串来检索值.

编辑:

正如大家现在所看到的,重构这个例子的真正问题在于重构User.TASKX.使它成为一个列表就足够了 - 因为它可以被相同的字符串ClassStructure索引.Tasks_CallIDs_Strings

推荐阅读
帆侮听我悄悄说星星
这个屌丝很懒,什么也没留下!
DevBox开发工具箱 | 专业的在线开发工具网站    京公网安备 11010802040832号  |  京ICP备19059560号-6
Copyright © 1998 - 2020 DevBox.CN. All Rights Reserved devBox.cn 开发工具箱 版权所有