Тема ревизии кода неисчерпаема. Сегодняшняя находка стала поводом для минутного ступора.

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

К счастью, в коде формы есть только одно место, где значения кнопок используются. Видимо, во избежании путаницы код оформлен следующим образом:

var ОбъёмТакиОграничен: boolean;
...
if КнопкаОбНеограничен.Выбрана then
  ОбъёмТакиОграничен := true
else
  ОбъёмТакиОграничен := false;
...
ВызовКакойТоФункции(ОбъёмТакиОграничен);

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

Стаканчик охлажденного "Шабли" урожая 2006 года на ужин - лучший итог дня.