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

    +148

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    if(!empty($_POST['Order']['tp1'])) {
    $step3_arr['tp1'] = intval($_POST['Order']['tp1']);
}

if(!empty($_POST['Order']['tp2'])) {
    $step3_arr['tp2'] = intval($_POST['Order']['tp2']);
}

if(!empty($_POST['Order']['tp3'])) {
    $step3_arr['tp3'] = intval($_POST['Order']['tp3']);
}

if(!empty($_POST['Order']['tp4'])) {
    $step3_arr['tp4'] = intval($_POST['Order']['tp4']);
}

if(!empty($_POST['Order']['eggs1'])) {
    $step3_arr['eggs1'] = intval($_POST['Order']['eggs1']);
}

if(!empty($_POST['Order']['eggs2'])) {
    $step3_arr['eggs2'] = intval($_POST['Order']['eggs2']);
}

if(!empty($_POST['Order']['eggs3'])) {
    $step3_arr['eggs3'] = intval($_POST['Order']['eggs3']);
}

if(!empty($_POST['Order']['eggs4'])) {
    $step3_arr['eggs4'] = intval($_POST['Order']['eggs4']);
}

    Форма заказа. это только часть длинного листинга if-ов :).

    Запостил: oxser, 16 Апреля 2013

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

    • ava Serabas 16.04.2013 13:44 # 0

      умник-mode on
      $array = array(...'tp1', 'tp2', 'tp3','tp4', 'eggs1', 'eggs2', 'eggs3', 'eggs4'...);
foreach($array as $k => $v) {
    if (!empty($_POST['Order'][$v])) {
        $step3_arr[$v] = $_POST['Order'][$v];
    }
}

      умник-mode off
      Ответить

      • ava guest 16.04.2013 14:53 # 0

        for($i=1;$i<=4;$i++) {
        if(!empty($_POST['Order']['tp'.$i])) $step3_arr['tp'.$i] = $_POST['Order']['tp'.$i];
        if(!empty($_POST['Order']['eggs'.$i])) $step3_arr['eggs'.$i] = $_POST['Order']['eggs'.$i];
        }
        Ответить

        • ava Serabas 16.04.2013 14:55 # 0

          если поциферное перечисление, то да, а если там много ключей разного вида (что вполне возможно), то вариант выше
          Ответить

      • ava scriptin 16.04.2013 20:17 # 0

        А intval?
        А зачем $k?
        Ответить

        • ava Serabas 16.04.2013 20:27 # 0

          $k по привычке всегда пишу
          про intval каюсь, забыл
          Ответить

          • ava scriptin 16.04.2013 20:34 # 0

            За исключением тривиальных случаев лучше давать индексной переменной более семантическое имя.
            Ответить

    • ava bot-minurast 16.04.2013 21:38 # −2

      tp, eggs - тупая пизда и яйца.
      Ответить

    • ava deep 17.04.2013 02:50 # −2

      jpeg - японский пизда и одно яйцо
      Ответить

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