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

    +160

    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
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    <?php
if ($res) {
    $i=0;
    while ( $arr = $res->fetchRow() )
    {   
        $id = $arr['id'];
        $userid = $arr['userid'];
        $alias = $arr['alias'];
        $startcity = $arr['startcity'];
        $endcity = $arr['endcity'];
        $godate = $arr['godate'];
        $gohour = $arr['gohour'];
        $gorecset = mapSetToWeekdays($arr['gorecset']);
        $recdays = implode(" ", $gorecset);     
    
        $godate = ereg_replace("Mon", $l['mo'], $godate);
        $godate = ereg_replace("Tue", $l['tu'], $godate);
        $godate = ereg_replace("Wed", $l['we'], $godate);
        $godate = ereg_replace("Thu", $l['th'], $godate);
        $godate = ereg_replace("Fri", $l['fr'], $godate);
        $godate = ereg_replace("Sat", $l['sa'], $godate);
        $godate = ereg_replace("Sun", $l['su'], $godate);
    
        $rides[$i] = array('id' => $id,
                            'userid' => $userid,
                            'alias' => $alias,
                            'startcity' => $startcity,
                            'endcity' => $endcity,
                            'gohour' => $gohour,
                            'godate' => $godate,
                            'recdays' => $recdays,                   
                            );
        $i++;
    }
    
    $SMARTY->assign("rides", $rides);
}

    Этот чудесный пример говнокода написал один финн, с которым мне приходится работать вместе.
    Как видно, он, имея 4 года опыта разработки на PHP, так и не знает про extract(), compact(), а также про возможность использовать массивы в качестве аргументов str_replace. Хотя, в данном случае можно было просто написать $rides[] = $arr;

    Запостил: rostislav, 12 Января 2011

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

    • ava brainstorm 12.01.2011 11:16 # 0

      это говнокод еще потому что как минимум нужно делать АПИ локализации типа геттекста раз есть переводы строчек.
      а с $godate вообще вроде есть же форматирование даты времени. не? )
      Ответить

    • ava Lure Of Chaos 12.01.2011 15:21 # 0

      похоже, если грамотно, то нужно переделывать не код, а архитектуру
      Ответить

      • ava Vasiliy 12.01.2011 15:44 # 0

        и заглянуть на php.net $id = $arr['id']; $userid = $arr['userid'];
        почитать про list например
        Ответить

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