Имя: Пароль:
IT
 
КривоЖ*** или Качество работы программиста
🠗 (Fish 08.09.2014 14:21)
0 karpik666
 
08.09.14
14:14
Добрый день, форумчане. Вот хочется узнать, все ли следят за качеством своего кода, за грамотностью написания комментариев и переменных, и просто за чистотой кода? Просто разбирал конфигурацию одной компании, и просто был в шоке, как можно было так издеваться над кодом. А что самое страшное этот продукт продается, и автоматизирует обширную область бизнеса. По приведенной ссылке можно ознакомиться с одни из модулей этих горе-программистов. https://yadi.sk/i/O0ZRZJ6sbE9ZK
Интересно ваше мнение: имеют ли право такие продукты вообще выходить на рынок, и сколько должна быть "красная цена" такого творчества?
p.s. Изменил тему сообщения.
1 PR
 
08.09.14
14:16
(0) А что поменялось-то по сравнению с прошлой фекаловеткой?
2 РенеДекарт
 
08.09.14
14:17
(0) скинул бы всю конфу.
Хоть какая-то польза...
3 zulu_mix
 
08.09.14
14:18
а что там такого криминального? а то лень читать
4 ДенисЧ
 
08.09.14
14:18
кусок запроса от уважаемой фирмы...
ЕСЛИ Тип = ""По умолчанию""
И Флаг = ИСТИНА
....
5 Злопчинский
 
08.09.14
14:18
(0) да пофиг. если времени дофига и работы особо нет - пишем красиво. если цейтнот - пишем чтобы работало.
.
красота кода и красивая архитектура - ценна только для поддержки. для работодателя - в целом пофиг.
.
где-то в конторах есть может долгосрочные палын развития, поддержки итд. хз... может быть...
большинстов мелких контор работает по принципу "здесь и сейчас". отсюда и пляшем.
.
имхо конечно
6 РенеДекарт
 
08.09.14
14:19
(0) а что не нравится? Стиль 1С? ну да, это 1С..
7 Скай
 
08.09.14
14:19
Правда считаешь, что кто-то будет ковыряться в 182 страницах копипасты? Это во-первых.
А во-вторых, ты уже написал что-то свое, более-менее крупное, отвечающее приведенным тобой требованиям?
В-третьих, любой продукт имеет право выйти на рынок. Рынок сам определит, жить ему или умереть.
8 Эльниньо
 
08.09.14
14:19
Всех одинэсников расстрелять, предварительно повесив.
Набрать новых.
9 РенеДекарт
 
08.09.14
14:20
(4)>ЕСЛИ Тип = ""По умолчанию""
- ну так действительно, если такой тип? Чем лучше "Если ФлагПоУмолчанию"? Разницы для 1С никакой. Соблюдай только "Строка" или "булево".
10 zulu_mix
 
08.09.14
14:21
(4) мне от одной уважаемой фирмы наваяли один раз:

если склад.наименование = "имярек1" тогда
текточка = справочники.подразделения.найтипонаименованию("имярек1");
иначеесли склад.наименование = "имярек2" тогда
текточка = справочники.подразделения.найтипонаименованию("имярек2");
....
и так васэмнадцать раз
11 fmrlex
 
08.09.14
14:22
Перфекционисты в треде. Всем заправить рубашки и снять носки из-под сандалий.
12 fmrlex
 
08.09.14
14:22
(0) Не завидуй тому, что они продают такое поделие. Поделись своим поделием, продаваемым.
13 Легат
 
08.09.14
14:22
Индусский код )
14 _fvadim
 
08.09.14
14:24
вспомнилось...

в аду для перфекционистов
ни серы нету ни огня
и лишь слегка несимметрично
стоят щербатые котлы
15 GreatOne
 
08.09.14
14:24
Если кто-то ругает кого-то, значит этот кто-то сам не без греха. А если этот кто-то еще и сильно возбухает, то по языку дать и пинка за одно. (0) Выкладывай свой код, будем ржать.

p.s. У меня сегодня вообще полная ж*па, в выходные перевел базу на SQL, утром шквал звонков(а я еще в кровати), нихрена не работает, реализации не проводит, в 1С у многих не заходит. Выяснилось, что прошлый прогер писал свой модуль на сервере, а там просто все пестрит Предупреждение(), КодВозвратаДиалога и прочее, что естественно на сервере перестало работать. После этого он му**к? Нет, он 4 года поддерживал работоспособность базы и полностью поставил им учет. А косяки и былая недальновидность у всех бывают.

В общем, (0), ты сам скорее всего не лучше.
16 Bigbro
 
08.09.14
14:25
если приходится такое поддерживать - соболезную..
17 Рэйв
 
08.09.14
14:25
(0)Как говорил мой шеф в моем первом франче -

"Пользователю совершенно похрен какой супер крутой и красивый код ты написал. Ему важно чтобы на форме было красиво и были рюшечки. Он платит за визуальный результат."
18 РенеДекарт
 
08.09.14
14:26
(15) а ты что, даже не попробовал сам запустить?
19 Рэйв
 
