Как оптимизировать мой, если еще код C#

Question

У меня есть этот код ниже:

TryUpdateModel(model); 
      if (ModelState.IsValid && model.FullNameIsChecked == true && model.LookingFor == null) 
      { 
       model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
       model.ContactList.Sort((x, y) => string.Compare(x.FullName, y.FullName)); 
      } 
      else if (ModelState.IsValid && model.FullNameIsChecked == false && model.LookingFor == null) 
      { 
       model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
       model.ContactList.Sort((x, y) => string.Compare(x.Email, y.Email)); 
      } 
      else if (ModelState.IsValid && model.LookingFor != null && model.FullNameIsChecked == true) 
      { 
       model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
       model.ContactList = contactRepository.GetAll(filter: x => x.FullName.Contains(model.LookingFor)); 
       model.ContactList.OrderBy(f => f.FullName); 
      } 
      else if (ModelState.IsValid && model.LookingFor != null && model.FullNameIsChecked == false) 
      { 
       model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
       model.ContactList = contactRepository.GetAll(filter: x => x.Email.Contains(model.LookingFor)); 
       model.ContactList.OrderBy(e => e.Email); 
      } 
      else 
      { 
       model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
      } 

Я использую .NET MVC 5 с двигателем Razor, и с кодом выше я делать операции сортировки. Могу ли я оптимизировать свой Если еще часть кода? Единственное, что у меня на уме, это использовать операционный оператор , но он будет выглядеть почти так же. Заранее спасибо

Answer 1

• я не могу видеть на ModelState

• Нет необходимости любое использование в вашем текущем состоянии кода для сравнения BOOL, если условие, т.е.

  if(true) 
  

  такой же, как

  if(bool variable=true) 
  

• использовать String.IsNullOrEmpty проверить null

  TryUpdateModel(model); 
  
      model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
  
       if( String.IsNullOrEmpty(model.LookingFor)) 
       { 
        if(model.FullNameIsChecked) 
        { 
         model.ContactList.Sort((x, y) => string.Compare(x.FullName, y.FullName)); 
        } 
        else 
        { 
         model.ContactList.Sort((x, y) => string.Compare(x.Email, y.Email)); 
        } 
       } 
       else 
       { 
        if(model.FullNameIsChecked) 
        { 
         model.ContactList = contactRepository.GetAll(filter: x => x.FullName.Contains(model.LookingFor)); 
         model.ContactList.OrderBy(f => f.FullName); 
        } 
        else 
        { 
         model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
         model.ContactList = contactRepository.GetAll(filter: x => x.Email.Contains(model.LookingFor)); 
        } 
       } 
  

Answer 2

Все ваши случаи вписываться в этом:

//Place this here since it's non conditional 
model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 


if(ModelState.IsValid) 
{ 
    if(model.LookingFor == null) 
    { 
     if(model.FullNameIsChecked){ 
      //... 
     }else 
     { 
      //... 
     } 
    }else 
    { 
     if(model.FullNameIsChecked){ 
      //... 
     }else 
     { 
      //... 
     } 
    } 
} 
else 
{ 
    //... Actually you don't need this case... 
} 

Answer 3

TryUpdateModel(model); 
if(ModelState.IsValid) 
{ 
    model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
    if(model.LookingFor == null) 
    { 
     if(model.FullNameIsChecked) 
     { 
      model.ContactList.Sort((x, y) => string.Compare(x.FullName, y.FullName)); 
     } 
     else 
     { 
      model.ContactList.Sort((x, y) => string.Compare(x.Email, y.Email)); 
     } 
    } 
    else 
    { 
     if(model.FullNameIsChecked) 
     { 
      model.ContactList = contactRepository.GetAll(filter: x => x.FullName.Contains(model.LookingFor)); 
      model.ContactList.OrderBy(f => f.FullName); 
     } 
     else 
     { 
      model.ContactList = contactRepository.GetAll(filter: x => x.Email.Contains(model.LookingFor)); 
      model.ContactList.OrderBy(e => e.Email); 
     } 
    } 
} 

Answer 4

ERM, таким образом, каждое условие проверяется только один раз.

if (ModelState.IsValid) 
{ 
    if (string.IsEmptyOrWhitespace(model.LookingFor)) 
    { 
     model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
    } 
    else 
    { 
     model.ContactList = contactRepository.GetAll(
       filter: x => x.FullName.Contains(model.LookingFor)); 
    } 

    if(model.FullNameIsChecked) 
    { 
     model.ContactList.OrderBy(f => f.FullName); 
    } 
    else 
    { 
     model.ContactList.OrderBy(e => e.Email); 
    } 
} 
else 
{ 
    model.ContactList = contactRepository.GetAll(filter: x => x.UserId == user.Id); 
} 

Answer 5

Когда model.LookingFor является ненулевым, вы звоните contactRepository.GetAll() дважды. Первый звонок, до contactRepository.GetAll(filter: x => x.UserId == user.Id), лишний, правда?

Иногда вы звоните в .Sort(), а иногда и вы звоните .OrderBy(). Почему непоследовательность?

Вы использовали x, f и e как фиктивные переменные. Я рекомендую использовать некоторую фиктивную переменную последовательно - возможно, c для «контакта».

В любом случае, этот код хочет сделать какой-то GetAll(), возможно, после чего будет сортироваться. Итак, давайте напишем один звонок до GetAll() и один звонок до OrderBy().

TryUpdateModel(model); 
model.ContactList = contactRepository.GetAll(filter: 
    !ModelState.IsValid ?  (c => c.UserId == user.Id) : 
    model.LookingFor == null ? (c => c.UserId == user.Id) : 
    model.FullNameIsChecked ? (c => c.FullName.Contains(model.LookingFor)) : 
           (c => c.Email.Contains(model.LookingFor)) 
); 
if (ModelState.IsValid) 
{ 
    model.ContactList.OrderBy(model.FullNameIsChecked ? 
     (c => c.FullName) : (c => c.Email) 
    ); 
}