VPF::Оцени свой опыт программирования на Си/Си++

Thunderbolt

Дата 17.9.2014, 21:17 (ссылка)

(нет голосов)

Загрузка ...

DevRel

Профиль
Группа: Участник
Сообщений: 122
Регистрация: 7.11.2007
Где: Тула

Репутация: 10
Всего: 16

Оцени свой опыт программирования на Си/Си++

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

Начать игру: http://q.viva64.com/

--------------------

Карпов Андрей, DevRel в PVS-Studio.

feodorv

Дата 2.10.2014, 07:15 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 11
Всего: 45

Спасибо за тест!

Одно но: не всегда понятно, куда именно нужно указать. Например:

Код


 if (areacode & sC0Min1Max) 
 {
     limit[0] = fAxisMin[0];
     limit[1] = fAxisMin[1];
 } 
 else if (areacode & sC0Max1Min) 
 {
     limit[0] = fAxisMax[0];
     limit[1] = fAxisMin[1];
 } 
 else if (areacode & sC0Max1Max) 
 {
     limit[0] = fAxisMax[0];
     limit[1] = fAxisMax[1];
 } 
 else if (areacode & sC0Min1Max) 
 {
     limit[0] = fAxisMin[0];
     limit[1] = fAxisMax[1];
  }
  ....
}

Я, естественно, ткнул мышью во второй экземпляр sC0Min1Max, на что тест мне написал "Incorrect", нужно было в первый. А какая разница?)))

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

Guinness

Дата 2.10.2014, 14:54 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 310
Регистрация: 21.6.2009
Где: Зеленоград

Репутация: нет
Всего: 10

sC0Min1Max - MIN и MAX, меняем fAxisMin[0] и fAxisMin[1]. А должны были бы fAxisMin[0] и fAxisMax[1]. я так понимаю.

borisbn

Дата 2.10.2014, 15:59 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 4875
Регистрация: 6.2.2010
Где: Ростов-на-Дону

Репутация: 22
Всего: 135

Я понял, что в первой строке должно быть
if (areacode & sC0Min1Min)

Это сообщение отредактировал(а) borisbn - 2.10.2014, 15:59

--------------------

Женщины отличаются от программистов тем, что у них чары состоят из стрингов

Guinness

Дата 2.10.2014, 16:04 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 310
Регистрация: 21.6.2009
Где: Зеленоград

Репутация: нет
Всего: 10

Цитата(borisbn @ 2.10.2014, 16:59

)

Я понял, что в первой строке должно быть
if (areacode & sC0Min1Min)

В смысле, да, неправильно объяснил.

Это сообщение отредактировал(а) Guinness - 2.10.2014, 16:04

feodorv

Дата 2.10.2014, 16:30 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 11
Всего: 45

Цитата(Guinness @ 2.10.2014, 15:54

)

sC0Min1Max - MIN и MAX, меняем fAxisMin[0] и fAxisMin[1]. А должны были бы fAxisMin[0] и fAxisMax[1]. я так понимаю

Хотя да, нужен именно первый экземпляр sC0Min1Max (вместо которого нужен sC0Min1Min). Прошу прощения)))

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »