如何摆脱重复的if语句？

Question

注视虽然该项目的一些代码，我的工作，我已经遇到这确实 以下一个相当丰厚的方法：

public string DataField(int id, string fieldName) 
{ 
    var data = _dataRepository.Find(id); 
    if (data != null) 
    { 
     if (data.A == null) 
     { 
     data.A = fieldName; 
     _dataRepository.InsertOrUpdate(data); 
     return "A"; 
     } 

     if (data.B == null) 
     { 
     data.B = fieldName; 
     _dataRepository.InsertOrUpdate(data); 
     return "B"; 
     } 

    // keep going data.C through data.Z doing the exact same code 
    } 
} 

显然具有26个if语句只是为了确定一个属性null，然后更新该属性并执行数据库调用 可能在实现中非常天真。什么是做这个工作单位的更好方式？

Answer 1

谢天谢地C# is able to inspect and assign class members dynamically，所以一个选项将创建一个 地图 列表并迭代。

public string DataField(int id, string fieldName) 
    { 
     var data = _dataRepository.Find(id); 

     List<string> props = new List<string>(); 
     props.Add("A"); 
     props.Add("B"); 
     props.Add("C"); 

     if (data != null) 
     { 
      Type t = typeof(data).GetType(); 
      foreach (String entry in props) { 
       PropertyInfo pi = t.GetProperty(entry); 
       if (pi.GetValue(data) == null) { 
        pi.SetValue(data, fieldName); 
        _dataRepository.InsertOrUpdate(data); 
        return entry; 
       } 
      } 
     } 
    } 

Answer 2

您可以循环遍历从'A'到'Z'的所有字符。这很难，因为你想访问你的'数据'对象的属性与相应的名称，但应该（据我所知）有可能通过C＃反射功能。

当你摆脱了连续的if语句的这仍然不会让你的代码不错：P

Answer 3

注： - 如果数据是对象而不是结构，则不要使用。

我想到的是，如果A-Z都是同一类型，那么理论上你可以直接访问内存来检查非空值。

start = &data; 

for (i = 0; i < 26; i++){ 
    if ((typeof_elem) *(start + sizeof(elem)*i) != null){ 
     *(start + sizeof(elem)*i) = fieldName; 
     return (char) (65 + i); 
    } 
} 

未经测试，但给一个想法;）

Answer 4

由于你总是先设定最低字母表场和回报，你可以使用你的类，它跟踪的第一个可用字段中的附加字段。例如，这可以是一个整数lowest_alphabet_unset，只要你一组数据你会更新{X}：

初始化：

lowest_alphabet_unset = 0; 

数据场：

lowest_alphabet_unset ++; 
switch (lowest_alphabet_unset) { 

    case 1: 
     /* A is free */ 
     /* do something */ 
     return 'A'; 
    [...] 

    case 7: 
     /* A through F taken */ 
     data.G = fieldName; 
     _dataRepository.InsertOrUpdate(data); 
     return 'G'; 
    [...] 
} 

Answer 5

var data = _dataRepository.Find(id); 

如果可能的话，你应该使用另一个没有这26个属性的数据类型。新的DataType应该有1个属性，Find方法应该返回该新DataType的一个实例;那么，如果以更自然的方式，你可以摆脱26。

要返回 “A”， “B” ...... “Z”，你可以这样做：

return (char)65; //In this example this si an "A" 

，并与来自data.Value 65和90（A之间的一些转换为数字工作到Z）。

Answer 6

有您的问题使用反射看中LINQ的解决方案：

，但因为它是之前所说：你的数据结构是不是通过

public String DataField(int id, string fieldName) 
{ 
    var data = new { Z = "test", B="asd"}; 
    Type p = data.GetType(); 

    var value = (from System.Reflection.PropertyInfo fi 
        in p.GetProperties().OrderBy((fi) => fi.Name) 
        where fi.Name.Length == 1 && fi.GetValue(data, null) != null 
        select fi.Name).FirstOrDefault(); 
    return value; 
} 

非常深思熟虑TA taaaaaaaaa 像你得到的财产，但更新尚未完成。