Попытка создать высокую сплоченность в подаче XML RSS - PullRequest
Новая
       4

Попытка создать высокую сплоченность в подаче XML RSS

0 голосов
/ 15 декабря 2011

Я пытаюсь повысить читабельность моего кода, создавая дополнительные функции, но разделение функции get_current_weather_data () вызывает следующую ошибку: 

Fatal error: Call to a member function children() on a non-object in C:\xampp\htdocs\Twinz\src\weather.php on line 37

Строка 37 должна быть последней фигурной скобкой после возвратаПогода;и эта ошибка произошла только после создания отдельной функции для get_city ().Я могу переместить это достаточно легко, но я хочу знать:

Разве нельзя разделить эту функцию и каково общее мнение.Что я пытаюсь сделать бессмысленно?

Эта функция загружает погодный фид Yahoo XML, который я обрабатываю и выводю необходимые данные;что происходит, город, температура и текущие условия.

<?php

function get_current_weather_data() {
// Get XML data from source
$feed = file_get_contents("http://weather.yahooapis.com/forecastrss?p=USNY0996&u=f");

// Check to ensure the feed exists
if (!$feed) {
    die('Weather not found! Check feed URL');
}

$xml = new SimpleXmlElement($feed);

get_city(); 

// Pull temperature 
$weather['temp'] = $xml->channel->item->children('yweather', TRUE)->condition->attributes()->temp;
echo $weather['temp'] . "<br />";

// Pull current conditions 
$weather['conditions'] = $xml->channel->item->children('yweather', TRUE)->condition->attributes()->text;
echo $weather['conditions'] . "<br />";

return $weather;
}

Функция get_city (): 

function get_city() {
// Pull city 
$weather['city'] = $xml->channel->children('yweather', TRUE)->location->attributes()->city;
echo $weather['city'] . "<br />";

}

1 Ответ

1 голос
/ 16 декабря 2011

Это простая проблема определения объема и разработки кода, которая не имеет ничего общего с SimpleXML или XML.

Вы используете $xml и $weather в своей функции get_city(), но не имеете доступа клюбая переменная $xml или $weather, и никакой способ вернуть эти данные обратно!

Перепишите так: 

function get_city(SimpleXMLElement $xml) {
    // use your $xml to get the city value
    return $city;
}
// and in your get_current_weather_data() function, use like so:

$weather = array();
$weather['city'] = get_city($xml);
...