С такими проектами приходится работать

← →
Германн © (2006-08-12 02:16) [80]

Вышеуказанное следует писАть как:
with MyComponent.Create() do try begin ... end; finally Free; end;

← →
Германн © (2006-08-12 02:19) [81]

> Zeqfreed_ (12.08.06 02:14) [79]
>
> > [74] Германн © (12.08.06 01:57)
>
> О, это его находка? Тогда премного ему благодарен, очень
> удобная штука!
>

Если я тебя правильно понял, то мы все ему премного благодарны :-)

← →
Gero © (2006-08-12 02:19) [82]

> [80] Германн © (12.08.06 02:16)

> begin
> ...
> end;

Лишнее.

← →
Marser © (2006-08-12 02:22) [83]

> [79] Zeqfreed_ (12.08.06 02:14)
> > [74] Германн © (12.08.06 01:57)
>
> О, это его находка?

Нет, конечно же :-)
Просто он когда-то подмечал "мелочи" вроде той, что я подметил у Германна, называя их такими, по которым видно программиста.

← →
Германн © (2006-08-12 02:23) [84]

> Gero © (12.08.06 02:19) [82]
>
> > [80] Германн © (12.08.06 02:16)
>
>
> > begin
> > ...
> > end;
>
> Лишнее.
>

Это только наглядная подсказка к "..." :-)

← →
Zeqfreed_ (2006-08-12 02:25) [85]

> [83] Marser © (12.08.06 02:22)

Приятно хоть чем-то быть похожим на программиста :)

← →
Джо © (2006-08-12 02:25) [86]

> [84] Германн © (12.08.06 02:23)
> Это только наглядная подсказка к "..." :-)

Они лишнее потому, что блок with будет относится ко всему блоку try/finally :)

← →
Германн © (2006-08-12 02:37) [87]

> Джо © (12.08.06 02:25) [86]
>
> > [84] Германн © (12.08.06 02:23)
> > Это только наглядная подсказка к "..." :-)
>
> Они лишнее потому, что блок with будет относится ко всему
> блоку try/finally :)
>

