in_array() expects parameter 2 to be array, integer given

Question

I want to create an overview of the first dates of every event. So the event-title must be unique. My idea was to create a helper-function where I loop over the result of my query and check the title of every item. To make sure every title passes only once, I want to push the title into an array ($checklist). If it does not exist, I add that item to the result-array. If it does, just continue to the next item.

I always get the error:

in_array() expects parameter 2 to be array, integer given

This is my code:

function showFirstEvenst($collection) {
    $checklist = array();
    $result = array();

    foreach ($collection as $item) {
        $title = strtolower($item['events']['title']);

        if (!in_array($title, $checklist)) {
            $checklist = array_push($checklist, $title);
            $result = array_push($result, $item);
        }
    }

    return $result;
}

I already tried to cast $checklist and $result as array in the foreach loop but without result.

What do I need to change?

php.net/manual/en/function.array-push.php returns an integer, and you overwrite your array in the loop — Qirel
– Qirel, Commented Jun 12, 2017 at 18:21
@bflydesign it won't make a huge difference, but yes, it should be. — Don't Panic
– Don't Panic, Commented Jun 12, 2017 at 18:27
Found this related question in regards to performance of $array[] = $value vs array_push($value, $array): stackoverflow.com/questions/559844/… — Graham S.
– Graham S., Commented Jun 12, 2017 at 18:35

Graham S. · Accepted Answer · 2017-06-12 18:24:50Z

5

Adding to @Lawrence Cherone and @Ravinder Reddy's answers, instead of using array_push, you could use native array syntax to push to the array:

if (!in_array($title, $checklist)) {
    $checklist[] = $title;
    $result[] = $item;
}

answered Jun 12, 2017 at 18:24

Graham S.

1,5501 gold badge21 silver badges29 bronze badges

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

1 Comment

Don't Panic Over a year ago

In fact, the array_push documentation recommends doing it this way when you're just appending one item.

Ravinder Reddy · Accepted Answer · 2017-06-12 18:22:24Z

3

array_push function will return the count of array after an element is added to array. so dont assing the output of the function to array.

Replace

  if (!in_array($title, $checklist)) {
                $checklist = array_push($checklist, $title);
                $result = array_push($result, $item);
            }

with

 if (!in_array($title, $checklist)) {
               array_push($checklist, $title);
               array_push($result, $item);
            }

answered Jun 12, 2017 at 18:22

Ravinder Reddy

3,8791 gold badge16 silver badges24 bronze badges

1 Comment

bflydesign Over a year ago

that is indeed the problem...wasn't looking for that.

Lawrence Cherone · Accepted Answer · 2017-06-12 18:21:32Z

2

Its happening because within your loop your assigning $checklist with the value of array_push() which will be the new number of elements in the array.

http://php.net/manual/en/function.array-push.php

answered Jun 12, 2017 at 18:21

Lawrence Cherone

46.7k7 gold badges66 silver badges107 bronze badges

Collectives™ on Stack Overflow

in_array() expects parameter 2 to be array, integer given

3 Answers 3

1 Comment

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related