Главная страница
    Top.Mail.Ru    Яндекс.Метрика
Форум: "Прочее";
Текущий архив: 2014.02.16;
Скачать: [xml.tar.bz2];

Вниз

Кто переименовывает, а кто нет? (опрос)   Найти похожие ветки 

 
Степан Потапов   (2013-09-02 18:52) [0]

Здравствуйте мастера! Тут смотрю в одной ветке народ ратует за переименование компонент на форме, решил провести небольшой опрос и узнать кто какого мнения и какой процент кодеров этим занимается? Пишите, не стесняйтесь! За себя напишу что мне просто лень :)


 
Кщд   (2013-09-02 19:05) [1]

>Степан Потапов   (02.09.13 18:52)
Edit1, Edit2, ..., Edit1534?
создаёте нечитаемый код?
дело сугубо Ваше


 
alexdn ©   (2013-09-02 19:08) [2]

> Edit1, Edit2, ..., Edit1534?
и чем же он нечитаемый?


 
Кщд   (2013-09-02 19:17) [3]

>alexdn ©   (02.09.13 19:08) [2]
всем понятно при прочтении в коде, что такое: s := V0*t + a*t^2/2.
но не всем ясно, что такое, опуская поля и очевидные преобразования:  Label362 := Edit324 * Edit142 + Edit89*Edit3/2

впрочем, это не относится к лицам, озаренным нестерпимым сиянием высших истин


 
DVM ©   (2013-09-02 19:17) [4]

Я вообще не понимаю откуда такой вопрос возник? Неужели не очевидно, что код в котором каждая переменная имеет осмысленное имя удобнее в чтении, чем где все называется одинаково и отличается лишь номерами.
Сравните edtUserName и Edit15. О чем вообще разговор, зачем заниматься добровольной обфускацией?


 
Кщд   (2013-09-02 19:17) [5]

>Кщд   (02.09.13 19:17) [3]
забыл квадратик, ах-ах


 
Pavia ©   (2013-09-02 19:27) [6]

У меня данные модель и предстовления разделены. Поэтому мне как-то без различно правильность именованные компонентов. Ведь на входе в данные идёт переименование. И оно достаточно информативно. А так как представление и ввод локализованы, то и нет надобности в именовании в виду того что такое не читаемое имя используется в 2 раза на входе и выходе. А тратить время на это не имеет смысла.


 
Кщд   (2013-09-02 19:48) [7]

>Pavia ©   (02.09.13 19:27) [6]
аналог MVC и автошема?
не смешите мои пинеточки)


 
Степан Потапов   (2013-09-02 19:59) [8]

Не могу сказать что s := V0*t + a*t^2/2 более читаемое выражение чем Label362 := Edit324 * Edit142 + Edit89*Edit3/2 Компилятор тоже наверно не будет в восторге и обязательно возразит в ответ :)


 
картман ©   (2013-09-02 20:02) [9]


> Степан Потапов   (02.09.13 19:59) [8]

тебя пока не должен волновать вопрос переименования компонентов.


 
Степан Потапов   (2013-09-02 20:04) [10]

>картман пример выражения от Кщд   (02.09.13 19:17) [3]


 
Кщд   (2013-09-02 20:13) [11]

>Степан Потапов   (02.09.13 19:59) [8]
через строчку читаем?
"опуская поля и очевидные преобразования"
ещё раз: лично я ничего не имею против людей, которые пишут нечитаемый код


 
Pavia ©   (2013-09-02 20:27) [12]


> >Pavia ©   (02.09.13 19:27) [6]аналог MVC и автошема?не
> смешите мои пинеточки)

Так расскажите как у вы делаете? Что-бы не смешить.


 
Кщд   (2013-09-02 20:33) [13]

>Pavia ©   (02.09.13 20:27) [12]
Вы пользуете нечто, напоминающее MVC
два автора, ратующие за Edit162, пишут автошему
очевидно, что претензии к Вам не было

