PHP unset Not Working as expected

Question

I am simply trying to remove all of the Array objects that have 'visible' set to '0'

Array:

{
"Count":5,
"0":{"id":"1","visible":"0"},
"1":{"id":"3","visible":"0"},
"2":{"id":"1","visible":"0"},
"3":{"id":"2","visible":"0"},
"4":{"id":"3","visible":"0"}
}

PHP:

function cleanup($arr) {
    for($i = 0; $i < (count($arr)-1); $i++) {
        if($arr[$i]['visible'] == false) {
            unset($arr[$i]);
        }
    }
    $newarr = array_unique($arr, SORT_REGULAR);
    $newarr['Count'] = count($newarr)-1;

    return $newarr;
}

Result:

{
"Count":2,
"3":{"id":"2","visible":"0"},
"4":{"id":"3","visible":"0"}
}

In my mind this should work and return {"Count":0}. Also Why have the 'keys' not been set to 0,1 instead of 3,4. Where am i going wrong?

Cyclotron · Accepted Answer · 2013-10-05 00:05:57Z

2

You are using count($arr)-1) inside the for loop, and it gets re-evaluated every iteration, so after you unset the first three times, i is 3, but count($arr)-1) is 1, and you exit the loop. You should set $j=count($arr)-1 before the for loop, and use for($i = 0; $i < $j; $i++)

In general it is bad programming practice (performance-wise) to use functions like count() inside the for loop

edited Oct 5, 2013 at 0:05

answered Oct 4, 2013 at 23:52

Cyclotron

3393 silver badges11 bronze badges

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

2 Comments

ChristopherStrydom Over a year ago

Thank you very much for the tips! This seems to have worked.. Even though i did have it outside of the for loop before :S

web2students.com Over a year ago

Why don't you unset in one go? unset($arr);

hek2mgl · Accepted Answer · 2013-10-04 23:46:17Z

1

unset() will not reorder the array indexes if you removing an index from the middle of an numerical array. You need to reindex the array by yourself. array_values() is helpful here.

function cleanup($arr) {
    for($i = 0; $i < (count($arr)-1); $i++) {
        if($arr[$i]['visible'] == false) {
            unset($arr[$i]);
        }
    }
    $newarr = array_values(array_unique($arr, SORT_REGULAR));
    return $newarr;
}

The Count property makes no sense for me therefore I dropped it away. You may use the function count() instead.

answered Oct 4, 2013 at 23:46

hek2mgl

159k31 gold badges263 silver badges280 bronze badges

1 Comment

ChristopherStrydom Over a year ago

Thanks a bunch! that cleared up one of my problems. I am using the Count function just so i can easily access the size of the array as it is loaded dynamically

Collectives™ on Stack Overflow

PHP unset Not Working as expected

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related