PHP Код на оценку знатокам

  • Автор темы Автор темы kav78
  • Дата начала Дата начала

kav78

Новичок
PHP Код на оценку знатокам

Здраствуйте уважаемые знатоки завсегдатого клуба!!!
PHP:
<?
$file=file_get_contents('http://www.gazeta.ru/export/rss/social20.xml');   
preg_match_all ('#<title>(.+)</title>#i', $file, $text);  
preg_match_all ('#<description>(.+)</description>#i', $file, $text1); 
preg_match_all ('#<link>(.+)</link>#i', $file, $text2);  
for ($i=1; $i< count($text[0]); $i++)
{
echo '<b>'.$text[1][$i].'</b><br/>'.$text1[1][$i].'<br/><a href="'.$text2[1][$i].'">Далее...</a><br/><br/>';
}
?>
Прошу поправить данный код, и если что не нак, то плиз дайте знать.
Я новичек потел над ним 4 часа.
Зарание спасибо!!!
 
вообщето xml парсят обычно специальными средствами (какой у тебя РНР ?).
Проверай переменные, например $file, может не удалось соединиться с урлом.
Почему ты назвал переменные $text, $text1 ... лучше $title, $description ...
 
http://ru2.php.net/xml - читать по 10 раз утром, в обед и вечером. Чтобы не приходили в голову мысли парсить xml регулярками
 
посмотри в сторону xml парсеров
или составь одно выражение
 
AmdY
php 4
Изучал я эти спец средства, голову ломал неделю, мне просто так легче. Вопрос другой: коректно ли написан код?
 
чего накинулись-то? человек регулярные выражения выучить хочет.
вот если он спустя полгода так писать будет, тогда - да, караул.
 
млин, зачем??? Объясните мне, дураку, зачем парсить xml регулярными выражениями??
не надо парсить его регулярным выражением в данном случае
но сделать одно выражение надо =))
 
Ну чё накинулись то :)
Код работает ?

Если да то какая разница правильно или нет, понимаю если бы за этот код были деньги уплачены, тогда ещё можно было бы спорить стоит код своих денег или нет (своего рода независимая экспертиза) а так то какая разница.
 
Сверху