// ???-?? ??????? ?????? ? ????? ??? ????? ????????
Первое место по праву. Нет, с кодировкой там всё в порядке. Но расшифровать удается только первые два слова. "кто-то"? "что-то"? "где-то"? //для читабельности :)
Поверьте, смайлик символизирует тут видимо ироничность дальнейшего кода. Поверьте, читабельностью дальнейший код не блещет.
//А ещё стояла задача тупо и жестко хардкодить!!! так что вот такой вот огород :)!!!
Вот тут правда огород. Хардкода на 250+ строчек. Хороший такой метод, длинный :)// // Импорт-экспорт документов
// // Неизвестно, где используется, поэтому скроем
Без комментариев.// // Неизвестно, где используется, поэтому скроем
// через четыре года
// здесь будет город сад ...
Там ничего не работало ;)// здесь будет город сад ...
'Попробуйте выполнить запрос в другое время (или через пару минут)'
А вот это гениальное сообщение выдается пользователю. В другое время, или через несколько минут... ага, или никогда.
if _Settings = nil then
begin
ShowMessage('Ошибка - Пропал объект _Settings! К разработчикам');
Exit;
end;
Этот код примерно тоже самое делает. Самой крутой вывод ошибок.begin
ShowMessage('Ошибка - Пропал объект _Settings! К разработчикам');
Exit;
end;
Надо сказать, что системе уже много много лет, и разработчиков там поковырялось, надо полагать, достаточно. Поэтому иногда встречаются такие забавные места:
{
IAP - 20030417
Временно.
}
Нет, я, в принципе, не против временного. Вы на дату посмотрите...IAP - 20030417
Временно.
}
И еще немного временного кода:
Showmessage('сейчас только возвращаю статус в первичную обработку без контролей! потом будет по-человечески');
Да, это сообщение для пользователя!// Удалить!!! т.е. Вернуть на место=снять комментарий
Увы, не смог помочь тому разработчику, не телепат. //временное (?) :)
Тут вообще похоже на переписку двух или трех разработчиков.
// !!! Или лучше по-другому??? !!!
// Да точно, лучше по-другому!
// Да точно, лучше по-другому!
И тут переписка :)
// Что новенького...
Ммм...
// это для того, чтобы повторно не запрашивать данные на одну и ту же запись (западло искать, почему иногда такое бывает)
Ага, правда, фиг его знает :)
{TODO: ???}
Да уш, я сам непонял.
Самое занятное, что, наверняка, это далеко не всё, что там есть. Каждую неделю находится что-то новенькое.
"'Попробуйте выполнить запрос в другое время (или через пару минут)'" - убило :))
ОтветитьУдалитьТо ли еще будет. У меня уже накопилось немножко еще.
ОтветитьУдалитьЖесть. Вообще не понимаю к чему такие комментарии? Я вижу смысл в коментариях только для документации и для TODO'шек всяких
ОтветитьУдалитьДокументация вполне отлично смотрится как отдельный документ, например вики. Что касается комментариев в коде, то код надо писать так, чтобы они были попросту не нужны. И ставить их только в исключительных случаях.
ОтветитьУдалить