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

    +150.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    $result_row = $db->query("SELECT DISTINCT good_vars.coef_value, good_vars.coef_type ... ");
    $row = $db->fetch_array($result_row);
    $coef_type = $row['coef_type'];
    $coef_value = $row['coef_value'];
    $array = explode(',', ($coef_type.','.$coef_value));
     
    $result = 0;
    if ($array[0] == 2) {
    	$result = $price * $array[1] / 100;
    } elseif ($array[0] == 3) {
    	$result = $array[1];
    }

    ох уж эти студенты...

    Запостил: wasa, 08 Декабря 2009

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

    • Похоже на фейк. Не может человек разумный быть настолько... эммм... неразумным
      Ответить
      • и нах такое делать?
        $coef_type = $row['coef_type'];
        $coef_value = $row['coef_value'];
        $array = explode(',', ($coef_type.','.$coef_value));
        Ответить
        • чтобы из ассоц. массива сделать нумерованный :-D
          Ответить
          • а fetch_array какой возвращает ?
            Ответить
            • XXXX_fetch_array() по умолчанию дублирует данные. Можно обращаться и по числовым и по строковым ключам.
              Но тут обёртка, фз что она возвращает.

              Скорее всего, было неизвестно как/лениво переписывать фрагмент 7-12, вот и написали костыль для перевода данных в нужный вид. Я такое встречал иногда у одногруппников.
              Ответить
        • Дык, я тоже посмеялся. Но потом подумал, что может быть запятые в $coef_type и $coef_value. Правда, если это и так, то это все равно так нельзя, запутанно слишком.
          Ответить
    • Достаточно было написать эту штуку...
      $array = explode(',', ($coef_type.','.$coef_value));


      Название функции в данном контексте подтверждает взрыв мозга.

      Можно было и так...
      $a = array();
      $a[] = $coef_type;
      $a[] = $coef_value;


      Вдруг человеку приспичило именно с этими параметрами поработать как с массивом?
      Ответить
      • А можно проще...
        $a = array($row['coef_type'], $row['coef_value']);

        Хрень конечно, если кто то действительно так пишет лучше пусть идет дороги мести... Хоть польза будет :)
        Ответить
    • А если в $row['coef_type'] и $row['coef_value']; данные через запятую? :)
      Ответить
      • тогда код работать не будет :)
        но там очевидно числа
        Ответить
        • Ага, дробные, в русской локали, с запятой в качестве десятичного разделителя.
          Ответить

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