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 

Proponuję skorzystać z tego linku:
http://www.nbp.pl/kursy/xml/a021z150202.xml

Łatwiej sobie przetworzyć xml, niż tak jak teraz to robisz poprzez preg_match.

@b4rt3kk czy Ty myślisz że Autor sam to pisał? Jak nie wie co do czego w tym to nie licz że zrobi sobie parsowanie XML'a

takie ? i moje pytanie czy da sie skrocic czy wszystkie tutaj rzeczy sa dobrze ?

[PHP] pobierz, plaintext 
<?php
 
function convert($text) {
    $charset = '';
 
    if ($charset && function_exists('iconv')) {
        return iconv('utf-8', $charset, $text);
    } elseif ($charset && function_exists('recode_string')) {
        return recode_string('utf8...' . $charset, $text);
    } else {
        return $text;
    }
}
 
class kursy {
 
    private $contents = '';
 
    function __construct($url, $cache = 'kursy_cache.txt', $lastupdate = '12:16 -1 day', $thisupdate = '12:16') {
        $recent = TRUE;
        $lastupdate = strtotime($lastupdate);
        $thisupdate = strtotime($thisupdate);
 
        if ((!file_exists($cache) AND ! is_writable(dirname($cache)) )
                OR ( file_exists($cache) AND ! (is_writable($cache)) )) {
 
            $cache = '';
        } else {
 
            if (@filemtime($cache) < $lastupdate) {
                $recent = FALSE;
            } elseif (time() > $thisupdate && @filemtime($cache) < $thisupdate) {
                $recent = FALSE;
            }
        }
 
 
        if ($cache == '' OR ! $recent) {
 
            $this->contents = file_get_contents($url);
            if ($this->contents == FALSE) {
                throw new Exception('Nie udało się pobrać kursów walut.');
            }
 
 
            if ($cache != '') {
 
                file_put_contents($cache, $this->contents);
            }
        } else {
 
            $this->contents = file_get_contents($cache);
        }
    }
 
    function znajdz($fields) {
        if (!is_array($fields)) {
            $fields = array($fields);
        }
 
        $last = libxml_use_internal_errors(TRUE);
        $info = new SimpleXMLElement($this->contents);
        libxml_use_internal_errors($last);
 
 
        $rates = array(
            'numer_tabeli' => (string) $info->numer_tabeli,
            'data_publikacji' => (string) $info->data_publikacji
        );
 
        foreach ($info->pozycja as $v) {
            $kod = (string) $v->kod_waluty;
            $rates[$kod] = array(
                'nazwa' => convert((string) $v->nazwa_waluty),
                'ilosc' => (string) $v->przelicznik
            );
            foreach ($fields as $field) {
                $rates[$kod][$field] = (string) $v->$field;
            };
        }
 
        return $rates;
    }
 
}
 
try {
 
    $kursy = new kursy('http://nbp.pl/kursy/xml/LastA.xml', 'kursy_cache.txt', '12:16 -1 day', '12:16');
    $waluta = $kursy->znajdz(array('kurs_sredni'));
    $filename = 'kursy/' . date("Y-m-d", time()) . '.txt';
    $fp = fopen($filename, 'w');
    $somecontent = $waluta['BGN']['ilosc'] . ' BGN: ' . $waluta['BGN']['kurs_sredni'] . '<br />
' . $waluta['CLP']['ilosc'] . ' CLP: ' . $waluta['CLP']['kurs_sredni'] . '<br />
' . $waluta['INR']['ilosc'] . ' INR: ' . $waluta['INR']['kurs_sredni'] . '
';
    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 ";
    }
} catch (Exception $e) {
    echo 'Błąd przy wyświetlaniu kursów walut.';
}
?> 
[PHP] pobierz, plaintext 

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?

No własnie musze go uprościc i zostawić tylko zapis waluty do pliku a co do tego echo co podałeś to ni jest ono potrzebne.

Są takie przypadki kiedy warto wiedzieć, dlaczego skrypt się "wysypał".

Ten moj działa tylko wydaje mi sie, ze sa funkcje ktore sa nie potrzebne i naewt o polowe dalo by sie to skrocic ale nie wiem jak ;/

Powie ktoś jak moge skrocic ten kod? tak aby zostały tylko funkcje:
Tworzenie pliku z data i zapis walut do tego pliku ze strony nbp