Баги, баги, баги... Как же их много... Немудрено и фобию заиметь. И ведь никогда не знаешь, чем обернётся очередная ошибка в коде. Страх перед неизвестным, желание держать всё под контролем — закономерные спутники магического мышления. Но подождите, какие ещё магические ритуалы в 21 веке? Тем более у программистов... https://pvs-studio.ru/ru/blog/posts/1139/

PVS-Studio

Тотемные утки и шаманские бубны: есть ли у разработчиков свои ритуалы и приметы?

Баги, баги, баги... Как же их много... Немудрено и фобию заиметь. И ведь никогда не знаешь, чем обернётся очередная ошибка в коде. Страх перед неизвестным, желание держать всё под контролем...

Ответ на задачу: V766 An item with the same key ''.'' has already been added. wxCodeCompletionBoxManager.cpp:19

Видите здесь неладное? Из-за такого количества одинарных кавычек глаз вполне может не заметить, что здесь повторно добавляется символ '.'. Возможно, что здесь забыли добавить какой-то другой символ. Либо это просто случайный дубликат, и его можно убрать.

Ответ на задачу: В ветке if результат выражения сохраняется в переменную localMax. Однако записанное значение не используется до записи нового значения '\' в localMax.

Ответ на задачу: Если указатель end нулевой, то к нему прибавляется длина строки. В результате получается невалидный указатель, который нельзя использовать. Скорее всего, на самом деле хотели написать вот это:
end = beg + strlen(beg) ;

Ответ на задачу: Приоритет у оператора '^' выше, чему у '??'. Если MinimumRevisionAgeToKeep будет иметь значение null, то результат вычисления hashCode в выделенной строке всегда будет равен 0. Для исправления достаточно взять выражение с '??' в скобки.

Ответ на задачу:
Рассмотрим первое условие метода Read. В нём производится проверка значения reader.TokenType. Если оно не соответствует JsonTokenType.StartObject, то будет выброшено исключение. Из этого следует, что после условия и до следующего вызова метода Read, reader.TokenType будет иметь значение JsonTokenType.StartObject. Получается, что в блок then второго if поток выполнения не зайдёт никогда. Возможно, после первого if был пропущен вызов Read.

Друзья, мы стремимся не только к качеству, но еще и к удобству 😌

В будущем мы планируем проводить больше вебинаров. В связи с этим у нас вопрос 👇🏻

Друзья, вот и видео с митапа, на котором Андрей Карпов рассказывал о статическом анализе. Приятного просмотра!

https://www.youtube.com/live/HIp7V7rdpXw?si=tSK82Fj9C8G8PESV

YouTube

Пишем качественный код

Программисты привыкли к тому, что за качество кода отвечают QA-инженеры, они же тестировщики. Можно совсем не следить за качеством и писать как бог на душу положит — все ошибки найдут во время тестирования. Это демотивирует и не нравится многим программистам.…