Форум: "Потрепаться";
Текущий архив: 2003.02.13;
Скачать: [xml.tar.bz2];
ВнизНаболело :( Найти похожие ветки
← →
Юрий Федоров (2003-01-29 18:24) [0]Можно ли по такому фрагменту кода что-либо сказать о программисте, его породившем, или я черезчур раздражителен сегодня ?
with wpmlWPAR do begin
wpmlContoursLayer^.ChangeLayerAttr(wpmlPrLinesIndexesArray, wpmlPrLinesArray, wpPrLineBkColor,
wpPrLineDrMode, wpPrLineBkMode, wpmlPrLinePen, mpCenter, wpMarkerDistance / 1000,
wpMarkerSize / 1000, mtNone, GetStockObject(NULL_BRUSH), wpmlMarkerPen, Nil, 0.01, FIR);
wpmlDashMLayer^.ChangeLayerAttr(@wpmlPrMarkersArray, wpMarkerSize,
wpMarkerPenStyle, wpMarkerPenWidth, wpMarkerPenColor, wpPrLineBkColor,
wpPrLineBkMode, wpPrLineDrMode);
end;
← →
Chubais (2003-01-29 18:32) [1]ничо особенного...
бывает гораздо хуже
← →
pl_quad (2003-01-29 18:40) [2]если работает, то ничего не трогать
← →
Юрий Федоров (2003-01-29 18:53) [3]В том то и дело что не работает :) В общем так :
1. D1
2. Отсутствие комментариев в коде
3. Максимальное количество параметров метода в модуле - 27 !!!
4. Любимая китайская технология реиспользования кода copy\paste используется в хвост и в гриву.
5. Отстутствие отладчика.
6. Постоянное переполнение стека за счет ограничений D1
7. Постоянное падение самой D1 в процессе работы
8. Отсутствие автора (уволился).
9. Шаг вправо(влево) - нехватка сегмента данных
10. Вчера должно было быть готово
И всем этим приходится заниматься мне :(
Поэтому я весь день ругаюсь матом :) И вот решил поделиться своим горем
← →
down (2003-01-29 19:00) [4]ммм... 1. разыменование (^) тут ни к чему
2. функции с таким кол-вом параметров - это нехорошо
ну и комментарий бы не помешал, а так вполне нормальный код
← →
Chubais (2003-01-29 19:03) [5]Юрий Федоров © (29.01.03 18:53)
я бы тоже в такой ситуации разговаривал матом в течении недели))
← →
uw (2003-01-29 19:47) [6]Уволился, потому что мало платили!
← →
Serious Sam (2003-01-29 20:42) [7]Чисто паскалевский стиль :)
Цэ-шник определил бы структуру и передавал бы её по ссылке...
← →
Ihor Osov'yak (2003-01-29 20:57) [8]2 Serious Sam (29.01.03 20:42)
Идиотизм не зависит от того, на каком языке писано...
Видел перлы и похлеще, на си писанные....
Зы - если бы не with вначале - сказал-бы типично сишный стиль :-)
← →
Кот Бегемот (2003-01-30 09:10) [9]
with wpmlWPAR do begin
// Нууу параметров наверное многовато у функции
wpmlContoursLayer.ChangeLayerAttr(
wpmlPrLinesIndexesArray, // Индексный массив ???
wpmlPrLinesArray, // Это вероятно массив линий
wpPrLineBkColor, // background color
wpPrLineDrMode, // draw mode
wpPrLineBkMode, // background mode
wpmlPrLinePen, // pen for line
mpCenter, // стиль выравнивания
wpMarkerDistance / 1000, // нууу каято дистанция маркера
wpMarkerSize / 1000, // marker size :))) ^
mtNone, // что такое mt - не знаю |
GetStockObject(NULL_BRUSH), // :) |
wpmlMarkerPen, // marker pen - но я бы поставил это сюда |
Nil,
0.01, // Вот это непонятно что
FIR); // ???
// Тут поменьше - уже лучше - параметры такие же
wpmlDashMLayer.ChangeLayerAttr(
@wpmlPrMarkersArray,
wpMarkerSize,
wpMarkerPenStyle,
wpMarkerPenWidth,
wpMarkerPenColor,
wpPrLineBkColor,
wpPrLineBkMode,
wpPrLineDrMode);
end;
← →
Anatoly Podgoretsky (2003-01-30 09:21) [10]Налицо увлечение венгерской нотацией, которое не пошло на пользу, плюс обострение за счет with и неумение (или незнание) писать в столбик, а это при таком количестве параметров ой как надо.
И как я понимаю к функции отсутствует описание парметров и другой информации
В результате следующему поколению пришлось ломать голову (надеюсь не сломал).
Это неправильно, что мало платили, надо еще меньше.
← →
Юрий Федоров (2003-01-30 09:23) [11]Спасибо всем за моральную поддержку :)
>>Chubais © (29.01.03 19:03)
Я по этой причине разговариваю в основном матом в течение последних полутора лет :)
>>uw © (29.01.03 19:47)
Платили весьма прилично, намного больше чем подавляющему большинству участников этого форума
>>Serious Sam (29.01.03 20:42)
>>Чисто паскалевский стиль :)
Написано было на паскале, во времена, когда D1 еще и в помине не было
← →
Игорь Шевченко (2003-01-30 09:39) [12]Мартин Фаулер: "Рефакторинг. Улучшение существующего кода" - рекомендую.
Заказ на www.books.ru, тирах 3000 экз.
Очень полезная в таких тяжелых случаях книга.
← →
uw (2003-01-30 09:44) [13]>Anatoly Podgoretsky & Юрий Федоров
Если бы ему платили много, черта с два он нашел бы другую работу, и человеку не надо было бы мучиться!
← →
Юрий Федоров (2003-01-30 09:51) [14]>>uw © (30.01.03 09:44)
При переходе на другую работу он потерял в зарплате в 3 (!!!) раза и вышел на средний для Москвы уровень
>>Игорь Шевченко © (30.01.03 09:39)
Спасибо, похоже придется прибегнуть к такой литературе :)
← →
Explorer (2003-01-30 09:58) [15]>uw ©
Да он, наверное, уволился из-за того, что сам забыл что написал!
← →
Anatoly Podgoretsky (2003-01-30 10:09) [16]Юрий Федоров © (30.01.03 09:51)
Это тоже неправильно, нельзя так много платить, он максимум тянет на половину среднего уровня и последствия для работадателя тяжелые, это не ему надо платить, а он должен оплачивать ущерб.
← →
Anatoly Podgoretsky (2003-01-30 10:09) [17]Разумеется совместно с тем, кто такое допустил.
← →
REA (2003-01-30 11:12) [18]Хорошо еще не aaa(bbb, ddd, eee) - и такое бывает
← →
uw (2003-01-30 11:13) [19]Нет, мужики, вам надо отсыпаться. Иначе вы не стали бы так серьезно относиться к моим предложениям. Но все-таки платить надо и тем, кто вообще не умеет писать на Delphi.
Вот у нас уволился оракловский администратор. Раньше у меня просто не было проблем. Теперь замучился ставить v.8.1.7 на P4. C помощью соседнего форума кое-как поставил, правда, немного криво. Но нет худа без добра. Побродил по конторе - нашел Oracle 9i. Дают мне три 5-дюймовых дискеты. У меня лицо вытянулось. Примерно как у Anatoly Podgoretsky после моего предложения платить тому парню. Оказывается, у этой дискеты отрезается одна из сторон, и туда вкладывается CD. А я почему-то не выкинул сотню старых дискет!
Что касается стиля нашего программиста, то каждый из нас пишет, как может, и даже хуже того. Ну кто помнит о комментариях, когда находится в состоянии потока. А у этого - длинные и весьма осмысленные идентификаторы. Префиксы, правда, несколько не нужны.
Я, как правило, против них. Нет, в константах и стандартных компонентах я их использую.
← →
Anatoly Podgoretsky (2003-01-30 11:26) [20]uw © (30.01.03 11:13)
Кто же отказывает, но не тройной же уровень и что фирме налевать на убытки?
А с дискетами ты лопухнулся, использую этот метод с начала 90 очень удобно, так же использубтся от них и конвертики, те которые толстые.
Что касается как может, то для себя пиши как хочешь, а в остальных случаях вредительство и в первую очередь со стороны руководства, а не конкретного программиста.
← →
Ketmar (2003-01-30 11:52) [21]>Кот Бегемот © (30.01.03 09:10)
mt - это Marker Type. просыпаться надо ПЕРЕД тем, как на работу идти %-)
Satanas Nobiscum! 30-Jan-XXXVIII A.S.
← →
Lola (2003-01-30 11:53) [22]Насчет комментариев, тут народ прав, писать надо. Я уже через месяц не могу понять как-что-куда, и ведь все-равно не пишу, потому-как просто не успеваю.
← →
Alexey_B (2003-01-30 12:19) [23]Комментарии - это издержки неправильного кода (из идеологии ХР).
В идеале - комментариев быть не должно, но дизайн должен быть абсолютно понятным. В остальном разумные оптимизации приведенного куска кода уже были предложены.
← →
stone (2003-01-30 12:32) [24]
> А с дискетами ты лопухнулся, использую этот метод с начала
> 90 очень удобно, так же использубтся от них и конвертики,
> те которые толстые.
У меня старых дискет нет :)) поэтому я храню диски в CD Envelope, а по простому конверт для CD. Стоит 2,5 р. штука, вмещает 2 диска, имеет дырки для подшивки. Все это дело в скоросшиватель формата А5 (35 р.). В результате получается альбомчик, содержащий 24-26 дисков. Очень удобно в плане хранения и поиска.
← →
Ketmar (2003-01-30 12:33) [25]>Alexey_B (30.01.03 12:19)
я уже цитировал: "comments are for beginners" (c) pascal aka Niklaus Beizert. правда, коде паскаля никто, кроме паскаля разобраться так и не смог %-)
Satanas Nobiscum! 30-Jan-XXXVIII A.S.
← →
uw (2003-01-30 12:33) [26]>Alexey_B
Точно!
← →
uw (2003-01-30 12:56) [27]>Юрий Федоров
Я тут подумал-подумал и решил, что за такие бабки, пожалуй, переписал бы эту D1 хоть бы и на Basic"е!
← →
Юрий Федоров (2003-01-30 13:01) [28]uw © (30.01.03 12:56)
Около 25 MB исходника, около 10% - ассемблерные вставки, все в таком стиле :) Берешься :)?
На самом деле тебе (как и мне) такие бабки не предлагают, там исключительный случай был (дополнение к вопросу о особом отношении к "своим" со стороны руководства)
← →
Ketmar (2003-01-30 13:38) [29]>Юрий Федоров © (30.01.03 13:01)
а сколько платят? берусь за соотв. сумму.
Satanas Nobiscum! 30-Jan-XXXVIII A.S.
← →
Anatoly Podgoretsky (2003-01-30 13:51) [30]Кило зеленых за мегабайт
← →
Ketmar (2003-01-30 13:56) [31]>Anatoly Podgoretsky © (30.01.03 13:51)
тогда я первый в очереди %-)
Satanas Nobiscum! 30-Jan-XXXVIII A.S.
← →
uw (2003-01-30 14:03) [32]Почему? Первый - я!
← →
Юрий Федоров (2003-01-30 14:51) [33]Представляете - кило за мегабайт ТАКОГО !
минимум нужно 100 кило
← →
Anatoly Podgoretsky (2003-01-30 15:13) [34]Ketmar © (30.01.03 13:56)
Да ты любишь зеленый цвет
← →
Marser (2003-01-30 15:25) [35]Мне тоже подобной читабельности сорс перепал на днях.Сочувствую.
← →
MC TOL (2003-01-30 15:36) [36]>uw © (30.01.03 11:13)
>Теперь замучился ставить v.8.1.7 на P4. C помощью соседнего
>форума кое-как поставил, правда, немного криво. Но нет худа без
>добра. Побродил по конторе - нашел Oracle 9i.
Oracle 9i, по слухам, пока сыроват и кривоват. А как поставить 8i на Р4 абсолютно прямо - могу рассказать. Сам до этого в свое время доходил.
← →
Ketmar (2003-01-30 15:38) [37]>Anatoly Podgoretsky © (30.01.03 15:13)
ну дык. не зря же...
Satanas Nobiscum! 30-Jan-XXXVIII A.S.
← →
uw (2003-01-30 22:14) [38]>MC TOL © (30.01.03 15:36)
Ну, брось в почту, если не трудно.
Страницы: 1 вся ветка
Форум: "Потрепаться";
Текущий архив: 2003.02.13;
Скачать: [xml.tar.bz2];
Память: 0.54 MB
Время: 0.009 c