XML Fatal Error:

Hi Leute,
ich habe folgenden Code der bei einer vorhandenen POST Variable ausgeführt wird und einen Wert in einer XML Datei ändern soll:

PHP-Code:

  $questID = $_POST['voting'];
  $filename = 'abstimmung.xml';
  if(file_exists($filename)) {
    $xml = simplexml_load_file($filename);
    if($xml) {
      if($questID == 1){
        $count = $xml->question[$questID]->answer->attributes()->count; // Line 15
        $count = $count + 1;
        unset($xml->question[$questID]->answer->attributes()->count);
        $xml->question[$questID]->answer->addAttribute('count', $count);
        $xml->asXML($filename);
      } 
    }
  } 


Die XML Datei sieht so aus:

<?xml version="1.0" encoding="windows-1250"?>
<root>

  <question id="1">
    <answer count="0">testtesttest</answer>
  </question>

  <question id="2">
    <answer count="0">testtesttest</answer>
  </question>

  <question id="3">
    <answer count="0">testtesttest</answer>
  </question>
  
</root>

Wenn ich nun das Script ausführen lasse kommt folgende Fehlermeldung:

Fatal error:  Call to a member function attributes() on a non-object in 
C:\xampp\htdocs\test2\abstimmung.php on line 15

Was ich nicht verstehe ist warum er sagt das es sich dabei um ein non-object handelt. Bei einer ausgabe mit print_r wird es aber als simpleXML Object angezeigt.

Sorry aber ich weiß echt nicht weiter. Bitte um eure Hilfe!

MfG
Cappo

KA, welches Verhalten du erwartest, aber wenn $_POST['voting'] auf 1 steht, wird <question id="2"> <answer count="..."> erhöht.

Ansonsten passiert nix. Keine Fehlermeldung keine Erhöhung. egal was in $_POST['voting'] steht.

Hallo Cappo,

genau genommen, ist schon $xml->question[$questID] ein non-Object, was ja noch nicht weiter schlimm ist. Dann holst du davon die Property answer, was aber lediglich eine Notice hervorruft. Daher der Tipp, dein Error-Reporting auch auf Notices auszudehnen.

Was dann aber ein richtiger Fehler ist, ist das Aufrufen einer Funktion (attributes()) von einem undefinierten Wert, was aber nicht der eigentliche Fehler ist, sondern die logische Konsequenz aus dem vorherigen.

Woran liegt es aber?

Get-, Post- und sonstige Eingabedaten sind bei PHP immer Strings. SimpleXML mimen zwar normale Objekte und Arrays, haben aber auch Unterschiede. Einer davon ist, dass man Array-Werte zwar über $array["1"] oder $array[1] ansprechen kann, die SimpleXML-PseudoArrays (z. B. $xml->question) aber zwingend einen numerischen Index erfordern. Wenn du also deine Post-Variable zu int umwandelst, sollte es das schon gewesen sein.

Übrigens: ich finde SimpleXML ist alles andere als simpel. Es sieht zwar erstmal so aus, aber am Ende stellt es sich als dreckiger Hack raus, der zu "magisch" ist, um in sauberem Code verwendet zu werden. Ich rate zu DOM und XPath.

Gruß,

Amica

Ich rate zu DOM und XPath.

Xpath ist wohl der richtige Ansatz
zumindest kommt man so leichter zur richtigen id.

PHP-Code:

<?php
error_reporting(-1);
ini_set('display_errors', TRUE);

   

$questID  = 1;
//  $questID = $_POST['voting'];
$filename = 'abstimmung.xml';
$xml      = simplexml_load_file($filename);
$elements = $xml->xpath("/root/question[@id='$questID']/answer");
$node     = $elements[0];
$count    = $node->attributes()->count + 1;
$node->attributes()->count = $count;
$xml->asXML($filename);


?>