Как упростить этот код C#

Question

Есть ли лучший способ написать это в vs2010 C#?

public bool IsAccept() 
    { 
     //check the status is accept 
     if (Status == null) 
      return false; 
     return Status.ToLower() == "accept"; 
    } 

    public bool IsRefer() 
    { 
     //check the status is refer 
     if (Status == null) 
      return false; 
     return Status.ToLower() == "refer"; 
    } 


    public bool IsAnyReviewState() 
    { 
     if (IsAccept() || IsRefer()) 
      return true; 
     return false; 
    } 

Возможно, упрощенный способ в C# 4, который я все еще изучаю.

Answer 1

public bool IsAccept() 
    { 
     return (Status == null)?false:(Status.ToLower() == "accept"); 
    } 

И так далее ...

Answer 2

Как насчет этого?

public bool IsAnyReviewState() 
{ 
    return new [] {"accept", "refer"}.Contains((Status??string.empty).ToString().ToLower()) 
} 

Answer 3

вы можете использовать оценку короткого замыкания, чтобы сделать код более точным. КСТАТИ. нет ничего особенного в C# 4.

public bool IsAccept() 
{ 
    //check the status is accept 
    return Status != null && Status.ToLower() == "accept"; 
} 

public bool IsRefer() 
{ 
    //check the status is refer 
    return Status != null && Status.ToLower() == "refer"; 
} 


public bool IsAnyReviewState() 
{ 
    return IsAccept() || IsRefer(); 
} 

Answer 4

1. Для простых аксессоров, как эти, синтаксис недвижимости лучше, чем методы.
2. Используйте String.Equals или String.IndexOf с StringComparison.OrdinalIgnoreCase вместо того, чтобы делать String.ToLower() ==. Также, если вы абсолютно должны нормализовать строки, используйте String.ToUpperInvariant, поскольку он сохраняет определенную информацию, которая теряется при выполнении ToLowerInvariant, это влияет на нелатинские языки, такие как турецкий, который имеет особое поведение своего I-подобного символа.
3. Рассмотрите возможность использования String.IsNullOrEmpty
4. Помните, что логические операторы возвращают булевы значения сами, так что вы можете упростить IsAnyReviewState массивно.

Вот как я бы написать свой код:

public bool IsAccept { 
    get { 
     return String.IsNullOrEmpty(this.Status) ? false : this.Status.Equals("accept", StringComparison.OrdinalIgnoreCase); 
    } 
} 

public bool IsRefer { 
    get { 
     return String.IsNullOrEmpty(this.Status) ? false : this.Status.Equals("refer", StringComparison.OrdinalIgnoreCase); 
    } 
} 

public bool IsAnyReviewState { 
    get { 
     return this.IsAccept || this.IsRefer; 
    } 
} 

Answer 5

я хотел бы использовать ternary operators как так:

public bool IsAccept() 
    { 
     //check the status is accept 
     return Status == null ? false : Status.ToLower() == "accept"; 
    } 

Answer 6

Лично я предпочитаю использовать string.Equals для сравнения строк. В случае IsAccept(), я хотел бы написать:

public bool IsAccept() 
{ 
    return string.Equals(Status, "accept", StringComparison.InvariantCultureIgnoreCase); 
} 

Это освобождает вас от написания оборонительных null -Проверяет и не требует .ToLower().

Меньше кода: = счастливый кодировщик

редактировать: имеет ли Status свойство нужно быть string? Вы могли бы заменить его для перечисления, может быть?

Answer 7

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

Если вы пытаетесь сделать его более читаемым, единственное, что приходит на ум, - это изменить операторы return.

Например, вместо

return Status.ToLower() == "accept"; 

считают

if(Status.ToLower() == "accept") 
     return true; 
    else 
     return false;