[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]

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.

[Pyton_000]

@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 (IMG:style_emoticons/default/smile.gif)

[kalibovers]

takie ? (IMG:style_emoticons/default/tongue.gif) 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 

[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

[kalibovers]

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.

[b4rt3kk]

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ł".

[kalibovers]

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