Найди ошибку в программе коде

Время на прочтение
2 мин

Количество просмотров 8.6K

Авторы анализатора PVS-Studio предлагают вам проверить свою внимательность и развлечься. Попробуйте быстро отыскать баг в фрагменте исходного кода и ткнуть в него мышкой.

Анализаторы кода работают без устали и умеют находить множество ошибок, которые сложно заметить. Мы отобрали несколько фрагментов кода, в которых выявили ошибки с помощью PVS-Studio. Все фрагменты взяты из известных Open Source проектов.

Предлагаем вам посоревноваться с анализатором в прозорливости и попробовать самостоятельно найти ошибки. Вам будет предложено 10 случайно выбранных заданий. За верный ответ начисляется одно очко, если баг найден в течение 1 минуты.

Ограничение в 1 минуту сделано для интереса. Иначе вы, скорее всего, верно найдёте и укажете каждую ошибку, так как фрагменты кода короткие. В любом случае относитесь к этому просто как к игре, а не как к настоящему тестированию программистских навыков у вас или ваших коллег

Когда нашли ошибку, выделите её кликом мышки и нажмите кнопку «Ответ». Бывает, что в коде есть сразу несколько мест, куда вы можете «ткнуть» — и ответ зачтётся как правильный. Поясним это на примере.

case FuriHalSubGhzPreset2FSKDev476Async:
  preset_name = "FuriHalSubGhzPreset2FSKDev476Async";
  break;
  FURI_LOG_E(SUBGHZ_PARSER_TAG, "Unknown preset");
default:

Этот код взят из проекта FlipperZero. Анализатор PVS-Studio сообщает, что часть кода никогда не выполняется: V779 [CWE-561, CERT-MSC12-C] Unreachable code detected. It is possible that an error is present. subghz_i.c 44

Кто-то поспешил и использовал макрос логирования после оператора break. Или это следствие неудачного рефакторинга. В любом случае ошибка очевидна, но вот куда именно ткнуть мышкой – вопрос сложнее.

С одной стороны, в качестве ответа можно выбрать оператор break. Он расположен до макроса FURI_LOG_E и прерывает выполнение оператора switch. Значит, проблема здесь.

С другой стороны, можно выбрать макрос логирования. Ведь это недостижимый код.

Так как же быть? Очень просто. В данном случае правильным ответом будет считаться как выделенный оператор break, так и макрос FURI_LOG_E.

Думаем, правила понятны. Желаем вам удачи: начать игру.

Не забудьте, показать этот Quiz вашим коллегам! Развлекайтесь, и безбажного вам кода!

Если хотите поделиться этой статьей с англоязычной аудиторией, то прошу использовать ссылку на перевод: Andrey Karpov. PVS-Studio’s challenge: can you spot an error?.

Найдите ошибку в коде

unsigned int i;
for (i = 100; i >= 0; --i)
	printf("%dn", i);

unsigned int i;
for (i = 100; i > 0; --i)
	printf("%dn", i);
printf("%dn", i);

unsigned int i;
for (i = 100; i > 0; --i)
	printf("%un", i);

Добрый день Программа должна высчитывать определитель квадратной матрицы, размер которой вводится с клавиатуры, но при n > 2 программа высчитывает ее неправильно.
К примеру, для матрицы:

41 67 34
 0 69 24
78 58 62

Определитель должен равняться 60762, но программа выводит 79212.

Код программы:

#include <stdio.h>
#include <stdlib.h>
int main()
{
  int n;
  printf("Vvedite razmer n kvadratnoi matrici: ");
  scanf("%d", &n);
  int i = 0, j = 0, k = 0;
  int a[n][n], d = 1, r = 0;
  for (i = 0; i < n; i++)
  {
    for (j = 0; j < n; j++)
    {
      a[i][j] = rand() % 100;
      printf("%d ", a[i][j]);
    }
    printf("n");
  }
  if (n == 1)
  {
    d = a[0][0];
    printf("Opredelitel matrici raven - %d", d);
    return 0;
  }
  if (n == 2)
  {
    d = a[0][0] * a[1][1] - (a[1][0] * a[0][1]);
    printf("Opredelitel matrici raven - %d", d);
    return 0;
  }
  if (n > 2)
  {
    for (i = 0; i < n; i++)
    {
      for (j = 0; j < n; j++)
      {
        if (j > i)
        {
          r = a[j][i] / a[i][i];
          for (k = 0; k < n; k++) a[j][k] -= r * a[i][k];
        }
      }
    }
  }
  for(i = 0; i < n; i++) d *= a[i][i];
  printf("Opredelitel matrici raven - %d", d);
  return 0;
}

Помогите, пожалуйста, найти ошибку.