Чем лечить людей которые пишут такие коды.

baev

‹°°¬•
Команда форума
в коде вместо HTML весь код страницы нахреначен этими "хелперами" всесто того чтобы вставить TD используется функция и для лткрывающего и для закрывающего для TABLE отдельный и так свои для почти всех элементов
— гм, похоже, «срисовано» с PostNuke (если вдруг кому интересно — смотрите файл includes/pnHTML.php).
Или с какого-нибудь из его многочисленных клонов.
 

HVitaminka

Новичок
triumvirat
В этом и вопрос
я не понимаю зачем выводить HTML методами класса
собственными кривыми когда есть DOM
А что такое DOM? Я только начала изучать php, сижу форум читаю и уже не впервый раз вижу этот DOM. Что это?
 

jonjonson

Охренеть
1. Код в не контекста задачи бесполезен и не обсуждаем.
2. Код читается нормально.
3. Код избыточен (возможен рефакторинг).
4. Если он работает, то что ещё нужно, кроме рефакторинга?
5. Если этот кусок кода одноразовый (автор не объяснил задачу) и выполняет задачу... то мне лично пофиг.
6. Эмоции должны быть конкретны, а не по поводу эстетической составляющей.
7. Я бы константу добавил или ещё что, но задача не понятна.
 

nalim

Новичок
jonjonson
Эмоции негативные! и свзяны с тем, что затраты не рефакторинг крайне высоки.
Трудно придумать такую задачу в рамках которой он был бы адекватен.
Это код высокопосещаемого интернет магазина,
для которого понадобилось делать редизайн
 

jonjonson

Охренеть
nalim, ты как-то влияешь на команду? Ты являешься её менеджером? Если нет, то конечно тяжело, когда на тебя скидывают "неудобоваримый код других разработчиков".

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

Плохого кода не избежать. Следовательно эмоций не хватит. Главное плохой код ограничить. Даже плохой код работает. Если задача дальнейшей его поддержки и изменения не стоит, то порой проще закрыть глаза на него, но при этом сделать кадровый выбор.

У меня достаточный опыт был с кучей говнокода. В последнем случае не я отказался от чужого говнокода, а отказались от меня. :)
Плюс в том, что я приобрёл хороший опыт. Если раньше меня сбивали с толку недостатки чужого кода, то теперь я его воспринимаю спокойно и больше слежу за своим. Говнокода много и с ним приходится работать. Где могу, стараюсь игнорировать или править. Если нет, то что-то переписываю. Если моего времени из-за этого на решение простых задач требуется больше (добавление функционала), то я ставлю в известность ответственного. Мне вовсе не жаль, если отказываются от моих услуг в пользу делающих код ногами. Это влияет на стабильность заработка, но не влияет на душевное спокойствие. Я уверен, что люди пишущие говнокод вряд ли станут перед проблемой отсутствия работы. Но ведь то же и со мной. Работа есть. Кроме того обладателям говнокода тоже требуются грамотные разработчики. Обычно это происходит на успешных проектах, когда говнокодер уже не может поддерживать свой код, а проекту нужно жить. На мёртвые проекты не следует заморачиваться. Плохой код в них зачастую не главное.

И ещё я порой ловлю себя на том, что мой код то же не идеален. Тоже порой требуется рефакторинг, а на него нет времени. Нельзя всё сделать идеально, но можно достичь некоторой идеальности в рамках момента сейчас. :)
 

HraKK

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

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

nalim

Новичок
гм не считаю себя идеалом ни-разу

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


Рефакторинг спасет мир, я это понял )
К сожалению я не тот менеджер который принимает решения, а скорее тот который принимает ответственность самодурство других)
но ЗП устраивает )
 

Духовность™

Продвинутый новичок
скажите, а проект после его написания надо напильником обрабатывать? кто как обычно делает?
 

nalim

Новичок
развитие темы глубоко и максимально полно меня удовлетворило
ремарки "сам дурак" не интересуют, спасибо )
 
Сверху