"Parse error: syntax error, unexpected T_CLASS" (PHP)

Jeroen1322 · February 11, 2015

Hi guys!

I am building a little CMS and i get this really stupid error: "Parse error: syntax error, unexpected T_CLASS on line 3".

The code for this file (simpleCMS.php) is:

<? php    class simpleCMS{        var $host;    var $username;    var $password;    var $table;        public function display_public() {            $q = "SELECT * FROM testDB ORDER BY created DESC LIMIT 3";            $r = mysql_query($q);        if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display .= <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                }        else {        $entry_display = <<<ENTRY_DISPLAY        <h2>This Page Is Under Construction</h2>        <p>            No entries have been made on this page.             Please check back soon, or click the            link below to add an entry!        </p>        ENTRY_DISPLAY;        }    $entry_display .= <<<ADMIN_OPTION        <p class="admin_link">            <a href="{$_SERVER['PHP_SELF']}?admin=1">Add a New Entry</a>        </p>    ADMIN_OPTION;        return $entry_display;}    public function display_admin() {            return <<<ADMIN_FORM                <form action="{$_SERVER['PHP_SELF']}" method="post">                    <label for="title">Titel:</label>                    <input name="title" id="title" type="text" maxlength="150" />                    <label for="bodytext">Bericht:</label>                    <textarea name="bodytext" id="bodytext"></textarea>                    <input type="submit" value="Verstuur!" />                </form>                ADMIN_FORM;}    public function write($p) {            if ( $p['title'] )            $title = mysql_real_escape_string($p['title']);            if ( $p['bodytext'])            $bodytext = mysql_real_escape_string($p['bodytext']);            if ( $title && $bodytext ) {            $created = time();            $sql = "INSERT INTO testDB VALUES('$title','$bodytext','$created')";            return mysql_query($sql);        }             else {                return false;                }}            public function connect() {            mysql_connect($this->host,$this->username,$this->password) or die("kon niet verbinden met de database. " . mysql_error());            mysql_select_db($this->table) or die("Weet je zeker dat die database wel bestaat?. " . mysql_error());            return $this->buildDB();}            private function buildDB() {            $sql = <<<MySQL_QUERY            CREATE TABLE IF NOT EXISTS testDB (                title	VARCHAR(150),                bodytext	TEXT,                created	VARCHAR(100)            )                MySQL_QUERY;            return mysql_query($sql);        }} ?>

And i have another file (display.php) and i get the same error. (i changed the login-credentials for obvious reasons) The code is:

<!DOCTYPE html><html lang="en">  <head>    <title>SimpleCMS</title>  </head>  <body><?php  include_once('simpleCMS.php');  $obj = new simpleCMS();  $obj->host = '#';  $obj->username = '#';  $obj->password = '#';  $obj->table = '#';  $obj->connect();  if ( $_POST )    $obj->write($_POST);  echo ( $_GET['admin'] == 1 ) ? $obj->display_admin() : $obj->display_public();?>  </body></html>

What seems to be causing the problem? I can't fingure it out!

Dredgy · February 11, 2015

"<? php"

Should be "<?php"

EDIT: There are a lot of errors scattered through that code.

Hazy125 · February 11, 2015

Well, you've come a long way since you posted on here last... I removed like 4 errors, but I seriously cannot get the final one- An end of file error. Everything looks to be fine. All brackets are closed, semicolons are there, no random quotes. You might have to play with this some more. I might just be missing something... Heredocs seriously screw with my brain for some reason... Anyways, commented code for your perusal:

<?php //As mentioned, fix the opening tag    class simpleCMS{        var $host;    var $username;    var $password;    var $table;        public function display_public() {            $q = "SELECT * FROM testDB ORDER BY created DESC LIMIT 3";            $r = mysql_query($q);        if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display = <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                }        else {        $entry_display = <<<ENTRY_DISPLAY        <h2>This Page Is Under Construction</h2>        <p>            No entries have been made on this page.             Please check back soon, or click the            link below to add an entry!        </p>        ENTRY_DISPLAY;        }    $entry_display .= <<<ADMIN_OPTION        <p class="admin_link">            <a href="{$_SERVER['PHP_SELF']}?admin=1">Add a New Entry</a>        </p>    ADMIN_OPTION;        return $entry_display;}/*== THE ENTRY DISPLAY FIXES ==In your if/else statement you were appending to $entry_display when entry_display might not existEXAMPLE: $entry_display .= <<<ENTRY_DISPLAY - At this point entry_display didn't exist, so this would throw an error. So I changed ".=" to just "=". You did get this right in the else though. Then under the if else statement $entry_display would have been initialised either way so it's fine to append ADMIN_OPTION there. Finally, you had white space after you closed your heredoc. This isn't allowed, so I just moved return $entry_display; up 1 line*/   public function display_admin() {            return <<<ADMIN_FORM                <form action="{$_SERVER['PHP_SELF']}" method="post">                    <label for="title">Titel:</label>                    <input name="title" id="title" type="text" maxlength="150" />                    <label for="bodytext">Bericht:</label>                    <textarea name="bodytext" id="bodytext"></textarea>                    <input type="submit" value="Verstuur!" />                </form>                ADMIN_FORM;}public function write($p) {            if ( $p[title] )//removed single quotes from all of these... Trust me            $title = mysql_real_escape_string($p[title]);            if ( $p[bodytext] )            $bodytext = mysql_real_escape_string($p[bodytext]);            if ( $title && $bodytext ) {            $created = time();            $sql = "INSERT INTO testDB VALUES('$title','$bodytext','$created')";            return mysql_query($sql);        }             else {                return false;                }}            public function connect() {            mysql_connect($this->host,$this->username,$this->password) or die("kon niet verbinden met de database. " . mysql_error());            mysql_select_db($this->table) or die("Weet je zeker dat die database wel bestaat?. " . mysql_error());            return $this->buildDB();}            private function buildDB() {            $sql = <<<MySQL_QUERY            CREATE TABLE IF NOT EXISTS testDB (                title	VARCHAR(150),                bodytext	TEXT,                created	VARCHAR(100)            )                MySQL_QUERY;            return mysql_query($sql);//Again, moved this up a line to removed whitespace after heredoc        }}?>