что пользую я:
1. автогенераторы для view(собственно, это происходит уже на уровне controller) в тех проектах, где пользуется MVC(либо некоторое приближение к ней);
2. смысловые названия переменных, где не используется.


 
Алканавт расправил плечи   (2013-09-02 20:42) [14]


> зачем заниматься добровольной обфускацией?
шоб конкуренты вагонку не потырили. очевидно же.


 
Пит   (2013-09-02 21:26) [15]

как-то странно поставлен вопрос. Не иначе разжечь холивар, но мелко слишком)
Вопрос по другому формулируется - а что, есть те, кто ратует за то, чтобы не переименовывать?

Самое разумное - всегда переименовывать компоненты, доступ к которым используется в ран-тайм ака в коде. Это в серьезных приложениях для которых планируется поддержка и развитие.

Переименовывать ли компоненты, настройка которых осуществляется исключительно в дизайн-тайм - дело вкуса в общем то. В принципе, можно и не переименовывать, например зачастую это различные label.


 
Jeer ©   (2013-09-02 21:46) [16]

Дык, че уж там - пишите на своем быдло-коде и сделайте свой препроцессор на один из стандартных ЯВУ.


 
Кщд   (2013-09-02 21:49) [17]

>Jeer ©   (02.09.13 21:46) [16]
о, давно)


 
Anatoly Podgoretsky ©   (2013-09-02 21:58) [18]

Будешь писать на быдло коде - сам знаешь кем станешь.


 
Кщд   (2013-09-02 22:01) [19]

>Anatoly Podgoretsky ©   (02.09.13 21:58) [18]
Анатолий, Вы же из Новосибирска, а не из Эстонии, в самом-то деле


 
картман ©   (2013-09-02 22:24) [20]


> Анатолий, Вы же из Новосибирска, а не из Эстонии, в самом-
> то деле

Чтоооо??


 
Ega23 ©   (2013-09-02 22:38) [21]

Если контрол в коде не используется, то нафига его переименовывать?


 
DVM ©   (2013-09-02 22:44) [22]


> Ega23 ©   (02.09.13 22:38) [21]

Чтоб единство стиля выдерживать. Сейчас он не используется, потом используется, а переименовать забыли. Некрасиво выглядят эти все Edit1, Label135


 
Алканавт расправил плечи   (2013-09-02 22:56) [23]


> Если контрол в коде не используется, то нафига его переименовывать?
На него могут ссылаться другие.

НЕ переименовываю (переименовываю по настроению) только чисто оформительские элементы, лейблы-сплиттеры-тулбары. Банальщине вроде DBText уже даю осмысленные имена.

Всякие штуки, существующие в единственном экземпляре, тоже можно бы не переименовывать, но очень уж отвратно единица в конце выглядит.


 
Ega23 ©   (2013-09-02 23:04) [24]


> Чтоб единство стиля выдерживать. Сейчас он не используется,
>  потом используется, а переименовать забыли. Некрасиво выглядят
> эти все Edit1, Label135

Особенно сплиттерам давать осмысленные имена. И каждой кнопочке на тулбаре.


 
Ega23 ©   (2013-09-02 23:05) [25]


> НЕ переименовываю (переименовываю по настроению) только
> чисто оформительские элементы, лейблы-сплиттеры-тулбары.
>  Банальщине вроде DBText уже даю осмысленные имена.


Ну да, именно это я и хоте сказать.


 
Алканавт расправил плечи   (2013-09-02 23:21) [26]


>  И каждой кнопочке на тулбаре.
О, в яблочко. Если формы с поименованными лейблами у нас изредка встречаются, то форм с поименованными кнопками на тулбаре не помню ни одной %-)


 
DVM ©   (2013-09-03 00:02) [27]


> Ega23 ©   (02.09.13 23:04) [24]
>
> > Чтоб единство стиля выдерживать. Сейчас он не используется,
>
> >  потом используется, а переименовать забыли. Некрасиво
> выглядят
> > эти все Edit1, Label135
>
> Особенно сплиттерам давать осмысленные имена. И каждой кнопочке
> на тулбаре.