???
Кто-то кого-то явно не понял! :-(

← →
Джо © (2006-08-12 02:46) [88]

> [87] Германн © (12.08.06 02:37)
> ???
> Кто-то кого-то явно не понял! :-(

Похоже, что я. Тогда сорри :)

← →
Германн © (2006-08-12 03:04) [89]

> Джо © (12.08.06 02:46) [88]
>
> > [87] Германн © (12.08.06 02:37)
> > ???
> > Кто-то кого-то явно не понял! :-(
>
> Похоже, что я. Тогда сорри :)
>

Должен, во избежание, добавить. Я "очень" люблю begin..end.
Лично для меня всё то, что не умещается в, примерно, 60 символов в строке, должно оформляться, особо. А begin..end очень часто помогает!

← →
Marser © (2006-08-12 03:08) [90]

> Лично для меня всё то, что не умещается в, примерно, 60
> символов в строке, должно оформляться, особо. А begin..end
> очень часто помогает!

То есть, вы его используете не только как составной оператор, но и дополнительно?

> Marser © (12.08.06 03:08) [90]
>
> > Лично для меня всё то, что не умещается в, примерно, 60
> > символов в строке, должно оформляться, особо. А begin.
> .end
> > очень часто помогает!
>
> То есть, вы его используете не только как составной оператор,
> но и дополнительно?
>

Кого его? Не понял :-( Уточни Сергей.
Если про begin..end, то да очень часто пишу:
if Sentence = Needed then begin MyVar:=... end;
Хотя в данном примере begin..end не нужен.

> Если про begin..end

Да.

> то да очень часто пишу:
> if Sentence = Needed then begin
> MyVar:=...
> end;
> Хотя в данном примере begin..end не нужен.

Кошмар. Не люблю лишнего в коде. С трудом приучаю себя ставить точку с запятой перед end, после того, как осознал потребность в этом при добавлении строчки кода в конец блока. Причём, борландовцы так и делают, очевидно, из тех же соображений.

> Marser © (12.08.06 03:22) [92]
>
> Кошмар. Не люблю лишнего в коде. С трудом приучаю себя ставить
> точку с запятой перед end, после того, как осознал потребность
> в этом при добавлении строчки кода в конец блока. Причём,
> борландовцы так и делают, очевидно, из тех же соображений.>

Кошмар. Не спорю. Но нагляднее. По крайней мере для меня.

Вот это "развили" тему :)

Прошу прощение, если кого-то задело слово "Женщиной". Ничего плого не имел ввиду :(

Вот мои собственные замечания по коду:

Программа начиналась разрабатываться в Delphi 2, поэтому ни о каком переводе из Turbo паскаля речи быть не могло. Очевидно, что разработчица - сишница, так как:
- названия классов начинаются с "С"
- имена полей классов начинаются с "m"
- для создания экземпляров классов используется оператор new

В проекте класс CPayment используется исключительно в качестве record, а pCPayment = ^CPayment; - это "слышал звон, но не знаю где он".

Почему-то меня больше всего не устроила утечка памяти, возникающая в результате new(item); (соответствующие операторы dispose в проекте отсутствуют). Утечка памяти конечно небольшая - 4 байта, и она обнаружилась не в рун-тайме.

Представьте: пол-проекта написано в таком стиле. Руководитель мне - работает - не трожь! Вот и приходится придерживаться такого же дурацкого стиля, скрипя зубами :(

> [2] Самое жуткое, что за такое кто-то столько бабла срубил...

На вряд-ли. Работала девушка около пол-года. Кстати, прославилась очень компитентным программистом. Мне до нее далеко (изучение предметной области - 90 %-ный залог успеха).

Во всей Пензе не более пяти фирм (а может еще меньше), в которых программят на Delphi, поэтому отбор был очень жестким (она в свое время прошла)

> [5] совсем не обязательно. фирма искала программиста за 500
> уёв. ну и нашла...

Это же Пенза. 500 уёв получает разве что один директор фирмы.

> [6] Есть мнение, что
> CPayment = class
> должно выглядеть (а возможно и задумывалось) как
> CPayment = record

Поддерживаю.

> [7] А по какому поводу негодование? Я кроме
> > m_Name : string;
> > m_Type : word; // нал\безнал
> никакого криминала не заметил...

Я негодую даже по такому поводу. Очень уж люблю придерживаться стандартов VCL, где все поля классов начинаются с "F".

> [8] Учти, что второй случай дискриминации женщины

Причем тут это. Эсли бы данный код писался мужчиной, я бы его нашел и пристрелил. В упор. Из BFG.
:))

> [11] Поясните новичку что тут не так.
> Насколько я понял, ошибка - работа через указатель, без
> которого можно обойтись.

А новичку все-равно не понять, имхо.

> [13] Ошибка, как считает, автор сабжа - в женщине.

Все опошлит! Не уж-то все к.т.н. такие :))

> [17] Программа, я так понимаю работает? И выполняет возложенное
> на нее? Если да, то какая разница, какой у нее код? Если
> смотреть со стороны пользователя.

Я не пользователь а программист, и смотрю на программный код с точки зрения программиста.

> [22] Покажите сие чудо.
> Желательно в анфас и профил.
>
> Можно - prifile.
> Не, интересно просто.

Не покажу. Сам не видел и не знаю ее.

> [57]

Да... И как только Керку удалось избежать суицида ;))

> [72] А тут не говорили о качестве и ценности продукта. Тут как
> раз о красоте кода и говорили. Если я не ошибаюсь, конечно
> :)

Все правильно. К качеству и ценности придраться нельзя.

> Медведъ (11.08.06 19:34) [12]
> а еще женщине нельзя за руль

эт ты зря! мы вот с женой едем в гости, я пью пиво с друзьями, а она меня потом пьяного домой везет! красота!

С такими проектами приходится работать Найти похожие ветки