ФорумСообществоФлейм → Когда код хуевый ...

Когда код хуевый ...

  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 6:18

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

    Как быть короче? Руки опускаются при виде всего этого непотребства. Сильно глубоко вникать и переписывать не очень-то и хочется, а править баги становится все сложнее и сложнее.
  • Sinkler

    Сообщения: 9658 Репутация: N Группа: в ухо

    Spritz 3 сентября 2010 г. 6:22, спустя 3 минуты 51 секунду

    В последнее время периодически с таким сталкиваюсь… Не вникаю, просто пытаюсь включить всю свою думалку, чтобы сделать всё быстрее, забив на скорость, понятия о красивом коде и прочее. Но, обычно, переделки разовые, продолжительной работой с такой хуетой, благо, не было…
  • VaseninM

    Сообщения: 2417 Репутация: N Группа: Адекваты

    Spritz 3 сентября 2010 г. 6:27, спустя 4 минуты 37 секунд

    Обычно такие вещи разовые и я даже так делаю:
    Вес: {$cart_content.ves}<br />

    :D
  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 6:38, спустя 11 минут 10 секунд

    в том-то и дело, что это постоянка. И с таким кодом я сталкиваюсь ежедневно с понедельника по пятницу.

    Дело не в том, что я выпендриваюсь. Нет. Он правда жутко запутанный и черезжопный
  • kostyl

    Сообщения: 5203 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 6:44, спустя 6 минут 41 секунду

    Абырвалг, попробуй отрефакторить, повыноси для начала методы с целью начала приближения к нормлаьной структуре, разделению кода и чё там тебе надо, чтобы проще сопровождать…. постарайся упаковать сильную жопу за каки нить фасадом.., только думать надо больше чем делать… подейла так, просто выноси совмещай, прчти не пиши или вообще не пиши (тобишь не переписывай) основной функционал, сделай структуру лучше, раздели и всё такое. Потом постепенно перепиши грамотно, со всемы выносами и заносами трудные куски, ведь ты должен перед этим был сделать так что бы их можно было бы переписать не особо затрагивая все остальное. Андестенд?
  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 7:02, спустя 17 минут 14 секунд

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

    ээ, это все не я писал. Мое ваще совсем отдельно идет. Я сейчас правкой багов занят. Например почему после выбора такой-то опции из списка такие-то слои не пропадают а должны? А потому что там происходит 4 проверки в разных местах и слой 3 раза прячится и 1 раз в итоге показывается. Примерно все так.
  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 7:38, спустя 36 минут 16 секунд


     $project = Mage::getModel(&quot;foo/banana&quot;)->getCollection()->addFieldToFilter(&#39;id&#39;, $project_id)->toArray();
     $template = Mage::getModel(&quot;foo/apple&quot;)->getCollection()->addFieldToFilter(&#39;bar_id&#39;, $project[&#39;items&#39;][0][&#39;template_id&#39;])->toArray();


    зачем писать эту пиздятину, если она нигде не используется? Или после селекта что-то изменится?
  • VaseninM

    Сообщения: 2417 Репутация: N Группа: Адекваты

    Spritz 3 сентября 2010 г. 7:29, спустя 23 часа 51 минуту 28 секунд

    Абырвалг, поюзай слаед или социаленжайн 3. А потом жалуйся на магенту)))
  • kostyl

    Сообщения: 5203 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 7:31, спустя 1 минуту 19 секунд

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

  • NRG

    Сообщения: 4761 Репутация: N Группа: в ухо

    Spritz 3 сентября 2010 г. 7:35, спустя 4 минуты 37 секунд


    Абырвалг, поюзай слаед или социаленжайн 3. А потом жалуйся на магенту)))
    Магента тут непричем, скорее всего это какието кастомные изменения
  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 7:37, спустя 1 минуту 17 секунд


    Абырвалг, поюзай слаед или социаленжайн 3. А потом жалуйся на магенту)))

    со слаеда начинал в дветыщишестом. И мне он даже когда-то нравился. Но это было давно.

    Я не жалуюсь на магенту, я жалуюсь на код, написанный моими предшественниками. {+++39+++}
    Магента тут непричем, скорее всего это какието кастомные изменения

    да. В самой первой строке темы написано, что я про модуль
  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 7:46, спустя 9 минут 34 секунды

    а еще у нас много магических чисел… {+++216+++} и логи записываются в базу аштээмлом.

    <ul class=&quot;error&quot;>
    <li>Вам пиздец, господа</li>
    </ul>


    Ага, прямо вот так и записываются блядь. Причем если мне нужно дописать лог, я получаю предидущее аштеемло, делаю конкатенацию и обновляю поле. Круто?) Естественно никакой фильтрации по типам логов/датам нету, равно как и пагинации.

    Пару раз кто-то при записи лога написал ul вместо li и весь лог шел по пизде.
  • kostyl

    Сообщения: 5203 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 7:59, спустя 12 минут 46 секунд

    Абырвалг, начни писать в другое место хули… это потом снесешь..
  • Абырвалг

    Сообщения: 6477 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 8:02, спустя 3 минуты 30 секунд

    там все лавинообразно. Я пока не могу выделить небольшой пучек, который можно переписать. Если переписывать - то много.
  • kostyl

    Сообщения: 5203 Репутация: N Группа: Джедаи

    Spritz 3 сентября 2010 г. 8:13, спустя 10 минут 32 секунды

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

Пожалуйста, авторизуйтесь, чтобы написать комментарий!