Ну сплитеров как правило не больше двух - вертикальный и горизонтальный, почему бы и не назвать их spltrVertical и spltrHorizontal например.

Кнопочки на тулбарах при установленном CnPack именуются автоматом, если кнопочкам назначены экшены. То же и к пунктам меню относится.

Короче, я все именую.

Появление в интерфейсе всяких Edit135 считаю признаком того, что форма проектировалась неаккуратно.


 
DVM ©   (2013-09-03 00:07) [28]

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


 
Германн ©   (2013-09-03 00:10) [29]


> DVM ©   (03.09.13 00:02) [27]
>
>
> > Ega23 ©   (02.09.13 23:04) [24]
> >
> > > Чтоб единство стиля выдерживать. Сейчас он не используется,
>
> >
> > >  потом используется, а переименовать забыли. Некрасиво
> > выглядят
> > > эти все Edit1, Label135
> >
> > Особенно сплиттерам давать осмысленные имена. И каждой
> кнопочке
> > на тулбаре.
>
> Ну сплитеров как правило не больше двух - вертикальный и
> горизонтальный, почему бы и не назвать их spltrVertical
> и spltrHorizontal например.


У меня меньше трех никак не получается. :( Но вертикальный всегда один.) Даже сам удивляюсь почему. Ведь на нетбуках ширина гораздо больше высоты!


> DVM ©   (03.09.13 00:02) [27]>
> Кнопочки на тулбарах при установленном CnPack именуются
> автоматом, если кнопочкам назначены экшены. То же и к пунктам
> меню относится.
>
> Короче, я все именую.

Меня просто коробит от тех имён, которые предлагает CnPack.


 
Jeer ©   (2013-09-03 00:11) [30]

>Короче, я все именую.

Аналогично - кнопки, панели, едиты, сплиттеры, боксы/листы/memo/гриды и пр.
Исключением могут быть лейблы, да и то среди них присутствуют именованные.


 
Германн ©   (2013-09-03 00:21) [31]


> Кщд   (02.09.13 22:01) [19]
>
> >Anatoly Podgoretsky ©   (02.09.13 21:58) [18]
> Анатолий, Вы же из Новосибирска, а не из Эстонии, в самом-
> то деле
>

Не ну.
АП действительно очень неординарная личность в программистском рунете. Не зря его самолично крестил будущий Патриарх Всея Руси! (Грешным делом даже думается уж не потому ли Алексия избрали патриархом, что именно он крестил АП :). Но вот в то что Алексий Второй специально ради этого ездил в Новосибирск как-то не очень верится. :)


 
DVM ©   (2013-09-03 00:24) [32]

Так Cnpack же обучить можно давать префиксы какие хочется, потом всю конфигурацию сохранить. А чтоб он пунктам меню и кнопками сам давал нормальные имена надо экшены называть нормально, например для экшена actOpenFile будет создан пункт меню mniOpenFile.


 
Jeer ©   (2013-09-03 00:31) [33]

>Германн ©   (03.09.13 00:21) [31]

Не оскорбляй чувства неверующих, иначе придется им свои поправки в УК вносить.


 
Германн ©   (2013-09-03 00:35) [34]


> DVM ©   (03.09.13 00:24) [32]
>
> Так Cnpack же обучить можно давать префиксы какие хочется

А. Так я ещё должен обучать китайцев?
:)


 
Юрий Зотов ©   (2013-09-03 01:19) [35]

> Степан Потапов

Хороший развод. Ths.


 
brother ©   (2013-09-03 03:57) [36]

Степа, Мы на такие разводки не ведемся...


 
Алканавт расправил плечи   (2013-09-03 08:14) [37]


> Ну сплитеров как правило не больше двух
Ну оно как… http://lurkmore.so/images/1/13/Keepitsimplestupid.jpg


