Forum PHP.pl

Elo, pomógłby mi ktoś zoptymalizować poniższy kod? Chodzi o to, że on pobiera dane z RSS i je wyświetla, tylko problem w tym, że tak obciążał serwer, że usługodawca kazał mi się wynieść z serwera.

Kod:

[PHP] pobierz, plaintext 
<?php
$site = "http://www.ttrworldtour.com/ranking/mens-ranking-list.html?type=116&tx_txresults_pi1%5BfeedId%5D=53";
$fp = @fopen($site,"r");
while(!feof($fp)) $raw .= @fgets($fp, 196);
fclose($fp);
    echo '<p style="margin: 0 15px 0 15px;"><a href="http://www.ttrworldtour.com/ranking/mens-ranking-list.html" target="_top"><strong>TTR TOP 5 MEN</strong></a></p>';
if( eregi("<item>(.*)</item>", $raw, $rawitems ) ) {
 $items = explode("<item>", $rawitems[0]);
    
 for( $i = 0; $i < count($items)-1; $i++ ) {
  eregi("<title>(.*)</title>",$items[$i+1], $title );
  eregi("<link>(.*)</link>",$items[$i+1], $link );
  eregi("<categorie>(.*)</categorie>",$items[$i+1], $cat);
  echo "<p style=\"margin: 0 15px 0 15px;\"><a href='".$link[1]."' target='_top'>".$title[1]."</a> </p>".$cat[1];
 }
}
 
echo '<br /><br />';
 
$sites = "http://www.ttrworldtour.com/ranking/womens-ranking-list.html?type=117&tx_txresults_pi1%5BfeedId%5D=53";
$fps = @fopen($sites,"r");
while(!feof($fps)) $raws .= @fgets($fps, 196);
fclose($fps);
    echo '<p style="margin: 0 15px 0 15px;"><a href="http://www.ttrworldtour.com/ranking/womens-ranking-list.html" target="_top"><strong>TTR TOP 5 WOMEN</strong></a></p>';
if( eregi("<item>(.*)</item>", $raws, $rawitemss ) ) {
 $itemss = explode("<item>", $rawitemss[0]);
 
 for( $i = 0; $i < count($itemss)-1; $i++ ) {
  eregi("<title>(.*)</title>",$itemss[$i+1], $titles );
  eregi("<link>(.*)</link>",$itemss[$i+1], $links );
  eregi("<categorie>(.*)</categorie>",$itemss[$i+1], $cats);
  echo "<p style=\"margin: 0 15px 0 15px;\"><a href='".$links[1]."' target='_top'>".$titles[1]."</a> </p>".$cats[1];
 }
}
?>
[PHP] pobierz, plaintext 

Z góry dziękuję za pomoc.

[PHP] pobierz, plaintext 
<?php
$site = "http://www.ttrworldtour.com/ranking/mens-ranking-list.html?type=116&tx_txresults_pi1%5BfeedId%5D=53";
$data = @file_get_contents($site) or die('blad polaczenia');
?>
<p style="margin: 0 15px 0 15px;"><a href="http://www.ttrworldtour.com/ranking/mens-ranking-list.html" target="_top"><strong>TTR TOP 5 MEN</strong></a></p>
<?php
    preg_match_all('#<item>(.*?)</item>#', $data, $items);
    $items = $items[0];
    $output = array();
    $count = count($items);
    <?php for($i=0; $i<$items; $i++): ?>
    <?php
        preg_match('#<title>(.*?)</title>#', $items[$i], $title);
        $output[$i]['title'] = $title[1];
        
        preg_match('#<link>(.*?)</link>#', $items[$i], $link);
        $output[$i]['link'] = $link[1];
        
        preg_match('#<categorie>(.*?)</<categorie>#', $items[$i], $categorie);
        $output[$i]['categorie'] = $categorie[1];
    ?>
    <p style="margin: 0 15px 0 15px;"><a href="<?php echo $link[1];?>" target='_top'><?php echo $title[1];?></a></p><?php echo $categorie[1];?>
    <?php endfor;?>
        
        <?
            // teraz robisz sobie file_put_contents('db.txt', serialize($output));
            // i zamiast laczyc sie co z zew. serwerem co chwila, robisz to np. raz na 5 minut
            // a przez reszte czasu czytasz dane z pliku 'db.txt'
            
        ?>
        
        <?
            
            // drugiej czesci skryptu nie podaje bo jest praktycznie identyczna 
        
        ?>
