Дыбр

[moonkan]

Получил с утра в понедельник благодарственно письмо от клиентов за быстро и качественно сделанную в пятницу работу (знали-бы они сколько я ругался матом =) ). Приятное начало недели, да.

Сейчас вот сижу в задумчивости, что или кто могли автора побудить назвать функцию removeCandidates, если сама функция, судя по коду, этих самых Candidates добавляет? Что он курил? Нет, функция не 100 раз переписанная - специально глянул в репозитории, за 2 года с момента появления в классе вообще не поменялась. Вообще тот же автор еще в пятницу нас порадовал открытием другого укуренного куска кода.
дословно не помню, но там было что то типа :
while ((buffer.canRead() && i = buffer.readNext())>0)
(может не совсем верно процитировал, но там было именно логическое умножение булина на считанный из буфера символ и сравнение этой операции с нулем ...)
Задавались втроем с коллегами ровно тем-же вопросом, что я сегодня - зачем и как?

Comments

[snopova.livejournal.com]

Помню из своего прошлого, что команда "remove" обозначала "переместить", а не "удалить", что неправильно, конечно, но кто-то может до сих ошибаться.

[tau-schmau.livejournal.com]

Точно. Remove - переместить кандидата из одного списка в другой, например, а удалить кандидата - это Delete! Нас не проведешь! :)

[pritkiy-kaban.livejournal.com]

Эх, не слушает народ опытных людей, а те уже сказали...

"Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live."

[nickshumi.livejournal.com]

Всегда меня эта фраза смущала...она некоторых может мотивировать.У меня вот есть арбалет. А протестировать не ком. Я не пишу плохой код, я ловлю психопатов на живца!)))

[p4553d.livejournal.com]

Последнее вполне корявенький, но рабочеспособный кусок С. Зависит сильно от семантики того самого буфера. Он может содержать всякую фигню, идет проверка что это не пустая строка или не достигли конца строки.

[moonkan.livejournal.com]

Я знаю, даже подобные примеры в Ц видел в каких-то исходниках. Но, имхо - слишком заумно сделано.

[ikaktys.livejournal.com]

у нас тут у одного старенького шефа на стене висел кусок высоковольтного кабеля, красного такого, сантиметров 5 толшщиной и сантиметров 70 длиной, он его называл "Erklärer" и обещал применять именно для тех кто такие выверты в с++ любит.

[p4553d.livejournal.com]

Вполне с ним соглашусь. Хотя в старом коде не заставил бы переделывать, в С это дань сокращениям и процедуральности языка.