> то как же у них хватает терпения при создании интерфейса
> на прописывание хинтов, горячих клавиш, клавиатурных сокращений,
>  идентификаторов справочной системы, выставление размеров
> контролов и внешнего вида окон согласно руководствам
Эти пункты тестировщики проверят. И задачу вернут. А названия лейблов не проверят. :-)


 
Кщд   (2013-09-03 10:30) [38]

>Германн ©   (03.09.13 00:21) [31]
бес попутал


 
Ega23 ©   (2013-09-03 10:41) [39]


> Но вот в то что Алексий Второй специально ради этого ездил
> в Новосибирск как-то не очень верится. :)


Алексий, ващета, на повышение уже лет 5 назад ушёл.


 
Anatoly Podgoretsky ©   (2013-09-03 11:01) [40]

Но я то пока еще нет.


 
robt5   (2013-09-03 21:05) [41]


> Edit1, Edit2, ..., Edit1534?создаёте нечитаемый код?

при таком количестве контролов - программа уже УГ
при коде любой читаемости


 
DVM ©   (2013-09-03 22:21) [42]

Такое количество контролов может образоваться легко на форме настроек программы когда много разделов и расположены они на разных вкладках пейджконтрола какого нибудь. Такое повсеместно.


 
Алканавт расправил плечи   (2013-09-03 22:41) [43]

господи, которого, в сущности, нет, упаси меня от программ, у которых на форме настройки не менее 1534 эдитов… %-)


 
DVM ©   (2013-09-03 22:55) [44]

Да таких программ полно. Ну может не эдитов так чекбоксов. Сотни их.


 
MonoLife ©   (2013-09-04 05:58) [45]

o! холивар..
переименовываю, конечно: вместо button1, NajmiPosmotriChegoPoluchitsya:)


 
Алканавт расправил плечи   (2013-09-04 07:46) [46]


> DVM ©   (03.09.13 22:55) [44]
Про сотню чекбоксов — не спорю, но полторы тыщи эдитов — это таки комично :-)
Думаю, [41] также попросту проигнорировал метафору и решил воспринять слова буквально.


 
robt5   (2013-09-04 10:52) [47]


>  и решил воспринять слова буквально

нет они реально существуют :)

>  на форме настроек программы

это от скудоумия
достаточно по 1 экземпляру дерева, чекбокса, едита, комбобокса и тракбара + небольшой алго для их дружбы

> Про сотню чекбоксов — не спорю

а зря, достаточно одного ТCheckListBox


 
DVM ©   (2013-09-04 11:11) [48]


> robt5   (04.09.13 10:52) [47]


> это от скудоумия
> достаточно по 1 экземпляру дерева, чекбокса, едита, комбобокса
> и тракбара + небольшой алго для их дружбы

И юзабельность такого будет равна 0 для пользователя. В дереве будет много-много узлов, плюс если параметры зависят друг от друга надо будет метаться между узлами дерева чтобы все настроить. Шило на мыло заменил, чекбоксы на узлы дерева. Ерунду предлагаешь. Насчет скудоумия ты тоже погорячился, это общепринятый и главное привычный пользователю подход в организации диалога настроек.


 
DVM ©   (2013-09-04 11:12) [49]


> robt5   (04.09.13 10:52) [47]


> а зря, достаточно одного ТCheckListBox

Достаточно одного мемо на все настройки, и в мемо настройки вбиваются ручками в xml формате, не? Ты о пользователях подумай и о внешнем виде такого диалога.


 
Кщд   (2013-09-04 11:19) [50]

>robt5   (03.09.13 21:05) [41]
это в Вас говорит узость мышления, недостаток опыта и бесфантазийное буквоедство
пример простой: кредитная анкета
Edit"ов, конечно, не 1500, но общее число контролов ввода приближается
некоторые контролы/блоки контролов активируются/показываются при заполнении других, некоторые - наоборот - деактивируются/скрываются
и мне жутко представить код, который реализует данную логику, оперируя автосгенерированными именами объектов


 
[ВладОшин] ©   (2013-09-04 11:19) [51]


> Степан Потапов

Женись. Будут у тебя Сын1 Степанович, Сын2 Степанович :)


 
DVM ©   (2013-09-04 11:32) [52]


