Наболело :(

← →
Юрий Федоров © (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

Точно!

>Юрий Федоров

Я тут подумал-подумал и решил, что за такие бабки, пожалуй, переписал бы эту D1 хоть бы и на Basic"е!

uw © (30.01.03 12:56)
Около 25 MB исходника, около 10% - ассемблерные вставки, все в таком стиле :) Берешься :)?

На самом деле тебе (как и мне) такие бабки не предлагают, там исключительный случай был (дополнение к вопросу о особом отношении к "своим" со стороны руководства)

Кило зеленых за мегабайт

Почему? Первый - я!

Представляете - кило за мегабайт ТАКОГО !
минимум нужно 100 кило

Мне тоже подобной читабельности сорс перепал на днях.Сочувствую.

>uw © (30.01.03 11:13)

>Теперь замучился ставить v.8.1.7 на P4. C помощью соседнего
>форума кое-как поставил, правда, немного криво. Но нет худа без
>добра. Побродил по конторе - нашел Oracle 9i.

Oracle 9i, по слухам, пока сыроват и кривоват. А как поставить 8i на Р4 абсолютно прямо - могу рассказать. Сам до этого в свое время доходил.

Наболело :( Найти похожие ветки