[PHP] pobierz, plaintext 

Lepiej skorzystać na przykład z simplexml" title="Zobacz w manualu PHP" target="_manual, prościej i wygodniej:

[PHP] pobierz, plaintext 
<?php
 
$xml = @simplexml_load_file("http://www.ttrworldtour.com/ranking/mens-ranking-list.html?type=116&tx_txresults_pi1%5BfeedId%5D=53");
 
foreach ($xml as $channel)
{
    foreach($channel as $item)
    {
        echo $item->title;
    }
}
 
?>
[PHP] pobierz, plaintext 

Scigamy sie?

The SimpleXML extension requires PHP 5.

@gox:

Mógłbyś mi to objaśnić?

Raczej nigdy nic nie robiłem w czytaniu/zapisywaniu itp. z plikami.

P.S.: I na jakiej niby to zasadzie? No to każdemu skrypt będzie albo najpierw zapisywał i dopiero czytał, albo zawsze czytał. Nie rozumiem tego w ogóle.

Ten post edytował in5ane 14.11.2008, 23:41:34

Ostrzegam ze kodzik nie jest zdebugowany

[PHP] pobierz, plaintext 
<?php
$site = "http://www.ttrworldtour.com/ranking/mens-ranking-list.html?type=116&tx_txresults_pi1%5BfeedId%5D=53";
$dbpath = './db/';
?>
<p style="margin: 0 15px 0 15px;"><a href="http://www.ttrworldtour.com/ranking/mens-ranking-list.html" target="_top"><strong>TTR TOP 5 MEN</strong></a></p>
<?php
    if ( ! is_dir($db_path))
        @mkdir($dbpath, 0777) or die('nie moge utworzyc bazy danych, zrob to recznie ');
    
    list($dbfile) = glob("{$dbpath}*");
    
    if ( is_file($dbfile) AND intval($dbfile) > ( time() - 60) ) {
        $output = @unserialize(@file_get_contents($dbfile)) or die('nie moge odczytac pliq');
    }
    else {
        preg_match_all('#<item>(.*?)</item>#', $data, $items);
        $items = $items[0];
        $output = array();
        $count = count($items);
        for($i=0; $i<$items; $i++) {
            preg_match('#<title>(.*?)</title>#', $items[$i], $title);
            $output[$i]['title'] = $title[1];
            preg_match('#<link>(.*?)</link>#', $items[$i], $link);
            $output[$i]['link'] = $link[1];
            preg_match('#<categorie>(.*?)</<categorie>#', $items[$i], $categorie);
            $output[$i]['categorie'] = $categorie[1];
        }
        @unlink($dbfile) or die('nie moge usunac pliq');
        @file_put_contents($dbpath.time(), serialize($output)) or die('nie moge zapisac pliq');
    }
    
    <?php foreach ( $output as $o )
        <p style="margin: 0 15px 0 15px;"><a href="<?php echo $o['link'];?>" target='_top'><?php echo $o['title'];?></a></p><?php echo $o['categorie'];?>
    <?php endforeach; ?>
[PHP] pobierz, plaintext 

http://www.ajaxray.com/blog/downloads/feedparser
i przykład:

[PHP] pobierz, plaintext 
<?php 
include('FeedParser.php'); 
$Parser     = new FeedParser();
 
$Parser->parse('http://www.sitepoint.com/rss.php');
 
