1. PHP / Говнокод #14389

    +165

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    /* Здесь мы имеем сраный унаследованный пиздец: количество товара передается в параметрах POST-запроса,
	имеющих имена вида productX_count, где X - ИД товара. Как предыдущие разработчики разобирали это счастье?
	Правильно, перебирали все (блядь, ВСЕ, блядь, 20000 штук!) товары и проверяли, есть ли соответствующее
	значение в запросе.
*/

  $items5 = GetIBlockElementList(5);
  while($arItem = $items5->GetNext())
    if(isset($_POST['product'.$arItem['ID']."_id"]) && is_numeric($_POST['product'.$arItem['ID']."_id"]))
    {
      $_PRODUCTS[$arItem['ID']]['count'] = $_POST['product'.$arItem['ID']."_count"];
    }

    В этом коде меня утешает только боль и ненависть комментария.

    Запостил: clauclauclau, 20 Января 2014

    Комментарии (12) RSS

    • ava Vindicar 20.01.2014 08:06 # +2

      Перебрать ключи запроса, начинающиеся на 'product' и кончающиеся на '_count', а потом отфильтровать кривые айдишки, не?
      Кстати, удивил закос под суперглобальный массив на 11й строке.
      Ответить

      • ava kegdan 20.01.2014 08:46 # 0

        Закос ли?
        Ответить

        • ava bormand 20.01.2014 09:29 # +1

          Закос. Подчеркивание и капс суперглобальности не дадут ;)
          Ответить

      • ava com1 20.01.2014 14:06 # +2

        указать в html <input type="text" name="product[{$id}]" value="{$count ? : ''}" /> и работать только с $_REQUEST['product'], перебирая только пришедшие позиции, не? )))

        закос под суперглобальность приколен, но приведенный код может являться частью функции... конечно приставка "супер" в этом случае отпадает, но вот "глобальность" может и появиться. )))
        Ответить

        • ava Vasiliy 20.01.2014 14:25 # +1

          и не факт совсем $_PRODUCTS можно легко таскать по всему коду сериализуя и разсериалируя в сессию
          Ответить

          • ava com1 20.01.2014 15:31 # 0

            +1. об этом у меня, глядя на приведенный код, даже мысли не возникло. )))
            Ответить

          • ava Lure Of Chaos 21.01.2014 02:20 # 0

            > $_PRODUCTS можно легко таскать по всему коду сериализуя и разсериалируя в сессию
            а скорей всего в $_POST
            Ответить

        • ava Vindicar 20.01.2014 17:12 # 0

          Ну это если совсем по-хорошему.
          Я-то предлагал в рамках "меньшего говна и малых правок".
          Ответить

          • ava com1 20.01.2014 23:17 # 0

            ну, то что я предложил можно тоже считать ГК. )) ни кто не запрещает конечному идиоту пользователю забить эти инпуты всякой вкусной херней. можно глянуть в сторону html5 и объявить <input type="number" ....>, но и в этом случае от клинических идиотов пользователей ни кто не застрахован.

            а разгребать увы... ))) (((
            Ответить

            • ava Vindicar 23.01.2014 09:08 # 0

              Ммм, type="number" тут незачем, юзверь же не будет вручную вводить айдишники товаров. А фильтровать вводимые данные всё равно потребуется.

              А так - вполне себе подход, получше моего.
              Ответить

    • ava psycho-coder 20.01.2014 10:02 # 0

      Лютый пиздец.
      Ответить

    • ava Homess 22.01.2014 22:52 # 0

      Узнаю быдлокодинг школоты под Битрикс...
      Ответить

    Добавить комментарий