4 c# refactoring switch-statement
我有以下代码,我目前正在使用....基本上,此方法为每个任务分配正确的布尔标志(TRUE/FALSE).随着越来越多的任务需要添加..我可以看到switch语句必须增长以满足每个任务.
必须有一种更简单的方法......保持方法小.
代码:(忘记命名约定,它已被更改为发布)
public ClassStructure.User AssignTaskStatusToUser(ClassStructure.User,
List<ClassStructure.Tasks> 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;
}
Run Code Online (Sandbox Code Playgroud)
ClassStructure.Tasks_CallIDs_Strings =字符串表示任务
data.Task_Flag =布尔值
User.TASKX =布尔值
欢迎任何反馈.我相信有一个简单的解决方案.
对于像这样的很多值,我会使用这样的地图:
Dictionary<ClassStructure.Tasks_CallIDs_Strings, Task_Flag>
Run Code Online (Sandbox Code Playgroud)
并通过映射CallIDs字符串来检索值.
编辑:
正如大家现在所看到的,重构这个例子的真正问题在于重构User.TASKX.使它成为一个列表就足够了 - 因为它可以被相同的字符串ClassStructure索引.Tasks_CallIDs_Strings