PHP foreach loop not reading values from array

Question

I have simple PHP array with data from checkbox. I need add values into array and then insert data into database. It works but foreach not infringement parameter.

So i testing with increment:

$arr = array();
array_push($arr, $_POST['chbox']);

and it looks like 123,125 in array (two elements) Next step is to return number of elements (or values in next step):

$id=0;
foreach(  $arr as $row)
{
    $id++;                              
};

and returns $id=1;

if i'm trying read values:

foreach(  $arr as $row)
{
    $row[$id]   
    $id++;
};

Return only 123

This is so weird. How would you know that $row[$id] returns that? There is no echo. Also, if you just want to turn string keys to numeric, take a look at array_values. — FirstOne
– FirstOne, Commented Mar 4, 2016 at 17:14
Wow, a lot of stabs at this. Right after your array_push, do a var_dump($arr); - what do you get? Then a foreach should be simple: foreach($arr AS $i => $row) {echo $row[$i];}. NOTE - if the $_POST is a checkbox, AND the checkbox is NOT checked, then that $_POST key will be EMPTY, and will not actually return anything. — random_user_name
– random_user_name, Commented Mar 4, 2016 at 17:15

David Nguyen · Accepted Answer · 2016-03-04 17:24:52Z

2

If you are doing a foreach, $row is the value already.

foreach($arr as $row) {
    echo $row;
    $id++;
}

edited Mar 4, 2016 at 17:24

answered Mar 4, 2016 at 17:04

David Nguyen

8,5182 gold badges35 silver badges50 bronze badges

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

1 Comment

RomanPerekhrest Over a year ago

in php, a loop "body" should not end up with semicolon

Patrick Moore · Accepted Answer · 2016-03-04 17:02:39Z

1

In your foreach() loop, $row is just one array value. Not the array. Replace with $arr should solve.

$id = 0;
foreach( $arr as $row ){
    echo $arr[$id];
    $id++;
}
echo 'Total items: ' . $id; // OR count( $arr );

answered Mar 4, 2016 at 17:02

Patrick Moore

13.4k5 gold badges41 silver badges63 bronze badges

2 Comments

FirstOne Over a year ago

Why would you loop with foreach if you are using the key ($id) like that? xD

Patrick Moore Over a year ago

This is how the OP structured the loop. I'm just correcting his variable mistake.

Nimatullah Razmjo · Accepted Answer · 2016-03-04 17:16:28Z

1

$arr =array(123,125);

foreach($arr as $arrr):
   echo $arrr.',';
endforeach;

Output will be:

123,125,

answered Mar 4, 2016 at 17:16

Nimatullah Razmjo

1,9615 gold badges23 silver badges42 bronze badges

2 Comments

random_user_name Over a year ago

Why the foreach convention here? Curly-braces are fairly standard - the format you are using is more typical of php embedded in a template.

Nimatullah Razmjo Over a year ago

You use Curly-braces. I just provide example, and It become habit for me to use Colon and semicolon Instead of Curly-braces. :)

Maytham Fahmi · Accepted Answer · 2016-03-04 17:24:06Z

1

Just try following

foreach($arr as $row){
     echo $row;
};

Edit 1: Better use var_dump($_POST["chkbox"]) or print_r($_POST["chkbox"]) to see the array you are getting. Then it will be easier for you to decide how to get data.

edited Mar 4, 2016 at 17:24

Maytham Fahmi

33.2k17 gold badges132 silver badges159 bronze badges

answered Mar 4, 2016 at 17:04

Ujjal Suttra Dhar

8468 silver badges12 bronze badges

1 Comment

Ujjal Suttra Dhar Over a year ago

Can you please explain what is wrong here? For you: pastebin.com/x5aSHMrp

Collectives™ on Stack Overflow

PHP foreach loop not reading values from array

4 Answers 4

1 Comment

2 Comments

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

2 Comments

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related