PHP and MySQL using the PDO class

Question

I am trying to update a database, here is my code

if (isset($_GET['placeorder']))
{
    include 'includes/db.php';

    try
    {
    $newBalance = $_POST['balance'] + $_POST['oldbalance'];
    $sql = 'UPDATE customer SET
            balance = :balance
            WHERE id = :id';
    $s = $pdo->prepare($sql);
    $s->bindValue(':balance', $newBalance);
    $s->execute();
    }
    catch (PDOException $e)
    {
        $error = 'Could not add the new balance for the customer' . $e->getMessage();
        include  'result.php';
        exit(); 

    }
    header('Location: .');
    exit();

What I am trying to do is update the balance for a customer that is coming from a form that was submitted. I am able to get the value in the code all the way up to where I get to $s->execute(); if I try to echo the value, which is $newBalance, it will not show after that line is executed, the page goes blank. Something is happening in the execute statement. $s->execute() that it does not allow my code to proceed. Any idea? Am I using the PDO class the wrong way. It is not getting to the "catch" statement. Any help would be great. The end result is that the page returns to where it started with the updated balance.

Are you sure you've got error reporting on? Put error_reporting(E_ALL); at the top of your code — SomeKittens
– SomeKittens, Commented Oct 31, 2012 at 3:48

doublesharp · Accepted Answer · 2012-10-31 03:47:27Z

1

You are only binding a value to the balance, not the id, you need a line like:

$s->bindValue(':id', $id);

It would be a good idea to make sure that $_POST['balance'] and $_POST['oldbalance'] are set as well before using them in your query:

$balance = isset($_POST['balance'])? $_POST['balance'] : 0;
$oldbalance = isset($_POST['oldbalance'])? $_POST['oldbalance'] : 0;
$newbalance = $balance + $oldbalance;

If you aren't getting an error, you likely don't have error reporting on, you can enable it by adding error_reporting(E_ALL); to the top of your page.

answered Oct 31, 2012 at 3:47

doublesharp

27.9k6 gold badges56 silver badges76 bronze badges

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

1 Comment

Jose Carrillo Over a year ago

Thanks doublesharp, that was exactly my issue. I've been looking at this thing far too long. Thans for the help. I do have error reporting turned on, but not binding "id" was causing the issue. Thanks again.

swapnesh · Accepted Answer · 2012-10-31 03:46:24Z

0

var_dump() both variables $_POST['balance'] & $_POST['oldbalance']. I am sure they are coming as string. Type casting is one of the solution. Try typecasting in this case to perform addition on int/float.

$newBalance = (int)$_POST['balance'] + (int)$_POST['oldbalance'];

answered Oct 31, 2012 at 3:46

swapnesh

26.8k24 gold badges97 silver badges129 bronze badges

1 Comment

Jose Carrillo Over a year ago

Thanks, swampnesh, the issue was that I was not binding the id value. Thanks for taking the time.

Collectives™ on Stack Overflow

PHP and MySQL using the PDO class

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related