"PHP" / Говнокод #2378 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 hardcoder, (Updated )

Комментарии (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(); пройти?
    Ответить
    • Ну конечно, любой нормальный не быдлокод-способ бы подошёл, всё верно.
      Ответить
      • Быдлокодеры делают мир таким смешным какой он есть! Смех это - красота, а красота - спасет мир! Пхааххахаа
        Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?


    8