> [ВладОшин] ©   (04.09.13 11:19) [51]

+1 :)


 
Anatoly Podgoretsky ©   (2013-09-04 11:42) [53]


> И юзабельность такого будет равна 0 для пользователя. В
> дереве будет много-много узлов, плюс если параметры зависят
> друг от друга надо будет метаться между узлами дерева чтобы
> все настроить. Шило на мыло заменил, чекбоксы на узлы дерева.

Все равно это шаг вперед, будет значительно меньшее количество узлов дерева, чем чек боксов. Хотя деревья мне очень не нравятся.


 
DVM ©   (2013-09-04 11:57) [54]


> Anatoly Podgoretsky ©   (04.09.13 11:42) [53]


> будет значительно меньшее количество узлов дерева

С чего вдруг их будет меньше? Их количество увеличится ровно во столько раз, во сколько раз мы уменьшим число чекбоксов.


 
Anatoly Podgoretsky ©   (2013-09-04 14:39) [55]

> DVM  (04.09.2013 11:57:54)  [54]

С того, что в узле будет гораздо больше элементов, ну конечно если не быть дураком. Все равно не нравится.


 
DVM ©   (2013-09-04 15:57) [56]


> Anatoly Podgoretsky ©   (04.09.13 14:39) [55]


> С того, что в узле будет гораздо больше элементов

У нас и до этого в узле было много элементов, а предлагается оставить по одному.


 
Anatoly Podgoretsky ©   (2013-09-04 16:08) [57]

> DVM  (04.09.2013 15:57:56)  [56]

По одному
Какой тогда смысл в дереве, это осина какая то, на ней только вешаться.


 
DVM ©   (2013-09-04 16:10) [58]


> Anatoly Podgoretsky ©   (04.09.13 16:08) [57]


> Какой тогда смысл в дереве

Вот и мне непонятно какой смысл, или я не понял фразы:


> это от скудоумия
> достаточно по 1 экземпляру дерева, чекбокса, едита, комбобокса
> и тракбара + небольшой алго для их дружбы


 
Anatoly Podgoretsky ©   (2013-09-04 18:20) [59]

Cмысл появится если в узле дерева будет больше одного параметра. Но все равно неудобно. Применение ограниченное.


 
Rouse_ ©   (2013-09-04 18:39) [60]

Если к объекту нет обращения из кода, его название программиста не должно интересовать. Впрочем если не лень тратить свое время и деньги работодателя, можно переименовывать все подряд.


 
Rouse_ ©   (2013-09-04 18:42) [61]

ЗЫ: а так вообще у нас практически все контролы завязаны на Action-ы и необходимость в переименовании отсутствует т.к. работа идет именно с Action а не с контролом.


 
robt5   (2013-09-04 18:47) [62]


> DVM ©   (04.09.13 16:10) [58]

дерево - это главные ветки по типам опций (как вкладки по старинке)
+ вложенные ветки - подтипы, это то что обычно объединяют на конкретной вкладке в группбокс
при выделении подтипа заполняется список (название-значение) всех опций в группе
при наведении\клике мыши по элементу списка вместо его значения появляется соответствующий по типу элемент ввода (чекбокс, эдит, тракбар и тд)

так понятней ???

в итоге имеем всего около 8 постоянных контролов для отображения опций любой хитровымученности


 
Дмитрий Белькевич   (2013-09-04 19:25) [63]

Контролов на главной форме - штук 300 - 350 где-то. Компоненты не переименовываю. Все остальные переменные - называю максимально понятно, бывает, со временем, переименовываю. К компонентам из кода почти не лезу.
Мне так удобно. Думаю, каждый сам решает, как удобно, если кто-то не навязывает.


 
Ega23 ©   (2013-09-04 19:40) [64]


> Контролов на главной форме - штук 300 - 350 где-то.

кошмар.


 
Rouse_ ©   (2013-09-04 19:46) [65]


> Ega23 ©   (04.09.13 19:40) [64]
> кошмар.

