php parse error: unexpected $end [duplicate]

Question

I have some PHP code that takes a post from the index and then allows you to log in. When i run it it say:

parse error: unexpected $end;

If anyone has any ideas please let me know ASAP because need it for a class in school.

<?php

    $uname = $_POST["username"];

    $resultCount = 0;

    class MyDB extends SQLite3
   {
      function __construct()
      {
         $this->open('users.db');
      }
   }
   $db = new MyDB();
   if(!$db){
      echo $db->lastErrorMsg();
   }

    $sql =<<<EOF
    SELECT $uname from users;
    EOF;

    $ret = $db->exec($sql);

    foreach($ret as $uname){
            $resultCount++;
    }

    if($resultCount > 1){
        echo "failed to log in!";
        echo "please return <a href='index.php'>home</a>";
    }
    else{
        echo "logged in succesfully!";
        echo "welcome" . $_POST[username];
    }
?>

$_POST[username] (2nd to last line) should be $_POST["username"]. There might be more syntax errors. — Halcyon
– Halcyon, Commented Jun 5, 2015 at 12:33
halcyon i did this and i then said unexpected '"', expecting T_STRING or T_VARIABLE or T_NUM_STRING — andrew
– andrew, Commented Jun 5, 2015 at 12:35
Use $sql = "SELECT $uname from users"; instead. ATTENTION! Smells like SQL injection issue... — Dietmar
– Dietmar, Commented Jun 5, 2015 at 12:36
SELECT $uname from users; this is going to bite you. You probably want to select a column for a specific user and not a specific column. And the heredoc EOF; cannot have any spaces before it, it has to be at the beginning of the line. — jeroen
– jeroen, Commented Jun 5, 2015 at 12:38

Halcyon · Accepted Answer · 2015-06-05 12:46:11Z

3

There might be a problem here:

   $sql =<<<EOF
   SELECT $uname from users;
   EOF;

EOF; should be at the start of the line, there can be no whitespace before it.

See: https://php.net/manual/en/language.types.string.php#language.types.string.syntax.heredoc

That means especially that the identifier may not be indented

$_POST[username] (2nd to last line) should be $_POST["username"]. This might just be a warning though.

edited Jun 5, 2015 at 12:46

answered Jun 5, 2015 at 12:40

Halcyon

57.8k10 gold badges93 silver badges131 bronze badges

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

Comments

viewers.gq · Accepted Answer · 2015-06-05 12:42:17Z

0

change the brackets the other way and plus u dont have to do that just add to the top of your page this code

<? session_start(); ?>

and when you echo there username put

<?=$_SESSION['sess_user'];?> or <? echo $_SESSION['sess_user'];?>

or you could go into detail and put <? $rot = $username ?> <? echo $rot ?>

answered Jun 5, 2015 at 12:42

viewers.gq

218 bronze badges

6 Comments

viewers.gq Over a year ago

the $_POST method OFTEN carry's loads of errors. so i advise my advice! try it and let me know how it works.

jeroen Over a year ago

And how and where would you assign a value to $_SESSION['sess_user'] if the $_POST method OFTEN carry's loads of errors?

viewers.gq Over a year ago

that's a ridiculous question, here is the answer pal. $_POST method is automatic, $_SESSION is manual, therefor session works on its on as long as you have an actual SQL database, and as long as you have started the session, if using post then everything must be typed perfectly.. tens of pointless sentences of code.

viewers.gq Over a year ago

and also at the same place as the $_POST would be. of course.

viewers.gq Over a year ago

there isnt any value to an echo.. wtf r u talking about bro. do u even know how to code?

|

Collectives™ on Stack Overflow

php parse error: unexpected $end [duplicate]

2 Answers 2

Comments

6 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

6 Comments

Linked

Related