ТОП-10 ошибок, найденных в C#-проектах за 2020 год

[questor]

А если брать не интересные ошибки, а самые массовые и типичные, то что будет в ТОПе?

[Andrey2008]

Статистики по C# у нас не так много, как по C++, но тем не менее:

• 5. V3019
• 4. V3080
• 3. V3095
• 2. V3001
• 1. V3022

[a-tk]

По идее V3019 должно кануть в лету из-за C#7 благодаря синтаксису вида

if (something is SomeType alias) {...}

V3080 и V3095 быть может благодаря nullable reference types в C#8 тоже когда-нибудь в необозримо отдалённом будущем…

[mayorovp]

Синтаксис и комментария выше не требует поддержки со стороны рантайма

[Arbane]

Возможно, будет то же. На моей практике больше всего ошибок находишь именно там, где код «очевиден», типа истории с «password». Так что с местами в целом согласен.

[imanushin]

if (string.IsNullOrEmpty(value))
{
   _rawOuterXml = string.Empty;
}

_rawOuterXml = value;

В отрывке выше хорошо видно, почему в языках крайне полезен функциональный синтаксис.

В условных Kotlin/Scala код корректнее выглядит так, как ниже. А потому в нем сложнее забыть про отсутствие else, так как без else выражение просто не скомпилируется.

_rawOuterXml = if(string.IsNullOrEmpty(value)) 
   {
       string.Empty;
   } 
   else 
   {
       value;
   }

Можно надеяться, что в C# добавят if-expression, по аналогии со switch-expression'ами в C# 8.

[mayorovp]

А чем тернарный оператор не угодил?

_rawOuterXml = string.IsNullOrEmpty(value) ? "" : value;

[imanushin]

Я про сложные выражения. Для коротких Вы полностью правы — лучше использовать ?:, который, по сути, тоже является функциональным.

А вот если в каждой ветки условия необходимо сделать что-то еще, то тернарного оператора не хватает, а потому желательно использование функционального if`a.

[mayorovp]

Или перестать страдать ерундой и выделить кусок кода в отдельный метод.

Ну или, как вариант, просто перестать бояться писать слово else…

[a-tk]

Хватит и

_rawOuterXml = value ?? "";