FirstOrDefault возвращает не индекс, а первый элемент коллекции, но по контексту я думаю, что в коллекции индексы. Так что первый элемент может быть 0.

Думаю, что самым лёгким вариантом будет использование дефолтного значения, если нет подходящего элемента:
FirstOrDefault(-1)

dmitrybarabash Nov 2 2023 at 13:27

Я понимаю, что суть ваших (очень интересных, спасибо вам) постов не в том, чтобы улучшить качество проверяемых продуктов, но всё же спрошу: раз уж вы для написания статей всё равно проверяете продукты, то их разработчикам о результатах сообщаете?

rip_m Nov 2 2023 at 13:32

Конечно, мы всегда сообщаем разработчикам о найденных ошибках и странностях в коде. В основном это issue на GitHub. По PowerToys issue уже создаю, позже добавлю его в коммент.

dmitrybarabash Nov 2 2023 at 13:35

Супер, вы молодец.

lexasss Nov 2 2023 at 15:14

О половине из этих ошибок в С# предупредит сама VS 2022 если в настройках проекта будет установлено использование Nullable. Иногда приходится редактировать свои старые проекты без Nullable - очень неудобно когда не знаешь надо ли делать проверку на null или там гарантированно есть значение.

rip_m Nov 2 2023 at 15:54

Nullable безусловно полезная штука, хотя и холиварная. Кому-то нравится, а кому-то нет. Мне, например среди C# ошибок из этой статьи больше всего нравится про бесполезный '??'. Про этот кейс могу точно сказать, что его VS 2022 не ловит. Да и статические анализаторы же не только разыменования нулевых ссылок ищут.

lexasss Nov 2 2023 at 16:49

Да, кейс с ?? очень хорош. Эта статья напомнила что пора бы опять задействовать ваш продукт, и как раз этот кейс обнаружился в одном из проектов.

К слову, если есть такой код:

int? val;
...
if (val != null)
  model.Use(val ?? 0); 
...
class Model {
  public void Use(int val) { ... }
}

то PVS-Studio выдаст предупреждение в последней строчке, но будет доволен таким кодом:

int? val;
...
if (val != null)
  model.Use(val);

Но тогда VS ругается. Компромис - явное приведение типа:

int? val;
...
if (val != null)
  model.Use((int)val);

a-tk Nov 2 2023 at 17:10

model.Use(val.Value)

Более свежая и прозрачная альтернатива:

int? val;
// ...
if (val is {} actual)
    model.Use(actual);

rip_m Nov 2 2023 at 17:31

Спасибо за пример. Я думаю, что анализатор мог бы для подобных случаев понижать уровень предупреждения, или мы и вовсе могли бы добавить это в исключения. Рассмотрим доработку :).
Если что по всем вопросам можно написать нам в поддержку.

qw1 Nov 4 2023 at 14:09

    HANDLE proc = OpenProcess(PROCESS_TERMINATE, false, g_settings_process_id);
    if (proc != INVALID_HANDLE_VALUE)

А вот что ваш анализатор не смог увидеть, это то, что OpenProcess возвращает NULL в случае неудачи, а не INVALID_HANDLE_VALUE.