cat.php$num = NULL;
foreach(glob('data/datablog/*.dir') as $fdir) {
$num++;
$div = ($num%2) ? '<div class="b"><img src="'.$conf['ico'].'act_b1.png" alt=""/>' : '<div class="a"><img src="'.$conf['ico'].'act_g1.png" alt=""/>';
if(file_exists($fdir)){
$id = preg_replace('|data/datablog/([0-9]+).dir|si','\1',$fdir);
$cb = count(scandir('data/datablog/'.(int)$id.'/')) - 2;
echo $div.' <a href="/view/'.(int)$id.'/">'.file_get_contents($fdir).'</a> ('.(int)$cb.')</div>';
} else {
echo 'Категории не созданы!';
}}
file_exists здесь не нужно т.к файл всегда буде существовать, glob не может выдать в списке файл которого нет.
$id = preg_replace('|data/datablog/([0-9]+).dir|si','\1',$fdir);
Регулярка тут тоже не нужна, достаточно будет написать $id = pathinfo($fdir, PATHINFO_FILENAME);
(int) там тоже не где не нужно.
ЗЫ
Я смотрю у тебя в файлах по десять рас фильтруются переменные
div('<a href="/comm/'.(int)$_GET['cat'].'/'.(int)$_GET['read'].'/">Комментарии</a> ('.$count.')','top');
div('Ссылка на блог:','b');
div('<input name="text" value="'.$home.'/view/'.(int)$_GET['cat'].'/'.(int)$_GET['read'].'/"/>','a');
Лучше это делать один рас в самом верху...
Дальше смотрел не охота т.к сразу видно что код начинающего
Изменил: Lugaro (31.10.2010 / 14:05)