looping through xml to exchange values of variables

member89

the xml file has only two entries in it. The php function works fine, reads the xml file and all the jazz. It takes the $filename, matches it to the xml attribute filename - then takes the caption from the corresponding tag and loads it into $title which is then loaded into $caption and viola it works. If it doesn't work then it returns a caption of no title (which also works btw).

My only problem right now is with the conditional statement inside the for loop. I have it set at a value of 2 because in the xml, there are two entries. However, when the page loads it loads the value of the second entry. If I change the value to 1, then it loads the first entry.

Any suggestions? Btw, I'm a flash developer learning php.

Below is my code:

$img = $filename;
include 'example.php';
	xml = new SimpleXMLElement($xmlstr);
		for ($i = 0; $i < 2; $i = $i + 1) {
			if ((string) $xml->image[$i]->filename == $filename) {
				foreach ($xml->image[$i]->title as $title) {
				}
			} else {
			$title = 'no title';
			}
	}
$caption = $title;

Here is the xml:

<images>
 <image>
     <filename>img_01.jpg</filename>
     <title type="caption">caption for image 1</title>
  </image>
   <image>
     <filename>img_02.jpg</filename>
     <title type="caption">caption for image 2</title>
  </image>
</images>

NogDog

I'm a bit confused as to what it is you want to accomplish. Can you explain what it is that you want to accomplish in a functional (i.e. non-coding-specific) what you want to happen? Assuming the sample XML is the input of this operation, what do you want the resulting output to be?

member89

I want to set $filename equal to the caption if the filename attribute and $filename contain the same image name and have it loop through all the nodes in the xml.

NogDog

$caption = 'no title';  // initialize to value to use if not found
foreach($xml->image as $image)
{
   if($image->filename == $filename)
   {
      $caption = $image->title;
      break;  // found it, so no need to keep looping
   }
}

member89

Thanks nogdog...

I had to modify it a bit once I added it to my script, nut now it works perfectly. I knew it had to be something simpler than what I thought.
Thanks again.

Weedpacket

You could also use an XPath query instead of the loop:

$caption = 'no title';

$images = $xml->xpath("/images/image[filename='$filename']/title");

if($images!==false) $caption = $images[0];

echo $caption;