У меня трудное время с правилом StyleCop SA1503 (CurlyBracketsMustNotBeOmitted).Получение правила StyleCop SA1503 CurlyBracketsMustNotBeДобавлено, чтобы быть более гибким
В моем коде я довольно часто имеют шаблон таким образом:
public void SomeFunction(string someArg)
{
if (string.IsNullOrEmpty(someArg)) throw new ArgumentNullException("someArg");
// rest of the function here
}
Смысл этого заключается в экономии места по вертикали при выполнении многочисленных проверок проверки на один аргумент и/или проверки на многих аргументов. Логика в такой проверке обычно проста и кратка, а также для исключения, которое бросается.
Однако, я бы никогда записи
if (someConditional)
DoSomeStuff();
Я всегда пишу
if (someConditional)
{
DoSomeStuff();
}
Так что в итоге:
- Используйте фигурные скобки, если если оператор разбит на несколько линий
- Не используйте фигурные скобки для простой проверки аргументов и т. Д., Которые можно легко (и читаемо) поставить на одну строку
Может ли StyleCop помочь мне здесь?
@Richard E: что вы в конечном итоге сделали? Я не хочу распускать правило, но я хочу написать свои предварительные условия, как описываемый вами образец. –
@Lieven: На данный момент мы оставили это правило и решили использовать фигурные скобки на наших однострочных условиях. С этой целью мы отключили правило StyleCop SA1501. –
OffTopic: не бросайте ArgumentNullException, если переменная пустая строка, вы должны использовать ее только для нулевых параметров. –