Легыч, открой проект, у нас их более тыщи, чего тебя так удивило? :)


 
Ega23 ©   (2013-09-04 19:58) [66]


> Легыч, открой проект, у нас их более тыщи, чего тебя так
> удивило? :)


На главной форме? Контролов?


 
Германн ©   (2013-09-04 20:08) [67]

Он их с пикселями наверно спутал.


 
Rouse_ ©   (2013-09-04 20:12) [68]


> Ega23 ©   (04.09.13 19:58) [66]
> На главной форме? Контролов?

Вроде того...


 
Rouse_ ©   (2013-09-04 20:16) [69]

Хотя я ошибся - всего 565 штук, но это только на главной форме без учета фреймов :)


 
Ega23 ©   (2013-09-04 20:21) [70]


> Хотя я ошибся - всего 565 штук, но это только на главной
> форме без учета фреймов :)


Это где там столько-то? На риббоне?


 
Rouse_ ©   (2013-09-04 20:27) [71]


> Ega23 ©   (04.09.13 20:21) [70]
> Это где там столько-то? На риббоне?

в DFM


 
Ega23 ©   (2013-09-04 20:37) [72]


> в DFM


Да ладно? Ну не верю я. Компонентов - ещё могу поверить. Но контролов...
Завтра поглядим.


 
Rouse_ ©   (2013-09-04 20:44) [73]


> Ega23 ©   (04.09.13 20:37) [72]
> Да ладно? Ну не верю я.

Знач статью по DХ так и не прочитал.
Печаль...
Много открытий предстоит значит завтра...


 
DVM ©   (2013-09-04 21:57) [74]

Каждую кнопку в тулбаре и пункт меню за отдельный контроль посчитали что ли?


 
Ega23 ©   (2013-09-04 22:42) [75]


> Каждую кнопку в тулбаре и пункт меню за отдельный контроль
> посчитали что ли?


Под рукой dx нету, но, походу, ItemLink это тоже контрол. А тогда - да, тогда их дофига.


 
Германн ©   (2013-09-05 01:06) [76]


> Знач статью по DХ

Это что? Ребята из DX умеют готовить только контролы?
Даже интересно стало попробовать поставить их триал, Вот только есть сомнения, что после пробы смогу его удалить без "жертв и повреждений".



Страницы: 1 2 вся ветка

Форум: "Прочее";
Текущий архив: 2014.02.16;
Скачать: [xml.tar.bz2];

Наверх





Память: 0.64 MB
Время: 0.004 c
2-1366331872
novai
2013-04-19 04:37
2014.02.16
перекрыть WindowProc


15-1377682520
Наталья
2013-08-28 13:35
2014.02.16
Подскажите новичку.


2-1366640331
delphistorm
2013-04-22 18:18
2014.02.16
ошибка при загрузке xml в TXMLDocument


2-1365333369
quart
2013-04-07 15:16
2014.02.16
gdi+ error


2-1366897985
HDC
2013-04-25 17:53
2014.02.16
отрисовка текста через TCanvas





Afrikaans Albanian Arabic Armenian Azerbaijani Basque Belarusian Bulgarian Catalan Chinese (Simplified) Chinese (Traditional) Croatian Czech Danish Dutch English Estonian Filipino Finnish French
Galician Georgian German Greek Haitian Creole Hebrew Hindi Hungarian Icelandic Indonesian Irish Italian Japanese Korean Latvian Lithuanian Macedonian Malay Maltese Norwegian
Persian Polish Portuguese Romanian Russian Serbian Slovak Slovenian Spanish Swahili Swedish Thai Turkish Ukrainian Urdu Vietnamese Welsh Yiddish Bengali Bosnian
Cebuano Esperanto Gujarati Hausa Hmong Igbo Javanese Kannada Khmer Lao Latin Maori Marathi Mongolian Nepali Punjabi Somali Tamil Telugu Yoruba
Zulu
Английский Французский Немецкий Итальянский Португальский Русский Испанский