WolfDeville · February 11, 2015

Hazy125, on 11 Feb 2015 - 10:32 PM, said:Hazy125, on 11 Feb 2015 - 10:32 PM, said:Hazy125, on 11 Feb 2015 - 10:32 PM, said:Hazy125, on 11 Feb 2015 - 10:32 PM, said:

Well, you've come a long way since you posted on here last... I removed like 4 errors, but I seriously cannot get the final one- An end of file error. Everything looks to be fine. All brackets are closed, semicolons are there, no random quotes. You might have to play with this some more. I might just be missing something... Heredocs seriously screw with my brain for some reason... Anyways, commented code for your perusal:

<?php //As mentioned, fix the opening tag    class simpleCMS{        var $host;    var $username;    var $password;    var $table;        public function display_public() {            $q = "SELECT * FROM testDB ORDER BY created DESC LIMIT 3";            $r = mysql_query($q);        if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display = <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                }        else {        $entry_display = <<<ENTRY_DISPLAY        <h2>This Page Is Under Construction</h2>        <p>            No entries have been made on this page.             Please check back soon, or click the            link below to add an entry!        </p>        ENTRY_DISPLAY;        }    $entry_display .= <<<ADMIN_OPTION        <p class="admin_link">            <a href="{$_SERVER['PHP_SELF']}?admin=1">Add a New Entry</a>        </p>    ADMIN_OPTION;        return $entry_display;}/*== THE ENTRY DISPLAY FIXES ==In your if/else statement you were appending to $entry_display when entry_display might not existEXAMPLE: $entry_display .= <<<ENTRY_DISPLAY - At this point entry_display didn't exist, so this would throw an error. So I changed ".=" to just "=". You did get this right in the else though. Then under the if else statement $entry_display would have been initialised either way so it's fine to append ADMIN_OPTION there. Finally, you had white space after you closed your heredoc. This isn't allowed, so I just moved return $entry_display; up 1 line*/   public function display_admin() {            return <<<ADMIN_FORM                <form action="{$_SERVER['PHP_SELF']}" method="post">                    <label for="title">Titel:</label>                    <input name="title" id="title" type="text" maxlength="150" />                    <label for="bodytext">Bericht:</label>                    <textarea name="bodytext" id="bodytext"></textarea>                    <input type="submit" value="Verstuur!" />                </form>                ADMIN_FORM;}public function write($p) {            if ( $p[title] )//removed single quotes from all of these... Trust me            $title = mysql_real_escape_string($p[title]);            if ( $p[bodytext] )            $bodytext = mysql_real_escape_string($p[bodytext]);            if ( $title && $bodytext ) {            $created = time();            $sql = "INSERT INTO testDB VALUES('$title','$bodytext','$created')";            return mysql_query($sql);        }             else {                return false;                }}            public function connect() {            mysql_connect($this->host,$this->username,$this->password) or die("kon niet verbinden met de database. " . mysql_error());            mysql_select_db($this->table) or die("Weet je zeker dat die database wel bestaat?. " . mysql_error());            return $this->buildDB();}            private function buildDB() {            $sql = <<<MySQL_QUERY            CREATE TABLE IF NOT EXISTS testDB (                title	VARCHAR(150),                bodytext	TEXT,                created	VARCHAR(100)            )                MySQL_QUERY;            return mysql_query($sql);//Again, moved this up a line to removed whitespace after heredoc        }}?>

With C# an "end of file" error usually (in my experience anyway) means that there are too many brackets. Try removing one bracket from the last line (doesn't look right but give it a try)