$channels      = $Parser->getChannels();     
$items         = $Parser->getItems();        
?>
 
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
    <title>Testing the PHP Universal Feed Parser</title>
    <style type="text/css">
    body{
        padding: 0px;
        margin: 50px 150px;
        border: 1px solid #ddd;
        font-family: verdana, arial;
    }
    
    h1#title {
        background-color: #eee;
        border-bottom : 1px solid #ddd;
        margin: 0px 0px 15px;
        padding:10px;
        text-align: center;
    }
    h1#title a{
        font-size: 18px;
    }
    
    p, a{
        padding: 5px;
        margin: 5px;
        font-size: 12px;
    }
    p#description{
        font-size: 13px;
        font-weight: bold;
    }
    
    p.feed-description{
        border-bottom : 1px solid #ddd;
    }
    
    p#info{
        background-color:#EDF001;
        color: 1px solid #f6afab;
        padding: 5px;
        text-align: center;
        color: #444;
    }
    </style>
</head>
<body>
    <h1 id="title"><a href="<?php echo $channels['LINK']; ?>"><?php echo $channels['TITLE']; ?></a></h1>
    <p id="description"><?php echo $channels['DESCRIPTION']; ?> </p>
    
    <?php foreach($items as $item): ?>
        <a class="feed-title" href="<?php echo $item['LINK']; ?>"><?php echo $item['TITLE']; ?></a>
        <p class="feed-description"><?php echo $item['DESCRIPTION']; ?></p>
    <?php endforeach;?>
    
    <p id='info'>
        This page is an example of using 
        <a href="http://www.ajaxray.com/blog/2008/05/02/php-universal-feed-parser-lightweight-php-class-for-parsing-rss-and-atom-feeds/">
            PHP Universal Feed Parser
        </a>.
    </p>
</body>
</html>
[PHP] pobierz, plaintext 

XML Parser, CURL
to wszystki wyglada jadnie jak sie czyta kod,
jest tylko jeden problem
taki skrypt jest kilka-kilkanascie-KILKADZIESIAT razy wolniejszy
niz inny wykonany na nizszych funkcjach,
jesli na prawde mowimy o optymalizacji, to jest to zla droga,
Nie idzie ta droga!

@gox: zdebugowałem ten kod, no ale wyskakuje jeszcze error o takiej treści:

[PHP] pobierz, plaintext 
<?php
$start=microtime();
$site = "http://www.ttrworldtour.com/ranking/mens-ranking-list.html?type=116&tx_txresults_pi1%5BfeedId%5D=53";
$dbpath = './db/';
?>
<p style="margin: 0 15px 0 15px;"><a href="http://www.ttrworldtour.com/ranking/mens-ranking-list.html" target="_top"><strong>TTR TOP 5 MEN</strong></a></p>
<?php
   if ( ! is_dir($db_path))
       @mkdir($dbpath, 0777) or die('nie moge utworzyc bazy danych, zrob to recznie tongue.gif');
   
   list($dbfile) = glob("{$dbpath}*");
   
   if ( is_file($dbfile) AND intval($dbfile) > ( time() - 60) ) {
       $output = @unserialize(@file_get_contents($dbfile)) or die('nie moge odczytac pliq');
   } else {
       preg_match_all('#<item>(.*?)</item>#', $data, $items);
       $items = $items[0];
       $output = array();
       $count = count($items);
       for($i=0; $i<$items; $i++) {
           preg_match('#<title>(.*?)</title>#', $items[$i], $title);
           $output[$i]['title'] = $title[1];
           preg_match('#<link>(.*?)</link>#', $items[$i], $link);
           $output[$i]['link'] = $link[1];
           preg_match('#<categorie>(.*?)</<categorie>#', $items[$i], $categorie);
           $output[$i]['categorie'] = $categorie[1];
       }
       @unlink($dbfile) or die('nie moge usunac pliq');
       @file_put_contents($dbpath.time(), serialize($output)) or die('nie moge zapisac pliq');
   }
