Как можно упростить этот повторяющийся код?

Question

Следующий код очень повторяющийся, но содержит очень незначительные различия в доступе к свойствам. Они изменяются между RM/FM/FE и PrimaryRM/PrimaryFM/PrimaryFE в зависимости от того, к какому столбцу относится GroupKeyGetter.

Я не могу найти способ преобразования этого метода в метод, так что мне не нужно иметь этот массивный блок кода, но должен быть способ сделать это лучше ...

allRMsColumn.GroupKeyGetter = delegate(object rowObject) 
{ 
    var users = ((Tenant) rowObject).RMs.Trim().Split(new[] {", "}, StringSplitOptions.RemoveEmptyEntries); 

    return users.FirstOrDefault(user => GlobalSettings.Users.Find(x => x.Name == user && x.Selected) != null) ?? ((Tenant)rowObject).PrimaryRM; 
}; 

allFMsColumn.GroupKeyGetter = delegate (object rowObject) 
{ 
    var users = ((Tenant)rowObject).FMs.Trim().Split(new[] { ", " }, StringSplitOptions.RemoveEmptyEntries); 

    return users.FirstOrDefault(user => GlobalSettings.Users.Find(x => x.Name == user && x.Selected) != null) ?? ((Tenant)rowObject).PrimaryFM; 
}; 

allFEsColumn.GroupKeyGetter = delegate (object rowObject) 
{ 
    var users = ((Tenant)rowObject).FEs.Trim().Split(new[] { ", " }, StringSplitOptions.RemoveEmptyEntries); 

    return users.FirstOrDefault(user => GlobalSettings.Users.Find(x => x.Name == user && x.Selected) != null) ?? ((Tenant) rowObject).PrimaryFE; 
}; 

Любые идеи вообще?

Answer 1

Вы можете сделать вспомогательные методы для извлечения выбранных User, например:

private string GetFirstSelectedUser(string userList) { 
    var users = userList.Trim().Split(new[] { ", " }, StringSplitOptions.RemoveEmptyEntries); 
    return users.FirstOrDefault(user => GlobalSettings.Users.Find(x => x.Name == user && x.Selected) != null); 
} 

В настоящее время все три делегата может быть закодирован так:

allRMsColumn.GroupKeyGetter = delegate(object rowObject) { 
    var renant = (Tenant) rowObject; 
    return GetFirstSelectedUser(tenant.RMs) ?? tenant.PrimaryRM; 
}; 

Если вы можете сделать реструктуризацию в Tenant и может группировать FEs с PrimaryFE, FMs с PrimaryFM и RMs с PrimaryRM, вы можете упростить свой встречный корыто еще дальше:

class GroupWithPrimary { 
    public string Group {get;} 
    public string Primary {get;} 
    public GroupWithPrimary(string group, string primary) { 
     Group = group; 
     Primary = primary; 
    } 
} 
private string GetFirstSelectedUser(GroupWithPrimary gp) { 
    var users = gp.Group.Trim().Split(new[] { ", " }, StringSplitOptions.RemoveEmptyEntries); 
    return users.FirstOrDefault(user => GlobalSettings.Users.Find(x => x.Name == user && x.Selected) != null) ?? gp.Primary 
} 
allRMsColumn.GroupKeyGetter = delegate(object rowObject) { 
    return GetFirstSelectedUser(((Tenant)rowObject).RM); 
}; 
allFMsColumn.GroupKeyGetter = delegate(object rowObject) { 
    return GetFirstSelectedUser(((Tenant)rowObject).FM); 
}; 
allFEsColumn.GroupKeyGetter = delegate(object rowObject) { 
    return GetFirstSelectedUser(((Tenant)rowObject).FE); 
}; 

Answer 2

Похоже, разница между этими тремя вариантами можно экстрагировать в funcs, например, так:

Func<object, object> CreateGroupKeyGetter(Func<Tenant, string> func1, 
      Func<Tenant, string> func2) 
{ 
    return rowObject=> 
      { 
       var users = func1(((Tenant) rowObject)).Trim().Split(new[] ", "}, StringSplitOptions.RemoveEmptyEntries); 

       return users.FirstOrDefault(user => GlobalSettings.Users.Find(x => x.Name == user && x.Selected) != null) ?? func2(((Tenant)rowObject)); 
      } 
} 

allRMsColumn.GroupKeyGetter = CreateGroupKeyGetter(t=>t.Rms, t=>t.PrimaryRm); 
allFMsColumn.GroupKeyGetter = CreateGroupKeyGetter(t=>t.Fms, t=>t.PrimaryFm); 
allFesColumn.GroupKeyGetter = CreateGroupKeyGetter(t=>t.Fes, t=>t.PrimaryFe); 



}