|
v7: Рефакторинг. Смена названия справочника | ☑ | ||
---|---|---|---|---|
0
Джордж1
03.01.16
✎
19:57
|
Есть старая самописная конфигурация.
В ней есть справочник Покупатели, который по факту стал справочником Контрагенты. Как правильно это сделать? Только глобальный поиск по конфигурации? А что бы в рабочей базе справочник не задвоился - надо сначала в рабочей конфигурации его переименовать? Если что 7-ка. |
|||
1
ДенисЧ
03.01.16
✎
19:58
|
Сначала нужно сходить в аптеку за вазелином...
Потом начинать глобальный поиск. |
|||
2
oslokot
03.01.16
✎
20:10
|
(0) Может оставить имя как есть, а заменить синоним?
|
|||
3
HawkEye
03.01.16
✎
20:14
|
(0) зачем?
|
|||
4
pessok
03.01.16
✎
20:24
|
(2) имхо вообще ничего не надо трогать
|
|||
5
Zamestas
03.01.16
✎
20:34
|
(0) Если "сначала в рабочей конфигурации его переименовать", то рабочая конфигурация может стать не рабочей.
|
|||
6
Джордж1
03.01.16
✎
20:35
|
Давно в 2008 была написана конфигурация до сих пор нормально работает.
За это время время у заказчика постепенно появлялись хотелки, а в конфигурации доработки-костыли. Сам уже нормально в коде разобраться, почему так сделаны те или иные вещи. Хочется попробовать сделать (не сразу, постепенно) конфигурацию более красивой и понятной |
|||
7
Джордж1
03.01.16
✎
20:35
|
(5)переименовать и сразу обновить
|
|||
8
pessok
03.01.16
✎
20:36
|
(6) переноси на БСП же :D
|
|||
9
pessok
03.01.16
✎
20:39
|
а если серьезно, то тебе еще надо будет все обращения ж переписать, а не просто обновить
|
|||
10
Джордж1
03.01.16
✎
20:39
|
(9)дак я на тестовой базе все перепишу и проверю
|
|||
11
pessok
03.01.16
✎
20:40
|
(10) сколько уже граблей было об головы разбито такими фразами))
|
|||
12
Джордж1
03.01.16
✎
20:41
|
(11)так разве есть другие варинты
|
|||
13
Garykom
гуру
03.01.16
✎
20:41
|
Если вазелин качественный и дофига лишнего времени то вперед.
Учесть что сразу все внешние обработки/отчеты и прочие обмены если есть тоже пилить |
|||
14
Garykom
гуру
03.01.16
✎
20:42
|
(13)+ т.е. сначала поищи и оцени объем работы, потом набросай план который придется курить
|
|||
15
Джордж1
03.01.16
✎
20:44
|
(13)Внешняя обработка всего одна, конфигурация не шибко большая.
Бесит уж больно когда в коде Покупателя, а на самом деле Поставщик. Лучше уж Контрагент будет, как в типовых |
|||
16
Джордж1
03.01.16
✎
20:45
|
(15)Просто с каждой доделкой вижу как все больше усугубляется ситуация и насколько дольше приходится разбираться вспоминать почему же это я в коде так написал
|
|||
17
pessok
03.01.16
✎
20:46
|
и все же переводи на БСП :) больше нетленок на восьмерке! :D
|
|||
18
HeKrendel
03.01.16
✎
20:47
|
Я конечно не программист, но в чем проблема перемеиновать вывод наименования справочника, без программного переименовывания?
|
|||
19
pessok
03.01.16
✎
20:48
|
(18) задумано не для того у него. задумано, чтобы в коде не думать - сжуя ли в переменной Покупатель по факту продавец
|
|||
20
Джордж1
03.01.16
✎
20:48
|
(17)Во первых с 8-кой я на Вы, во вторых - сейчас все замечательно и быстро работает - с 8-кой это сомнительно, в третьих, скажем так заказчик очень консервативен - боиться что-то менять
(18)Я как бы не пользователю работу облегчаю, а программисту |
|||
21
Джордж1
03.01.16
✎
20:49
|
(19)На формочках то написано Продавец
|
|||
22
Джордж1
03.01.16
✎
20:55
|
Если я правильно понял термин Рефакторинг - это переделка внутренностей программы, без изменения интерфейса
|
|||
23
HawkEye
03.01.16
✎
21:20
|
(22) резюме: бесполезная работа, с вполне реальными рисками... за чей счет банкет?
|
|||
24
Джордж1
03.01.16
✎
21:31
|
Мне же эту программу в дальнейшем и сопровождать.
Заказчик платит достойную постоянную плату |
|||
25
Feanor
03.01.16
✎
21:41
|
(24) а Заказчик тебе точно спасибо скажет за привнесенные глюки в связи с таким рефакторингом? Лучше сделай для Заказчика действительно что-то полезное :)
|
|||
26
HawkEye
03.01.16
✎
21:47
|
(24) я правильно понимаю, что если тебе понадобиться новый ноут для сопровождения этой нетленки, заказчик оплатит тебе новый ноут? )
не понимаю, какая разница как в метаданных называется справочник? да хоть "А", абстрагируйся... |
|||
27
Джордж1
03.01.16
✎
21:55
|
(26)Название справочника не единственная провблема
|
|||
28
Маркусс
04.01.16
✎
07:42
|
(24) смени синоним и выкати счёт часов на 100 за это
|
|||
29
Маркусс
04.01.16
✎
07:45
|
(21) за формочки ещё часов 50
|
|||
30
arsik
гуру
04.01.16
✎
09:00
|
я за периодический рефакторинг самописных проектов.
Сейчас работаю к конторе, самописная конфигурация (пилится группой программистов). Там больше всего времени уходит на оценку - как отразится мое изменение на других подсистемах. Предлагал сделать глубокий рефакторинг, но руководству важнее пока пилить функционал. |
|||
31
Мимохожий Однако
04.01.16
✎
09:30
|
(16)Для этого придумали комментарии, справки и документацию. Добавь в свои доработки нужные и будет счастье.
Если дело только в отображении на формочках и если об этом просили пользователи, то есть смысл что-то менять. Если код работает, то лучше заняться чем-нибудь более существенным. Принцип "Не навреди" ещё никто не отменял. (27)Перед боем обычно составляют план. В данном случае я вижу только раздражение без спокойного анализа конфигурации и потребностей заказчика. |
|||
32
arsik
гуру
04.01.16
✎
10:11
|
(31) "Лучше день потратить, потом за 5 минут долететь"
|
|||
33
Джордж1
04.01.16
✎
10:11
|
(31)
1.Этож каждую строчку комментировать надо. Где ты такое видел 2.План прикину. Еще раз повторюсь - заказчика все устраивает. Не устраивает меня - т.к. каждый раз запутываешься в груде костылей |
|||
34
Масянька
04.01.16
✎
10:15
|
(33) Старая эсная поговорка - "Работает - не трожь!" (С)
|
|||
35
arsik
гуру
04.01.16
✎
10:32
|
(34) это поговорка времен 7ки. Ну или ох-уэнная отмазка от работы.
|
|||
36
Мимохожий Однако
04.01.16
✎
10:33
|
(33)Не надо каждую строчку. Перед фрагментом пишешь всё, что думаешь о предстоящем алгоритме, а по каждой строке...Это сюрреализм какой-то
|
|||
37
Масянька
04.01.16
✎
10:34
|
(35) Дык, речь о ней, родимой... А ты - иди, посмотри на 8.4 :)))))))))
|
|||
38
Мимохожий Однако
04.01.16
✎
10:34
|
(35)Работа ради работы...Это онанизм
|
|||
39
arsik
гуру
04.01.16
✎
10:49
|
(38) Глупость какую то сказали. Выше привели причины, ради чего рефакторинг нужен.
|
|||
40
pessok
04.01.16
✎
10:53
|
(39) ну вот большинство считает, что можно просто поменять Синоним и, при необходимости, перерисовать рожи
|
|||
41
Мимохожий Однако
04.01.16
✎
10:53
|
(39)Называй как хочешь, но если заказчик не выдвигает пожеланий, то менять чужой код без необходимости нет смысла. А вот если вагон свободного времени и хочется...то см. (38). Рефакторинг ради рефакторинга. Оно так и есть, то самое....
|
|||
42
pessok
04.01.16
✎
10:54
|
(40) собсно, я тоже пртдерживаюсь. единственный раз переписывал свою нетленку (потому что адское гуано, посмотрите мои первые темы тут) и то - сразу на УФ
|
|||
43
Масянька
04.01.16
✎
10:55
|
(40) Если клиент (для кого писалась программа) хочет.
|
|||
44
arsik
гуру
04.01.16
✎
11:39
|
(41) Заказчик может не знать о такого рода проблемах. Заказчику важно решить задачу А за время Б. После рефакторинга задачу А можно решить за время Б/2.
Выше описано, что проблемы есть и решить их можно рефакторингом. Откуда "рефакторинг ради рефакторинга"? |
|||
45
Джордж1
04.01.16
✎
11:50
|
(40)Синоним оказывается уже давно поменян был
(41)код не чужой, он мой // Пока суть да дело сижу, ваяю. Без проблем конечно не обойдешься, но костылей на глазах меньше становиться. // Если что то подсистема работы с контрагентами не критичная. Это розничная сеть, у которой есть продажи по безналу немного |
|||
46
Aprobator
04.01.16
✎
12:47
|
Поменять представление справочника при показе пользователю и не заниматься фигней.
|
|||
47
arsik
гуру
04.01.16
✎
13:40
|
(46) Я так раньше тоже думал, пока не столкнулся с реальными трудностями. Сейчас основная масса времени тратится на разбор того, на что повлияют изменения. И это от того, что система не рефакторилась. И с 2008 года там столько костылей, что каждый релиз - это боль. :)
|
|||
48
Мимохожий Однако
04.01.16
✎
14:36
|
(44)Не мной замечено, что если программиста не остановить, он будет улучшать код до бесконечности. Здравый смысл подсказывает, что пока не столкнулся с плохими костылями, их не надо трогать и "рефакторить". Особенно, если Заказчик об этом не просит.
(47)То, что основное время при программирование тратится на анализ вполне нормальная ситуация. |
|||
49
Масянька
04.01.16
✎
14:44
|
(47) Поставь себя на место пользователя - заказчика программы: заказал, написали, поставили, заплатил, сидишь работаешь, никого не трогаешь... Приходит чел (не могу назвать программистом): Нужно кое-что поправить, кое-что переделать. Ты: Меня все устраивает, все работает... Чел: Вы просто не понимаете - новые технологии, бла-бла-бла.
|
|||
50
Mikeware
04.01.16
✎
14:46
|
(49) иногда приходится объяснять. Демонстрировать. например, демонстрировать "подбор по вхождению", или кубы внутри 1с, или драг-н-дроп.
|
|||
51
Джордж1
04.01.16
✎
15:51
|
Еще раз, я делаю это для себя, без дополнительных денег. - просто для удобства дальнейшей работы.
Попутно кое-что аккуратно правлю и в интерфейсе. Заказчик не всегда видит что можно что-то в лучшую сторону поменять, у меня же есть большой опыт работы именно с этим заказчиком и я могу с большой вероятностью делать изменения и быть уверенным что ему это понравиться. |
Форум | Правила | Описание | Объявления | Секции | Поиск | Книга знаний | Вики-миста |