|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Shocker.ProА будучи в команде изволь выполнять требования к качеству кода , а не только к соответствию ТЗ.Я не против, если эти требования чётко описаны и ТЗ на них ссылается. Тогда они должны исполняться и проверяться на этапе сдачи в эксплуатацию. Но тут речь идёт о каких-то мифических "лучших традициях", которые из года в год меняются с точностью до наоборот. Не нужно это путать. А если чётко описанных требований нет, как это часто бывает, то нужно писать в том же стиле, в каком в данном проекте решены аналогичные задачи, независимо от того, нравится тебе такой стиль написания или нет. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:02 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
skyANAАлексей Кпропущено... Существует единственный критерий правильности программы: программа должна соответствовать техническому заданию. Всё остальное, это субъективное мнение критиков. И со временем система обрастает техническим долгом и реализация новых требований проходит всё с большим количеством мата, но программа продолжает соответсвовать техническому заданию Это уже вопрос к самим требованиям, которые должны быть заложены ещё до написания программы. Другими словами, не нужно перекладывать ответственность с архитекторов на рядовых программистов. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:06 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КhVosttОн просто не знает, что критерии для оценки качества кода существуют.Существует единственный критерий правильности программы: программа должна соответствовать техническому заданию. Всё остальное, это субъективное мнение критиков. Это вопрос из разряда цвета фломастеров. Это не про инженерную дисциплину, уж прости. Какие критики? При чём тут соответствие ТЗ и качества кода? Да, веришь нет, но можно такой говнокод написать, что даже у миддла волосы дыбом встанут, но формально работа программы будет соответствовать ТЗ. Я правильно понимаю, согласно твоим идеям, некоторым водителям можно бухать за рулём, если им кажется, что они всё равно нормально водят, и довезут тебя куда надо за требуемое время? В общем, странный и неуместный подход. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:13 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Shocker.ProНу и будет рекомендация тимлида тебя уволить к чертовой матери, не более. Хвост же говорит - хочешь писать говнокод, пиши говнокод. Сам с собой. А будучи в команде изволь выполнять требования к качеству кода, а не только к соответствию ТЗ. Алексей К очень хороший специалист. Но он не командный разработчик, у него либо не было такого опыта вообще, либо он очень мал. Поэтому такие взгляды и такое отношение. Как хочу, так пишу. Творчество, понимаешь. В принципе, для single разработчика, такой подход никому не мешает. Особенно, если он собирается работать там до пенсии, а после хоть трава не расти ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:16 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
hVosttЯ правильно понимаю, согласно твоим идеям, некоторым водителям можно бухать за рулём, если им кажется, что они всё равно нормально водят, и довезут тебя куда надо за требуемое время?Нет, не правильно понимаешь. Есть закон, запрещающий "бухать" за рулём. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:17 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КЯ не против, если эти требования чётко описаны и ТЗ на них ссылается. Тогда они должны исполняться и проверяться на этапе сдачи в эксплуатацию. Это вопрос именно разработки, профессионализма, культуры, профессиональной этики. Где-то в ТЗ прописано, чтобы ты не справлял нужду за рабочим столом, нет? Но ты же так не делаешь? Хотя если ты будешь работать исключительно один, и это не будет никому мешать, почему нет? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:17 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КА если чётко описанных требований нет, как это часто бывает, то нужно писать в том же стиле, в каком в данном проекте решены аналогичные задачи, независимо от того, нравится тебе такой стиль написания или нет. Даже если аналогичная задача решена 12 лет назад на ASP.NET WebForms таким образом, что тестами ни фига не покрыто и с трудом разберёшься где что? А вышло так потому, что 12 лет назад надо было быстро быстро на рынок выходить, главное чтобы работало, а на качество... А качество денег принесёт? Пока ты будешь требования к нему расписывать, проектировать, вылизывать, то рынок тю-тю, займут. Извини, но я лучше предожу своё решение на техническое ревью, основанное на моём понимании принципов проектирования и опыте, чем буду ждать "чётко описанных требований". ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:18 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КhVosttЯ правильно понимаю, согласно твоим идеям, некоторым водителям можно бухать за рулём, если им кажется, что они всё равно нормально водят, и довезут тебя куда надо за требуемое время?Нет, не правильно понимаешь. Есть закон, запрещающий "бухать" за рулём. Ну так во многих компаниях точно также приняты стайл гайды, и существует практика контроля за качеством кода. Но в ТЗ это не приписывается. Будешь писать говногод, будет тяжелее, дороже и дольше его развивать и сопровождать. Всё просто. При чём тут ТЗ, вообще?? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:19 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КskyANAпропущено... И со временем система обрастает техническим долгом и реализация новых требований проходит всё с большим количеством мата, но программа продолжает соответсвовать техническому заданию Это уже вопрос к самим требованиям, которые должны быть заложены ещё до написания программы. Другими словами, не нужно перекладывать ответственность с архитекторов на рядовых программистов. А пример привести можешь того, в каком виде и куда должны быть заложены требования к качеству кода? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:20 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
hVosttГде-то в ТЗ прописано, чтобы ты не справлял нужду за рабочим столом, нет? Это тоже будет считаться административным правонарушением. Попробуешь спросить ещё раз? :-) ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:20 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
skyANAА качество денег принесёт? Пока ты будешь требования к нему расписывать, проектировать, вылизывать, то рынок тю-тю, займут. Поэтому надо всегда учиться работать профессионально и поддерживать качество на уровне, а не ждать пока тебя мордой 10 раз будут тыкать на соответствие каким-то требованиям. Потому что, видимо, для некоторых по-другому ну никак. Прям как в армии, пока матом не наорёшь и пию..лей не выпишешь, ничего делать нормально не будут. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:22 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КhVosttГде-то в ТЗ прописано, чтобы ты не справлял нужду за рабочим столом, нет? Это тоже будет считаться административным правонарушением. Попробуешь спросить ещё раз? :-) Продолжать можно долго, потому что ТЗ не описывает многих аспектов разработки. Иначе ТЗ будет дороже самой разработки. Кроме того, разработка и чёткое ТЗ, это очень редкий случай, я его в практике вообще не видел. Разработка это всегда развитие продукта, отсюда практики CI, DevOps и т.д. Какое ТЗ? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:24 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
skyANAАлексей Кпропущено... Это уже вопрос к самим требованиям, которые должны быть заложены ещё до написания программы. Другими словами, не нужно перекладывать ответственность с архитекторов на рядовых программистов. А пример привести можешь того, в каком виде и куда должны быть заложены требования к качеству кода?Ты не понимаешь. Ещё раз. Требовать исполнение можно только в том случае, если требования чётко сформулированы НА БУМАГЕ. Всё остальное можно рассматривать только в качестве рекомендаций. Например, можно требовать соблюдения правил именования Camel Case , которые чётко описаны, сослаться на эти правила в ТЗ. Нужны ещё примеры? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:27 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
hVosttПродолжать можно долго, потому что ТЗ не описывает многих аспектов разработки. Иначе ТЗ будет дороже самой разработки. Часто так и бывает. hVosttКроме того, разработка и чёткое ТЗ, это очень редкий случай, я его в практике вообще не видел.Видимо в этом и есть проблема. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:30 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей Ксформулированы НА БУМАГЕ.ну дак они есть. Корпоративные. По стилю кодирования, по госТайне, по ... ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:32 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
hVosttПродолжать можно долго, потому что ТЗ не описывает многих аспектов разработки.А потом архитектор ходит и ноет по поводу того, что программисты не исполняют требования, которые он забыл озвучить перед началом проекта. Детский сад... ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:32 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Petro123Алексей Ксформулированы НА БУМАГЕ.ну дак они есть. Корпоративные. По стилю кодирования, по госТайне, по ...Ну и замечательно. Если эти требования есть, то их надо исполнять. Я об этом и говорю. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:33 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КВидимо в этом и есть проблема. В чём проблема, в том, что требования с течением времени меняются, уточняются, информационные системы не являются замороженным функционалом в вакууме, а развиваются и эволюционируют? Вот это проблема так проблема ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:34 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КskyANAпропущено... А пример привести можешь того, в каком виде и куда должны быть заложены требования к качеству кода?Ты не понимаешь. Ещё раз. Требовать исполнение можно только в том случае, если требования чётко сформулированы НА БУМАГЕ. Всё остальное можно рассматривать только в качестве рекомендаций. Например, можно требовать соблюдения правил именования Camel Case , которые чётко описаны, сослаться на эти правила в ТЗ. Нужны ещё примеры? Требования к качеству кода, как правило это внутренние требования, которые исходят от тех лида/архитекта и тп. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:34 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
просто примера ради, достался мне 1 проект, в проекте есть файлы по 5к строк кода! отладка и правка багов в этих файлах превращается в сущий АД. Потому, что перемещаться по файлу, искать нужный код, который уже есть, понимать, что конкретно выполняет этот класс невозможно. Потому, что этот класс делает всё, что угодно. И когда сталкиваешься с таким кодом, приходит чёткое понимание того, почему классы должны быть SOLID и реализовывать только тот функционал, который можно описать 1 предложением. Все эти 5 тысяч строк кода еще и на регионы разбиты, типа тут у нас методы, тут конструкторы, тут свойства и поля. Спасает только Bookmarks, навигация по коду и т.д. И то не шибко сильно. Код выполняет поставленные перед ним задачи. Но сопровождать такой код, полная жопа. При этом, так как у кода высокая связанность, разбить это в данный момент крайне сложно и контрпродуктивно. Я убью кучу часов просто на то, чтоб код стал читаемым и вменяемым. А часов у меня таких нет, от меня требуют решения проблем заказчика, а не решения проблем разработчиков. По этому, этот говно код останется там на веки вечные, пока не закончится жизненный цикл этого приложения. Всё это не важно, когда пишешь код для себя, но когда пишешь код и знаешь, что его могут поддерживать другие люди, старайтесь ориентироваться на эталонный код. А не считать, что любой код, это говнокод ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:35 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Petro123казинакна этом форумене видел))). Ты че такой агрессивный? я белый и пушистый это так.... потроллить местных всезнаек ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:39 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
ПарамонТребования к качеству кода, как правило это внутренние требования, которые исходят от тех лида/архитекта и тп.Какая разница от кого они приходят. Я говорю о том, что они должны быть чётко сформулированы. Ещё пример. Если есть требования, что тело метода должно входить на экран монитора, тогда точно назови максимальное число строк в теле метода, а то мониторы бывают разные. А если не можешь, то вообще не заикайся на эту тему в присутствии коллектива. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:40 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КhVosttПродолжать можно долго, потому что ТЗ не описывает многих аспектов разработки.А потом архитектор ходит и ноет по поводу того, что программисты не исполняют требования, которые он забыл озвучить перед началом проекта. Детский сад... Я не ною :) Ведётся документация разработчика, есть корпоративные стайлгайды, наборы рекомендаций, архитектурные решения документируются в коде и описываются в вики, за качеством кода строго слежу, программисты как и все люди косячат, совершают ошибки. Одна из важных задач архитектуры -- свести к минимуму ошибки. Пишутся тесты, много тестов. Всё это позволяет сосредоточиться на задаче. Как показатель качества кода и архитектуры, новые программисты уже в первые дни могут писать полезный код, на разбираясь неделями в кодовой базе. Потому что применяются паттерны и принципы, знакомые профессиональным разработчикам, используется общеизвестная терминология. Не нужно размахивая руками объяснять почему это плохо, а это хорошо, апеллируя к какому-то своему прошлому опыту. Существуют критерии, принципы, практики и методики. Те, кто на это клал, у нас не работает и не задерживается. И в других компаниях тоже, так как мы общаемся с многими другими компаниями и ведём общую деятельность. Когда ты законсервирован один в своём коде, то взгляды конечно будут сильно отличаться. Что мы как раз и наблюдаем. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:41 |
|
Исключения vs коды возвратов
|
|||
---|---|---|---|
#18+
Алексей КЕщё пример. Если есть требования, что тело метода должно входить на экран монитора, тогда точно назови максимальное число строк в теле метода, а то мониторы бывают разные. А если не можешь, то вообще не заикайся на эту тему в присутствии коллектива. Нет такого требования. Есть другая метрика, которая позволяет оценить, не является ли метод сильно раздутым: код для покрытия тестами сложнее и больше, чем исходный код. Ну и принципы никто не отменял. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.07.2018, 11:43 |
|
|
start [/forum/topic.php?fid=20&msg=39672979&tid=1399295]: |
0ms |
get settings: |
10ms |
get forum list: |
13ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
154ms |
get topic data: |
9ms |
get forum data: |
2ms |
get page messages: |
68ms |
get tp. blocked users: |
1ms |
others: | 304ms |
total: | 569ms |
0 / 0 |