?>
   
   <?php foreach ($output as $o) { ?>
       <p style="margin: 0 15px 0 15px;"><a href="<?php echo $o['link']; ?>" target='_top'><?php echo $o['title']; ?></a></p><?php echo $o['categorie']; ?>
   <?php } ?>
   <br /><br /><br /><br />
<?php
echo round(microtime()-$start, 6);
?>
[PHP] pobierz, plaintext 

cos kiepsciutensko zdebugowales w takim razie :0

[PHP] pobierz, plaintext 
<?php
$start=microtime();
$site = "http://www.ttrworldtour.com/ranking/mens-ranking-list.html?type=116&tx_txresults_pi1%5BfeedId%5D=53";
$dbpath = './db/';
$storing_time = 10;
?>
<p style="margin: 0 15px 0 15px;"><a href="http://www.ttrworldtour.com/ranking/mens-ranking-list.html" target="_top"><strong>TTR TOP 5 MEN</strong></a></p>
<?php
  if ( ! is_dir($dbpath))
      @mkdir($dbpath, 0777) or die('nie moge utworzyc bazy danych, zrob to recznie tongue.gif');
  
  list($dbfile) = glob("{$dbpath}*");
  
  if ( is_file($dbfile) AND ( time() - preg_replace('#[^0-9]#', '', $dbfile) ) < $storing_time )  {
        echo 'back';
      $output = @unserialize(@file_get_contents($dbfile)) or die('nie moge odczytac pliq');
  } else {
        $data = @file_get_contents($site) or die('blad polaczenia');
      preg_match_all('#<item>(.*?)</item>#s', $data, $items);
      $items = $items[0];
      $output = array();
      $count = count($items);
      if ( $count < 1) die('pusto');
      for($i=0; $i<$count; $i++) {
          preg_match('#<title>(.*?)</title>#s', $items[$i], $title);
          $output[$i]['title'] = $title[1];
          preg_match('#<link>(.*?)</link>#s', $items[$i], $link);
          $output[$i]['link'] = $link[1];
          preg_match('#<description>(.*?)</<description>#s', $items[$i], $description);
          $output[$i]['description'] = $description[1];
      }
      @unlink($dbfile);// or die('nie moge usunac pliq');
      @file_put_contents($dbpath.time(), serialize($output)) or die('nie moge zapisac pliq');
  }
?>
  
  <?php foreach ($output as $o) { ?>
      <p style="margin: 0 15px 0 15px;"><a href="<?php echo $o['link']; ?>" target='_top'><?php echo $o['title']; ?></a></p><?php echo $o['description']; ?>
  <?php } ?>
  <br /><br /><br /><br />
<?php
echo round(microtime()-$start, 6);
?>
[PHP] pobierz, plaintext 

Sprawdziłem czas generowania obu skryptów (twój gox oraz ten mój z pierwszego postu) i czasy są takie same mniej więcej, czy to oznacza, że tak samo będą obciążały serwer czy nie?

Podobne? znaczy się, oba maja zero przed przecinkiem

Jesli mozesz, to wrzuc na forum to co Ci napisal admin.
Wywrozyc mozna ze chodzilo o zapetlanie sie skryptu i tymsamym pozeranie czasu procesora.
Wielokrotnie uruchomiony proces może powodowac spadek wydajnosci systemu.
Polec administratorowi wydzielanie zasobów dla kazdego usera,
będzie mógł w ów czas przestawić ząbek pochylenia swojego administracyjnego fotela o jeden do tyłu

Moge zagwarantowac, ze moj skrypt nie bedzie powodowal przeciazenia serwera ze wzgledu na nieoptymalne napisanie.
oczywiscie moze to zrobisz jeszcze wydajniej, ale z pewnoscia problemy z przeciazeniem serwera od tego nie znikną.

Moja rzezba cashuje wyniki na 10 sek. co oszczedzi troche transferu i spowoduje szybsze ladowanie się strony.

OK, dzięki ;-)