[PHP]Połączenie dwóch kodów Opcje

[kalibovers]

Czy da sie połączyć te 2 kody czyli zeby pobierał 3 kursy ze strony nbp i wklejał je do pliku bo uzywałem tego skryptu ale nie wyswietla nawet a jako zmienna nie działa. Jakies pomoce ?
Ps. uzyłem juz opcji "szukaj"

[PHP] pobierz, plaintext 
    <?php
    $strona = preg_replace('/[\r\n]/',' ',file_get_contents('http://www.nbp.pl/Kursy/KursyA.html'));
      preg_match('/z dnia <b>(.*?)<\/b>/', $strona, $zdnia);
    $content = "Kurs średni wg NBP<br>z  dnia $zdnia[1] ";
      preg_match_all('/<tr valign="middle">.*?<\/tr>/', $strona, $sa);
      for ($i=0; $i<count($sa[0]); $i++) {
       preg_match('/bg1">(.*?)<\/td/', $sa[0][$i], $waluta);
 
       preg_match('/bg1">([0-9],[0-9]*?)<\/td/', $sa[0][$i], $kurs);
         if (stristr($waluta[1],'euro')) $content .= "<br><br><b>1 EUR - $kurs[1] PLN</b> ";
    if (stristr($waluta[1],'dolar am'))$content .= "<br><br><b>1 USD - $kurs[1] PLN</b> ";
 
    }
 
    print $content;
    ?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php 
$filename = 'kursy/'.date("Y-m-d", time()).'.txt';
$fp = fopen($filename, 'w'); 
$somecontent = "lew bułgarski	1 BGN	2,1655
peso chilijskie	100 CLP	0,6036
rupia indyjska	100 INR	6,1285\n";
 
if (is_writable($filename)) {
 
 
    if (!$handle = fopen($filename, 'a')) {
         echo "Nie można odtworzyc ($filename)";
         exit;
    }
 
    if (fwrite($handle, $somecontent) === FALSE) {
        echo "Nie można nadpisać ($filename)";
        exit;
    }
 
    echo "Zapisano ($somecontent) do ($filename)";
 
    fclose($handle);
 
} else {
    echo "Nie można zapisać $filename ";
}
?>
[PHP] pobierz, plaintext 

[b4rt3kk]

W jednym pliku masz ten cały kod? Jeśli tak to odradzam, dla zachowania porządku nie powinieneś umieszczać klas z funkcjami i samym wywołaniem.

Jak już zdecydowałeś się opakować wszystko w try...catch to pozostań konsekwentny i nie rób echo dla błędu, tylko wyrzuć nowy wyjątek, który przechwycisz w bloku i wyświetlisz.

Mechanizm cache jest zupełnie do niczego. Pobierasz też np. kilka razy zawartość url - marnotrastwo czasu i zasobów.

Generalnie kod jest mało czytelny i wymagałby nieco optymalizacji i uproszenia.

Jeszcze jedna uwaga:

[PHP] pobierz, plaintext 
} catch (Exception $e) {
echo 'Błąd przy wyświetlaniu kursów walut.';
}
[PHP] pobierz, plaintext 

Będziesz wiedział jaki błąd spowodował wyrzucenie wyjątku, skoro nawet nie wyświetlasz jego treści?

Ten post edytował b4rt3kk 3.02.2015, 12:07:49