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

    +164

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    for($i = 0; $i < 1000; $i++) {
            if(isset($_POST['discount_sum_'.$i]))
            {
                $a_discount_id_service[] = htmlspecialchars(@$_POST['id_service_'.$i]);
                $a_discount_sum[]        = htmlspecialchars(@$_POST['discount_sum_'.$i]);
                $a_discount_type[]       = htmlspecialchars(@$_POST['discount_type_'.$i]);
                $a_discount_time_type[]  = htmlspecialchars(@$_POST['discount_time_type_'.$i]);
            }
            else {
                break;
            }
    }

    По POST'у передаётся некоторое количество 'discount_sum_'.$i, 2 или 3, например 'discount_sum_1', 'discount_sum_2', 'discount_sum_3', в зависимости от точно известного количества записей в одной из таблиц, и передаваемое количество точно известно в момент передачи, но ведь лучше перебрать от 1 до 1000! :))

    Запостил: hardcoder, 06 Января 2010

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

    • >Запостил: hardcoder, 06 Января 2010
      о да, я с redtube'ского хардкора еще не отошел, а ты меня просто убиваешь
      Ответить
      • Простите, не понял.
        Причём тут редтьюбовский хардкор?
        И чем я Вас убиваю?
        Ответить
        • говнокод хороший)
          а редтюбовское хардкорное порно это то что я только что смотрел
          Ответить
    • Автор запостил говнокод... но причину почему-то указал другую... Это типа рождественский прикол? ;-)
      Ответить
      • Что Вы имеете в виду под словом причина?
        Описание к говнокоду? А какая основная причина? Обоснуйте.
        Ответить
    • Ой ужас. Но, помнится, в каком-то проекте я тоже так наговнокодил ;)
      Ответить
      • значит ты, быдлокодер! Пхахахаха
        Ответить
        • Это было лет десять назад ) Пхахахаха
          Движение от быдло- к нормальному коду не так просто, но есть и будет есть.
          Ответить
      • А я вроде так не делал, но мне досталось от одного G-кодера вот в таком виде :)
        Ответить
    • Почему от 1 до 1000? Этот код перебирает до первого не преданного параметра, но не больше 1000.
      Ответить
      • Там систему можно было по другому вообще сделать!
        Ответить
      • Я не сказал где он останавливается, а указал именно тот факт, что перебирает, что вообще само по себе говнокод.
        Ответить
    • видимо автор был настолько возбуждён, что не дочитал до строчки
      else break;
      Ответить
      • это говнистость кода не уменьшает
        Ответить
      • почему не дочитал? я указал причину говнокодистости, то что там стоит break не отменяет цикла от 1 до 1000, хоть он и закончится на первом неопределённом параметре в POST'e. Не туда смотрите.
        Ответить
    • А что слабо было передавать массив значений, а не кучу значений? Чтото вроде:
      <input type="text" value="блабла" name="discount_time_type[]">
      А потом с помощью foreach(); пройти?
      Ответить
      • Ну конечно, любой нормальный не быдлокод-способ бы подошёл, всё верно.
        Ответить
        • Быдлокодеры делают мир таким смешным какой он есть! Смех это - красота, а красота - спасет мир! Пхааххахаа
          Ответить

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