PHP - finding unique values within an array

Question

I am creating an ecommerce site and am trying to get a list of top selling products.

The following example shows an array of product ids from sales, and is outputting the top selling products in the form of 'product id' x 'number of sales'

<?php
$product_ids = [58,58,68,39,4,118,14,54,39,118,58];
$top_selling_products = "";

$product_sales = array_count_values($product_ids);
arsort($product_sales);
$i = 0;
foreach($product_sales as $sales){
    $i++;
    $product_id = array_search($sales,$product_sales);
    $top_selling_products .= $product_id.' x '.$sales.'<br />';
    if($i == 5){break;}
}

echo $top_selling_products;
?>

However, it's not exactly doing what I want, at present this is outputting the following:

Where it should be:

It is getting the last item in the array with the same index so when there are two or more products with the same amount of sales it is not picking them up.

What would be the best way to get all unique ids?

u_mulder · Accepted Answer · 2018-06-08 10:21:49Z

2

You already have what you need in $product_sales:

foreach($product_sales as $id => $count){
    $i++;
    $top_selling_products .= $id.' x '.$count.'<br />';
    if($i == 5){break;}
}

answered Jun 8, 2018 at 10:21

u_mulder

54.8k5 gold badges54 silver badges72 bronze badges

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

1 Comment

Paddy Hallihan Over a year ago

@u_molder thanks for your answer this works perfectly

Mitya · Accepted Answer · 2018-06-08 10:28:28Z

1

Your code can be simplified to use something like array_map():

arsort($product_sales);
$output = array_map(function($sales, $pid) {
    return $pid.' x '.$sales;
}, $product_sales, array_keys($product_sales));
echo implode('<br />', $output);

answered Jun 8, 2018 at 10:28

Mitya

34.7k15 gold badges70 silver badges125 bronze badges

Comments

Progrock · Accepted Answer · 2018-06-08 10:28:58Z

1

<?php

$product_ids      = [58,58,68,39,4,118,14,54,39,118,58];
$product_sales    = array_count_values($product_ids);
arsort($product_sales);
$top_five_sellers = array_slice($product_sales, 0, 5, true);
foreach($top_five_sellers as $id => $count)
    echo $id.' x '.$count."\n";

Output:

answered Jun 8, 2018 at 10:28

Progrock

7,4831 gold badge21 silver badges25 bronze badges

Comments

Niklesh Raut · Accepted Answer · 2018-06-08 10:48:03Z

0

You can achieve your goal by using array_count_values + Simple foreach + Sort array by Keeping key with arsort

$product_ids = [58,58,68,39,4,118,14,54,39,118,58];
$arr = array_slice(array_count_values($product_ids), 0, 5, true);
arsort($arr);
$str = "";
foreach($arr as $k=>$v)
  $str .= "$k X $v \n";
echo $str;

Live Demo

Output is as :

edited Jun 8, 2018 at 10:48

answered Jun 8, 2018 at 10:38

Niklesh Raut

34.9k17 gold badges82 silver badges112 bronze badges

Collectives™ on Stack Overflow

PHP - finding unique values within an array

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related