PHP SimpleXML Undefined Member Error

Question

I can't figure out why this is an error. Everything makes sense to me. $logFile is a SimpleXMLElement and theirfore should be able to use the getName() method.

Error:

Fatal error: Call to a member function getName() on a non-object in C:\xampp\htdocs\objLogParser.php on line 29

I've marked the error in my code.

<?php
class objLogParser
{
    private $fileName;
    private $logFile;

    //Constructor
    public function __construct($varFileName)
    {
        $this->fileName = $varFileName;
        //Load as string
        $xmlstr = file_get_contents($varFileName);
        $logFile = new SimpleXMLElement($xmlstr);

        //Load as file
        $logFile = new SimpleXMLElement($varFileName,null,true);
    }

    public function printNodes()
    {

        $this->printHelper($this->logFile,0);
    }


    public function printHelper($currentNode, $offset)
    {

        echo $this->offset($offset);
        echo $currentNode->getName();  ////////////////////LINE 29 ERROR
        if($currentNode->attributes()->count() > 0)
            echo "({$currentNode->attributes()})";
        echo " {$currentNode}"; 

        foreach ($currentNode->children() as $child) {
            echo "<br>";
            printHelper($child, ($offset+1));
        }
    }

    function offset($offset)
    {
        for ($i = 0; $i < $offset; $i++)
            echo "_ ";
    }



}
?>

nickjyc · Accepted Answer · 2011-10-11 23:57:47Z

1

It should be $this->logFile->getName();. Essentially, replace $logFile with logFile.

answered Oct 11, 2011 at 23:57

nickjyc

1286 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

RedLeader Over a year ago

Thanks for the post. Made the change, updated error, still remains.

nickjyc Over a year ago

The new error message is saying that $currentNode is not an object. Judging by the code that you gave, I assume that you meant to put $this->logFile instead of $logFile in your __construct function.

Matthew Turland · Accepted Answer · 2011-10-12 01:45:18Z

1

$this->logFile is never being set. $logFile is set in __construct(), but is never actually used. $this->logFile and $logFile aren't the same variable. $logFile is limited to the scope of __construct(); you can't access it outside that method. $this->logFile can be accessed by any method in the class. Try changing $logFile in __construct() to $this->logFile.

answered Oct 12, 2011 at 1:45

Matthew Turland

7633 silver badges11 bronze badges

1 Comment

RedLeader Over a year ago

Yes that did it, I also had some more similar issues without the $this in front of it. Thanks so much.

Collectives™ on Stack Overflow

PHP SimpleXML Undefined Member Error

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related