Problem in using UPDATE command in PHP

Question

I am have some problem in updating my qotwVote1a table's Vote1a field through PHP. Could you please have a look at the code, and tell me what am i doing wrong in here.

$result = mysql_query("SELECT * FROM qotwVote1a WHERE QuestionId='".$questionId."' AND MemberId='".$id."'");

    while($row = mysql_fetch_array($result))
    {
        $originalVote=$row['Vote1a'];
        $newVote=$originalVote + $vote;
        //echo ($newVote);
    }

$sql = <<<END
UPDATE qotwVote1a
SET Vote1a = '$newVote',
WHERE QuestionId = '$questionId' AND MemberId = '$id'
END;

mysql_query($sql);
if (mysql_error()) {
  die("Error executing query '$sql': " . mysql_error());
}

Using this code I got an error:

"Error executing query 'UPDATE qotwVote1a SET Vote1a = '2', WHERE QuestionId = '57' AND MemberId = 'zee'': You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE QuestionId = '57' AND MemberId = 'zee'' at line 3"

Regards Zeeshan

Eric · Accepted Answer · 2009-07-15 19:51:47Z

5

You have a comma after $newVote. Remove it, and you'll be peachy-keen.

Also, you don't need to wrap numbers in quotation marks, and don't do it if your column is an integer or float type. Doing so just causes those to get converted to numbers, anyway, so it's not a big deal.

UPDATE qotwVote1a
SET Vote1a = '$newVote'
WHERE QuestionId = '$questionId' AND MemberId = '$id'

answered Jul 15, 2009 at 19:51

Eric

95.7k13 gold badges117 silver badges116 bronze badges

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

Comments

Gumbo · Accepted Answer · 2009-07-15 19:52:25Z

0

There’s a comma in you MySQL query after the SET clause that’s misplaced. So try this:

$sql = <<<END
UPDATE qotwVote1a
SET Vote1a = '$newVote'
WHERE QuestionId = '$questionId' AND MemberId = '$id'
END;

answered Jul 15, 2009 at 19:52

Gumbo

657k112 gold badges792 silver badges852 bronze badges

Comments

MacAnthony · Accepted Answer · 2009-07-15 19:52:57Z

0

It's looks like you are missing some code as the query supplied is not the query giving the error. The problem in the query is an extra comma after the "Vote1a = '2'" statement though.

answered Jul 15, 2009 at 19:52

MacAnthony

4,5212 gold badges25 silver badges27 bronze badges

Collectives™ on Stack Overflow

Problem in using UPDATE command in PHP

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related