суббота, 11 февраля 2012 г.

По роду деятельности занимаюсь доработкой\разработкой софта на делфи, и в текущем проекте встречаются комментарии или просто даже некоторые странные места, вызывающие странные ощущения. Порой забавные, порой заставляющие задуматься. Так как набралось их уже не один и не два, решил поделиться. Итак:

// ???-?? ??????? ?????? ? ????? ??? ????? ????????
Первое место по праву. Нет, с кодировкой там всё в порядке. Но расшифровать удается только первые два слова. "кто-то"? "что-то"? "где-то"? 

//для читабельности :) 

Поверьте, смайлик символизирует тут видимо ироничность дальнейшего кода. Поверьте, читабельностью дальнейший код не блещет.

//А ещё стояла задача тупо и жестко хардкодить!!! так что вот такой вот огород :)!!!
Вот тут правда огород. Хардкода на 250+ строчек. Хороший такой метод, длинный :)

//   // Импорт-экспорт документов
//   // Неизвестно, где используется, поэтому скроем
Без комментариев.

  // через четыре года
  // здесь будет город сад ...
Там ничего не работало ;)

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

if _Settings = nil then
begin
  ShowMessage('Ошибка - Пропал объект _Settings! К разработчикам');
  Exit;
end;
Этот код примерно тоже самое делает. Самой крутой вывод ошибок.
Надо сказать, что системе уже много много лет, и разработчиков там поковырялось, надо полагать, достаточно. Поэтому иногда встречаются такие забавные места:

{
 IAP - 20030417
 Временно.
}
Нет, я, в принципе, не против временного. Вы на дату посмотрите...
И еще немного временного кода:

Showmessage('сейчас только возвращаю статус в первичную обработку без контролей! потом будет по-человечески');
Да, это сообщение для пользователя!

// Удалить!!! т.е. Вернуть на место=снять комментарий 
Увы, не смог помочь тому разработчику, не телепат.

//временное (?) :) 
Тут вообще похоже на переписку двух или трех разработчиков.

// !!! Или лучше по-другому??? !!!
// Да точно, лучше по-другому!
И тут переписка :) 

// Что новенького...
Ммм... 

// это для того, чтобы повторно не запрашивать данные на одну и ту же запись (западло искать, почему иногда такое бывает)
Ага, правда, фиг его знает :)

{TODO: ???} 
Да уш, я сам непонял.

Самое занятное, что, наверняка, это далеко не всё, что там есть. Каждую неделю находится что-то новенькое.

4 комментария:

  1. "'Попробуйте выполнить запрос в другое время (или через пару минут)'" - убило :))

    ОтветитьУдалить
  2. То ли еще будет. У меня уже накопилось немножко еще.

    ОтветитьУдалить
  3. Жесть. Вообще не понимаю к чему такие комментарии? Я вижу смысл в коментариях только для документации и для TODO'шек всяких

    ОтветитьУдалить
  4. Документация вполне отлично смотрится как отдельный документ, например вики. Что касается комментариев в коде, то код надо писать так, чтобы они были попросту не нужны. И ставить их только в исключительных случаях.

    ОтветитьУдалить