- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 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;
brainstorm 12.01.2011 11:16 # 0
а с $godate вообще вроде есть же форматирование даты времени. не? )
Lure Of Chaos 12.01.2011 15:21 # 0
Vasiliy 12.01.2011 15:44 # 0
почитать про list например