Edit: Try changing:

 if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display .= <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                }        else {        $entry_display = <<<ENTRY_DISPLAY

to:

 if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display .= <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                    else {        $entry_display = <<<ENTRY_DISPLAY

Hazy125 · February 11, 2015

With C# an "end of file" error usually (in my experience anyway) means that there are too many brackets. Try removing one bracket from the last line (doesn't look right but give it a try)

Edit: Try changing:

 if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display .= <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                }        else {        $entry_display = <<<ENTRY_DISPLAY

to:

 if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display .= <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>        ENTRY_DISPLAY;      }                    else {        $entry_display = <<<ENTRY_DISPLAY

Nah, that won't work. Leaves the if statement open. The problem is the heredocs, that is, all of the things that have "<<<" in front of them. They're a special way to write strings in php that passes all variables, and quotations so that you don't have to worry about escaping quotes and concatenation of variables.

@Jeroen1322 I did fix it, but the solution is less than desirable, I replace all heredocs with just double quotes and manually escaped the required things. Not really elegant, but as mentioned above, heredocs do my head in so I couldn't really find an easier solution

Ciccioo · February 11, 2015

when you enter string literals with the syntax

$str = <<<IDho ho hoID;

the final ID; must be alone in the line. you cannot indent it

SSL · February 11, 2015

Instead of monkeying with HEREDOC, an exercise in despair, just use separate template files and include them.

Blade of Grass · February 12, 2015

Building on Hazys fixes.

<?php //As mentioned, fix the opening tag    class simpleCMS{        var $host;    var $username;    var $password;    var $table;        public function display_public() {            $q = "SELECT * FROM testDB ORDER BY created DESC LIMIT 3";            $r = mysql_query($q);        if ( $r !== false && mysql_num_rows($r) > 0 ) {        while ( $a = mysql_fetch_assoc($r) ) {            $title = stripslashes($a['title']);            $bodytext = stripslashes($a['bodytext']);            $entry_display = <<<ENTRY_DISPLAY            <h2>$title</h2>            <p>                $bodytext            </p>ENTRY_DISPLAY; // Do not indent this      }                }        else {        $entry_display = <<<ENTRY_DISPLAY        <h2>This Page Is Under Construction</h2>        <p>            No entries have been made on this page.             Please check back soon, or click the            link below to add an entry!        </p>ENTRY_DISPLAY; // Do not indent this        }    $entry_display .= <<<ADMIN_OPTION        <p class="admin_link">            <a href="{$_SERVER['PHP_SELF']}?admin=1">Add a New Entry</a>        </p>ADMIN_OPTION; // Do not indent this        return $entry_display;}/*== THE ENTRY DISPLAY FIXES ==In your if/else statement you were appending to $entry_display when entry_display might not existEXAMPLE: $entry_display .= <<<ENTRY_DISPLAY - At this point entry_display didn't exist, so this would throw an error. So I changed ".=" to just "=". You did get this right in the else though. Then under the if else statement $entry_display would have been initialised either way so it's fine to append ADMIN_OPTION there. Finally, you had white space after you closed your heredoc. This isn't allowed, so I just moved return $entry_display; up 1 line*/   public function display_admin() {            return <<<ADMIN_FORM                <form action="{$_SERVER['PHP_SELF']}" method="post">                    <label for="title">Titel:</label>                    <input name="title" id="title" type="text" maxlength="150" />                    <label for="bodytext">Bericht:</label>                    <textarea name="bodytext" id="bodytext"></textarea>                    <input type="submit" value="Verstuur!" />                </form>ADMIN_FORM; // Do not indent this}public function write($p) {            if ( $p[title] )//removed single quotes from all of these... Trust me            $title = mysql_real_escape_string($p[title]);            if ( $p[bodytext] )            $bodytext = mysql_real_escape_string($p[bodytext]);            if ( $title && $bodytext ) {            $created = time();            $sql = "INSERT INTO testDB VALUES('$title','$bodytext','$created')";            return mysql_query($sql);        }             else {                return false;                }}            public function connect() {            mysql_connect($this->host,$this->username,$this->password) or die("kon niet verbinden met de database. " . mysql_error());            mysql_select_db($this->table) or die("Weet je zeker dat die database wel bestaat?. " . mysql_error());            return $this->buildDB();}            private function buildDB() {            $sql = <<<MySQL_QUERY            CREATE TABLE IF NOT EXISTS testDB (                title	VARCHAR(150),                bodytext	TEXT,                created	VARCHAR(100)            )MySQL_QUERY; // Do not indent this            return mysql_query($sql);//Again, moved this up a line to removed whitespace after heredoc        }}?>

Not a fan of HEREDOCs either, but I fixed your indentation, so they shouldn't cause an issue anymore.

Sign In

"Parse error: syntax error, unexpected T_CLASS" (PHP)

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Link to comment

Share on other sites

Link to post

Share on other sites

Create an account or sign in to comment

Create an account

Sign in

Featured Topics

Topics

Latest From Linus Tech Tips:

I Was Never Meant to Have This Prototype CPU

Latest From Tech Quickie:

Why Do Speakers Hiss?

Latest From TechLinked:

Intel: “It Wasn’t Me”

Latest From GameLinked:

Is Nintendo being FRAMED?

Latest From ShortCircuit:

How is this even handheld?! - OneXPlayer X1

Latest From Mac Address:

Why did you buy an Apple Vision Pro?

Latest From Channel Super Fun:

I Swapped the CEO's Assistant For a Day!