重寫一些嵌套的if語句的選項

Question

我有一個方法使用了一些嵌套的if語句，所以我想知道是否有更好的方法來編寫相同的邏輯。

比如我想刪除避免兩次

_typologyRepository.Update(typology); 
_typologyRepository.Save(); 

你能指出我出了正確的方向？由於

public void Update(Typology typology, string nameOriginalValue) 
    { 
     if (typology.Name == nameOriginalValue) 
     { 
      _typologyRepository.Update(typology); 
      _typologyRepository.Save(); 
     } 
     else 
     { 
      if (IsUniqueName(typology.Name)) 
      { 
       _typologyRepository.Update(typology); 
       _typologyRepository.Save(); 
      } 
      else 
       _validatonDictionary.AddError("Name", errorMessageNameUnique); 
     } 
    } 

Answer 1

if (typology.Name == nameOriginalValue || IsUniqueName(typology.Name)) 
{ 
    _typologyRepository.Update(typology); 
    _typologyRepository.Save(); 
} 
else 
{ 
    _validatonDictionary.AddError("Name", errorMessageNameUnique); 
} 

Answer 2

RedFilter的答案是它應該怎麼寫。另外一個關於你的代碼的說明：

通常，當人們使用if/elses級聯時，它們將所有內容都保存在相同的縮進級別。 RedFilter的答案是更好的，因爲你不需要級聯if/elses，但如果你確實需要它們，大多數人會這樣寫：

public void Update(Typology typology, string nameOriginalValue)    
    {                    
     if (typology.Name == nameOriginalValue)         
     {                   
      _typologyRepository.Update(typology);         
      _typologyRepository.Save();           
     }                   
     else if (IsUniqueName(typology.Name))          
     {                   
      _typologyRepository.Update(typology);         
      _typologyRepository.Save();           
     }                   
     else                  
      _validatonDictionary.AddError("Name", errorMessageNameUnique);   
     }                   
    }