I'm basicly having trouble collecting xml data from a page and displaying it how I want it to be.
What the code should do:
Using Curl the code should "grab" the xml data from a specified URL.
It should then "navigate" to <SONGHISTORY><SONGS> where the two key pairs (PLAYEDAT & TITLE) are located.
From there it should allow me to output as many results (up to 10) or as little (no lower than 1) as I need.
What the code is actually doing:
The code connects and grabs the data fine.
The problem is, it will only output one pair ie played and title.
What I have done and what I need:
With help, I have determined that the reason for this is because I am asigning all the results into one variable, therefore it will only echo out one set of results.
What I can't work out is how to remidy this.
I also need to work out how to define constant variables instead of using global.
I just don't understand the syntax
I have variables defined in a config file like, $host, $port, $password but the only way I can use them is by using global which I am told is wrong.
The Code:
This is the code Im having trouble with.
Firstly the way I summon my code (which is actually a part built class) and display the results I require in index.php
Some of the variables (as I said earlier) are taken from config.php
Code: Select all
$simpleplaying = new simple();
$simpleplaying = $simpleplaying->simplestats();
if($error == 1)
{
echo '<div id="playing"><span class="status_red"> ' . $error_message1. '</span></div>';
}
else
{
echo '<div id="playing"><span class="title">Now Playing: </span><span class="text"> '.
$simple[6].'</span></div>'."\n";
echo '<div id="listen_link">-<span class="text"><a href="'. $listen_link .'"
class="listen_link">Click To Listen</a></span></div>';
}
Code: Select all
class advanced {
function advancedstats()
{
date_default_timezone_set('Europe/London');
//Globals Required
global $host, $port, $admin_password, $error, $played, $title;
//Get Advanced Statistics
$advancedurl = "http://".$host.":".$port. "/admin.cgi?pass=".$admin_password.
"&mode=viewxml&page=0";
$options = array(
CURLOPT_RETURNTRANSFER => true, // return web page
CURLOPT_HEADER => false, // don't return headers
CURLOPT_FOLLOWLOCATION => false, // follow redirects
CURLOPT_ENCODING => "", // handle all encodings
CURLOPT_USERAGENT => "Mozilla/5.0", // who am i
CURLOPT_AUTOREFERER => false, // set referer on redirect
CURLOPT_CONNECTTIMEOUT => 2, // timeout on connect
CURLOPT_TIMEOUT => 2, // timeout on response
CURLOPT_MAXREDIRS => 0, // stop after 10 redirects
);
$ch = curl_init( $advancedurl );
curl_setopt_array( $ch, $options );
$advsts = curl_exec( $ch );
$err = curl_errno( $ch );
curl_close( $ch );
if(!$err)
{
$xml = new SimpleXmlElement($advsts);
$i=0;
foreach ($xml->xpath('///SONGHISTORY') as $songhistory)
{
foreach($songhistory->SONG as $songs)
{
$i++;
$songs->PLAYEDAT = date("G:i", (int) $songs->PLAYEDAT);
$played = $songs->PLAYEDAT;
$title = $songs->TITLE;
if($i >= $lastplayed)
break;
}
}
}
else
{
$error = 1;
}
}
}
Code: Select all
$songs->PLAYEDAT = date("G:i", (int) $songs->PLAYEDAT);
Also thought I would add that the reason I made this a class is because I am going to be handeling the data it brings back and displaying it in lots of diferent ways, so it was suggested I used a class and pass the data to various functions within that class which would then handle it how I wanted it so I could display it how I wanted it. If that makes sense?
Thanks for looking guys, I hope someone knowledgable can lend a hand.
I've trawled google for hours but when you don't know the technical name for the problems you're having, it's hard finding solutions.
And some of the examples on the net are not much help if you can't work out how they apply to what you are trying to do when you are only a novice lol.
Cheers again!
PS, I just thought I would add, that under normal circumstances I am loving the problem solving aspect of coding and rarely like to let something beat me, but sometimes you just have to know when to say enough is enough and ask for some help.
That time is usually when the words escape and caps lock are embossed in reverse into your forehead lolz.