remove duplicate arrays in loop

Question

I have an associative array that might contain duplicates. I am trying to loop through the array and compare the current element with the next element in the array. If there is a duplicate, it should be removed.

The code below removes one instance of the element. In the test array I'm using, I have 3 duplicate part numbers, but my code only removes one. I'm left with two. I only want one to remain.

  $length = count($items);

   for($i = 0; $i < $length -1; $i++){
    if($items[$i]['part_number'] == $items[$i+1]['part_number']){
      unset($items[$i+1]);
      $items = array_values($items);
    }
   }

What am I doing wrong here?

Possible duplicate of Remove duplicates from an array based on object property? — Chrille
– Chrille, Commented Feb 13, 2018 at 7:42

bumperbox · Accepted Answer · 2018-02-13 07:50:58Z

2

You need to loop backwards through the array, and delete the current item.

$length = count($items);

for($i = $length - 1; $i > 0; $i--){
    if($items[$i]['part_number'] == $items[$i-1]['part_number']){
        unset($items[$i]);
    }
}

answered Feb 13, 2018 at 7:50

bumperbox

10.2k6 gold badges46 silver badges66 bronze badges

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

3 Comments

Paul Dessert Over a year ago

That almost works. It seems to be removing too many elements. I have a multidimensional array with 9 records. Each 012 are duplicates, 345 are duplicates and 678 are duplicates. I only want 0,3 and 6. In your example, 6 is removed.

Paul Dessert Over a year ago

Strange. My code is acting differently. I'll double check my array. Thanks for the help!

Paul Dessert Over a year ago

I forgot to reindex the array. $items = array_values($items); Working now.

hoyeonUm · Accepted Answer · 2018-02-13 07:45:03Z

0

becuase your code is The $ items value is in the for statement.

if you want unique array, you have to array_unique function

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

answered Feb 13, 2018 at 7:45

hoyeonUm

498 bronze badges

2 Comments

bumperbox Over a year ago

Note from the php docs you linked to: Note that array_unique() is not intended to work on multi dimensional arrays.

hoyeonUm Over a year ago

oaky, how about phpflow.com/php/remove-duplicates-from-multidimensional-array link ?

vpalade · Accepted Answer · 2018-02-13 08:00:20Z

0

In your case after you unset element, $i++ in for loop, you reindexed your array and you skip one element. Add $i-- if you unset item. Or you can reindex your array after for loop.

edited Feb 13, 2018 at 8:00

answered Feb 13, 2018 at 7:47

vpalade

1,4372 gold badges16 silver badges20 bronze badges

Comments

Fil · Accepted Answer · 2018-02-13 08:00:43Z

0

This is also a very simple example you can start improving with.

<?php
$test = ['sample', 'sample', 'sample', 'not', 'not', 'no', 'no'];
$test2 = [];

$k = 0;
foreach ($test as $key => $value) {
  if ($key == 0) {
    $test2[$k] = $value;
    $k++;
  } else {
    if ($test2[$k - 1] != $value) {
      $test2[$k] = $value;
      $k++;
    }
  }
}

$test = $test2;

var_dump($test);

edited Feb 13, 2018 at 8:00

answered Feb 13, 2018 at 7:54

Fil

8,91118 gold badges64 silver badges91 bronze badges

Comments

Shobi · Accepted Answer · 2018-02-13 08:18:43Z

0

One dirty hack is to check again if you have a duplicate by decreasing $i.

for($i = 0; $i < $length -1; $i++){
    if($items[$i]['part_number'] == $items[$i+1]['part_number']){
        unset($items[$i+1]);
        $items = array_values($items);
        $i--;
    }
}

This way it will again test your previous value against next item in array.
So if 0==1, then next time if 0==2.
Your code did 0==1 then (2)==(3).

edited Feb 13, 2018 at 8:18

Shobi

11.7k8 gold badges54 silver badges94 bronze badges

answered Feb 13, 2018 at 7:46

Andreas

24k7 gold badges34 silver badges75 bronze badges

Collectives™ on Stack Overflow

remove duplicate arrays in loop

5 Answers 5

3 Comments

2 Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

3 Comments

2 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related