Почему если источник при парсинге не работает, то сайт тоже не работает?

Question

[Justin Bieber]

Парсим курс валюты с нацбанка, вчера сайт нацбанка не работал и из-за этого перестает работать сайт который парсит.
Если например указать источник неправильно, то на сайте показывает по нулям. Почему если источник не был доступен, не показывало на сайте по нулям?

<?php  
     
    $sContent  = file_get_contents("http://www.nationalbank.kz/rss/rates_all.xml", "r");
    $oXml = simplexml_load_string($sContent);
    $iUSD = 0;
    $iRUR = 0;
    $iEUR = 0;
    foreach($oXml->channel->item AS $oValue) { 
        if ($oValue->title == "USD") { 
            $iUSD = $oValue->description; 
        } 
        if ($oValue->title == "RUB") { 
            $iRUB = $oValue->description; 
        } 
        if ($oValue->title == "EUR") { 
            $iEUR = $oValue->description; 
        } 
    } 
     
    echo "<span>USD " . $iUSD . "</span>";
    echo "<span>RUB " . $iRUB . "</span>";
    echo "<span>EUR " . $iEUR . "</span>";
?>

Answer 1

[Алексей Уколов]

Читайте логи, там всё написано.

Answer 2

[xmoonlight]

потому, что это находится в одном потоке, а там исполнение кода - всегда асинхронное.
1. парсите и кэшируйте.
2. обрабатывайте ошибки
3. используйте кэш при ошибках

Answer 3

[neol]

Потому что default_socket_timeout в PHP равен 60 (секундам), а максимальное время выполнения file_get_contents зависит от этого параметра. Соответственно если сайт, с которого вы дёргаете информацию, лежит, то запросы к вашему сайту возвращают ответ только после истечения этого таймаута.

Любой PHP код, взаимодействующий с сетью, желательно выносить в задачи, выполняемые через cli по cron (хотя не всегда это возможно, например при авторизации через сторонний сервис), а на сайт выводить кешированые результаты. При этом стоит делать проверки, что этот код не запустится 100500 раз.

Плюс к этому у вас нет проверок, что file_get_contents и simplexml_load_string возвращают корректный результат.

Answer 4

[Оптимус Пьян]

Завершается скрипт наверное когда данных нет.
Поставьте $sContent = @file_get_contents или в if оберните

Comments

[Justin Bieber]

$sContent = @file_get_contents стоит же

[neol]

А ещё можно зажмуриться. Тоже хороший метод решения проблем.

[Justin Bieber]

? )

[neol]

Justin Bieber: Зажмурившись вы не увидите проблемы и тем более не исправите её, как и при игнорировании ошибки. Так что эти способы почти равнозначны. Правда зажмуриться лучше, т. к. ошибку сможет увидеть кто-то другой, а игнорирование скорее всего не только не сделает лучше, но и сделает хуже.

[Justin Bieber]

neol: не напишите мне правильный парсер погоды и курса?

[neol]

Justin Bieber: Нет. Подсказать я ещё могу, а самому писать скучно.