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

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
  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 rostislav, (Updated )

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

  • это говнокод еще потому что как минимум нужно делать АПИ локализации типа геттекста раз есть переводы строчек.
    а с $godate вообще вроде есть же форматирование даты времени. не? )
    Ответить
  • похоже, если грамотно, то нужно переделывать не код, а архитектуру
    Ответить
    • и заглянуть на php.net $id = $arr['id']; $userid = $arr['userid'];
      почитать про list например
      Ответить

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

Переведи на "PHP", guest!

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


    8