Question

I want to update some entries in my database, basically I am counting the number of checkboxes that have been selected on the previous page and multiplying them with 25 then adding that value to the current value in the DB.

This is my code:

<?php
  if($_POST['code_approve'])
  {
    for($i=0;$i<count($_POST['checkbox']);$i++)
    {
      $approval_id = $checkbox[$i];
      $checkboxCount = count($_POST['checkbox']);
      $countx25 = $checkboxCount * 25;
      $sql = "UPDATE table SET status='approved', used='processed' WHERE id='$approval_id'";
      $sql2 = "UPDATE members SET balance = balance+'$countx25'";
      $result2 = mysql_query($sql2);
      $result = mysql_query($sql);
    }
    if($result)
    {
      echo "$countx25";
    }
   }
 ?>

It seems, that for some reason it is multiplying $countx25 with the number of checkboxes before inserting it into MySQL. This if($result){echo "$countx25";}} always shows me the right value though.

If i select 1 it prints 25, 2 prints 50, 3 prints 75 and so on, but for the MySQL part, if i select 1 it adds 25 to current value, 2 adds 100, 3 adds 225 ?!

What’s the error here ?

Answer 1

As I stated in my comment. You are running this for each checkbox you get via $_POST. Why do you even use the for loop if you use count to count the checkboxes. Remove the for loop and it will work as you intend it to.