08.09.14
14:26
хотя за кодом все таки стараюсь следить. Но без фанатизма:-)
20 GreatOne
 
08.09.14
14:27
(18) пробовал, но реализацию не проводил, а там модуль только для нее. Почти все протестили, а реализацию нет. Да я еще вчера пробно для одной точки поставил, думаю малой кровью тест сделаю. Они молчали, значит все ок. А сегодня узнали, что у них ничего не работало, но они просто не сообщили...
21 Холст
 
08.09.14
14:27
сюда напрашивается притча про программиста Васю и Петю
22 Зойч
 
08.09.14
14:27
Что в (0) не так???
23 lodger
 
08.09.14
14:28
(15) а собственно самому попинать из-под разных ролей?
в новой платформе так вообще есть такая страшная менюшка - Реfuckторинг :)
запускаешь поиск по всей конфе, и не надо ждать пока юзер сам воткнется в ошибку.
24 РенеДекарт
 
08.09.14
14:28
(17) россиянскому пользователю вообще все похре. Даже рюшечки на форме. Ему нужна кнопка "Сделать все за месяц".
Он должен нажимать её раз в месяц, и приходить за зарплатой.
25 GreatOne
 
08.09.14
14:28
+ к (20), он для каждого пользователя руками среду прописал с запуском лохматой версии 1С, котороя конечно не совпадала с версией сервера 1С, многих просто не пускало.
26 GreatOne
 
08.09.14
14:29
(23) увы, познается в бою, я далек от опытного программиста.
27 РенеДекарт
 
08.09.14
14:30
(23) ничего толком этот рефакторинг не делает.
Показать, что иснтрукция не работает на клиенте - поставь "тонкий клиент", и запусти проверку синтаксиса.
28 _fvadim
 
08.09.14
14:30
щас занимаюсь проектом, достался по наследству, по началу тоже пытался привести его в приличный вид - времени уходит прорва, сроки стали поджимать, в итоге - пишу костыли и велосипеды, местами так и хочется навтыкать комментариев вроде "да, я знаю что это г0внокод и так делать нельзя"
29 PR
 
08.09.14
14:30
(23) Что, у тебя именно так и написано, Реfuckторинг?
30 РенеДекарт
 
08.09.14
14:32
(28)> "да, я знаю что это г0внокод и так делать нельзя"
- еще не дорос, значит ))
я так и пишу, чтобы потом самому стыдно не было.
31 РенеДекарт
 
08.09.14
14:33
(29) 1С стебется в демо-релизах....
32 MrStomak
 
08.09.14
14:34
(0) В коде есть комментарии, описаны для чего переменные, названия переменных читаемые, в чём там издевательство над кодом?
Мне, гуамнокодеру, не понять, в общем.
33 КонецЦикла
 
08.09.14
14:35
(0) Не видел ты жести
Иди работай и не гони пургу
34 РенеДекарт
 
08.09.14
14:41
А еще есть обфускация кода.
И все гонокодеры стремятся максимально такили иначе свой код "обфурсить". И 1С усиленно их поощряет в этом гонокодировании.
35 Kamas
 
08.09.14
14:44
(30) я обычно просто метку для себя ставлю типо //поправить. Потом когда время есть пробежался поправил, жаль, что времени часто нет((
36 Килограмм
 
08.09.14
14:44
чуть полистал код исходный, но скучно быстро стало
37 Kamas
 
08.09.14
14:45
(34) я не обфусцирую у меня и так местами хрен поймеш))
38 hovnokoder
 
08.09.14
14:45
(0) главное чтобы работало правильно, а что там написано в коде - это дело десятое...
И если есть комментарии для чего нужен тот или иной кусок кода - то это вообще замечательно.
39 wms
 
08.09.14
14:50
недавно отчет встречал на 8000строк.
начал разбираться там 2 вида отчета с группировкой и без.
ит типа если флаг группировки=истина тогда один кусок
иначе почти тот же самый только в запросах нет группировки.
работает не трожь.

прада когда просили 3-й вид отчета сделать то код отчета сократил более чем в 2-раза
40 Albaness
 
08.09.14
14:53
(0) громче всех кричит " Держи вора" сам воришка)))
41 karpik666
 
08.09.14
16:49
Ничего себе, сходил за хлебушком. Вернулся домой, а тут=) много чего о себе узнал=) ну во всяком случае я не говорил, что великий мастер писать код, во-вторых я просто спросил мнение окружающих о таком. Я сейчас не работаю во франчайзи, может поэтому у меня есть время отлаживать и рефакторить свой код. Просто было интересно, как разработчики сами умудряются отлаживать свой код, когда у них такое происходит=) Ну а на самом деле, большое спасибо=)
42 MrStomak
 
08.09.14
17:27
(41) Озвучь, что ты в приведённом примере считаешь издевательством над кодом и что породило в тебе столько бурлений копромасс?
Чтобы обнаруживать ошибки, программист должен иметь ум, которому доставляет удовольствие находить изъяны там, где, казалось, царят красота и совершенство. Фредерик Брукс-младший