increment counter status wise in php foreach

Question

I am working on cakephp project where i've to generate the reports. I've two tables one table stores the value of customer and another table store the value of leads assigned to each user. Each lead is having status. I want to increment the counter of progressleads if status is 0, if status is 1 then counter of wonleads should be increment and so on...

I am joining two tables and getting array like this =>

From this array i want one record for each users with total leads, inprogress leads, won leads counter. I've tried following code :-

$customerdetails = array();
            $totalopp = 0;
            $progressopp = 0;
            $oppval = 0;
            $wonopp = 0;
            $lostopp = 0;
            $billedopp = 0;
            $onholdopp = 0;
            $newcustid = NULL;
            foreach($customer as $k => $val){
                $custid = $val["Customer"]["id"];
                if($newcustid != $custid){
                    $oppstatus = $val["Opportunity"]["status"];
                    $oppval += $val["Opportunity"]["opo_value"];
                    $totalopp++;
                    if($oppstatus == 0){
                        $progressopp++;
                    }
                    if($oppstatus == 1){
                        $wonopp++;
                    }
                    if($oppstatus == 2){
                        $lostopp++;
                    }
                    if($oppstatus == 3){
                        $billedopp++;
                    }
                    if($oppstatus == 4){
                        $onholdopp++;
                    }
                     $newcustid = $custid;
                }
                    $customerdetails[$custid]["customername"] = $val["Customer"]["customer_name"];
                    $customerdetails[$custid]["opportunities"] = $totalopp;
                    $customerdetails[$custid]["value"] = $oppval;
                    $customerdetails[$custid]["inprogress"] = $progressopp;
                    $customerdetails[$custid]["won"] = $wonopp;
                    $customerdetails[$custid]["lost"] = $lostopp;
                    $customerdetails[$custid]["billed"] = $billedopp;
                    $customerdetails[$custid]["onhold"] = $onholdopp;

            }

After printing $customerdetails array i am getting following results =>

Array
(
    [14] => Array
        (
            [customername] => ABC
            [opportunities] => 6
            [value] => 1146000
            [inprogress] => 4
            [won] => 0
            [lost] => 2
            [billed] => 0
            [onhold] => 0
        )

    [15] => Array
        (
            [customername] => DEF
            [opportunities] => 2
            [value] => 95000
            [inprogress] => 2
            [won] => 0
            [lost] => 0
            [billed] => 0
            [onhold] => 0
        )

    [19] => Array
        (
            [customername] => ST
            [opportunities] => 5
            [value] => 1146000
            [inprogress] => 3
            [won] => 0
            [lost] => 2
            [billed] => 0
            [onhold] => 0
        )

    [16] => Array
        (
            [customername] => TEST
            [opportunities] => 4
            [value] => 1145000
            [inprogress] => 2
            [won] => 0
            [lost] => 2
            [billed] => 0
            [onhold] => 0
        )

)

As you can see there are only 4 leads assigned to ABC but it is showing opportunities as 6 similarly the other counters are also displaying incorrect. Can anyone help me what i am doing wrong here?

Alok Patel · Accepted Answer · 2016-06-10 06:10:05Z

1

I think you're applying wrong logic here. array_column will do the trick. See the following code.

<?php

$customers=array_column($result,'customer');
$opportunity=array_column($result,'opportunity');

$finalResult=array();

foreach($customers as $k=>$customer) {
     $opp=$opportunity[$k];
     if(!array_key_exists($customer["id"],$finalResult)) {
         $finalResult[$customer["id"]]=array();
         $finalResult[$customer["id"]]["opportunities"]=0;
         $finalResult[$customer["id"]]["value"]=0;
         $finalResult[$customer["id"]]["inprogress"]=0;
         $finalResult[$customer["id"]]["won"]=0;
         $finalResult[$customer["id"]]["lost"]=0;
         $finalResult[$customer["id"]]["billed"]=0;
         $finalResult[$customer["id"]]["onhold"]=0;
     }
     $finalResult[$customer["id"]]["customerid"]=$customer["id"];
     $finalResult[$customer["id"]]["customername"]=$customer["name"];
      $finalResult[$customer["id"]]["opportunities"]++;
      $finalResult[$customer["id"]]["value"]+=$opp["value"];
      if($opp["status"]==0) {
          $finalResult[$customer["id"]]["inprogress"]++;
      }
      if($opp["status"]==1) {
          $finalResult[$customer["id"]]["won"]++;
      }
      if($opp["status"]==2) {
          $finalResult[$customer["id"]]["lost"]++;
      }
      if($opp["status"]==3) {
          $finalResult[$customer["id"]]["billed"]++;
      }
      if($opp["status"]==4) {
          $finalResult[$customer["id"]]["onhold"]++;
      }     
}

?>

You just need to get the customer column and opportunity column individually and then traverse customers column.

Now when you're traversing, first check if you've already find that customer or not by saving values in $finalResult array. If yes then just increment the counters accordingly if not create an array then increments the counter accordingly.

I think this will do the trick.

Output array might look like this:

Array
(
    [14] => Array
        (
            [opportunities] => 4
            [value] => 13500
            [inprogress] => 2
            [won] => 2
            [lost] => 0
            [billed] => 0
            [onhold] => 0
            [customerid] => 14
            [customername] => ABC
        )

    [15] => Array
        (
            [opportunities] => 1
            [value] => 45000
            [inprogress] => 1
            [won] => 0
            [lost] => 0
            [billed] => 0
            [onhold] => 0
            [customerid] => 15
            [customername] => DEF
        )

    [19] => Array
        (
            [opportunities] => 2
            [value] => 51000
            [inprogress] => 1
            [won] => 0
            [lost] => 1
            [billed] => 0
            [onhold] => 0
            [customerid] => 19
            [customername] => ST
        )

    [16] => Array
        (
            [opportunities] => 1
            [value] => 100000
            [inprogress] => 0
            [won] => 0
            [lost] => 1
            [billed] => 0
            [onhold] => 0
            [customerid] => 16
            [customername] => TEST
        )

)

edited Jun 10, 2016 at 6:10

answered Jun 10, 2016 at 5:53

Alok Patel

8,0726 gold badges35 silver badges48 bronze badges

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

5 Comments

user3327608 Over a year ago

Thanks for Your help but can u please tell me what is $result contains?

Alok Patel Over a year ago

$finalResult is just an array which will contain the final result. Check the updated answer for the format of array.

user3327608 Over a year ago

This is the first record which i got like this :- Array ( [14] => Array ( [opportunities] => 1 [value] => 50000 [inprogress] => 1 [won] => 0 [lost] => 0 [billed] => 0 [onhold] => 0 [customerid] => 14 [customername] => ABC )

Alok Patel Over a year ago

eval.in/586550 check here. It is getting summed up according to customer id.

user3327608 Over a year ago

Its working now. Thanks very much for your constant effort. I really appreciate it. thanks again. I am accepting it as answer.

Manohar Khadka · Accepted Answer · 2016-06-10 05:37:38Z

0

What is your logic behind doing this :

if($newcustid != $custid){
  $totalopp++;
}

And you assigning $newcustid to null. I think that condition leading you to wrong opportunities value.

answered Jun 10, 2016 at 5:37

Manohar Khadka

2,1952 gold badges20 silver badges31 bronze badges

1 Comment

user3327608 Over a year ago

When both are not same at that time i've to increment the counter values. I want counter values per user basis. I am new to php. Please help me

Collectives™ on Stack Overflow

increment counter status wise in php foreach

2 Answers 2

5 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related