Solved

php array copying problem... maybe reference problem

Posted on 2006-11-10
5
172 Views
Last Modified: 2012-05-05
The code below works great, except when i execute this line $jobnames[$jobnamelist[$i][$j]][$key1][2][] = $filelist[$file_key];

I can print the array and it has all the values in it.
however the next loop that array that I had copied into $jobnames[$jobnamelist[$i][$j]][$key1][2] is not there.
It like disappears.  

The print statements is below the code


// loops over job numbers
   for($i = 0; $i < count($joblist); $i++)
   {
     $check1 = 1;
     // loops over job names for the job number
     for($j = 0; $j < count($jobnamelist[$i]); $j++)
     {
        // loops over each
           $check2 = 1;
           foreach ($jobnames[$jobnamelist[$i][$j]] as $key1=>$filearray)
           {
              $matchcount = 0;
              foreach ($filelist as $file_key=>$filestats)
              {
                 $jobnames[$jobnamelist[$i][$j]][$key1][2] =  array();
                 $tmpfile = str_replace( "*", ".*", str_replace(".", "\\.", $filearray[0]));
                 if(preg_match("/" . $tmpfile . "/", $filestats[0]))
                 {
                    $jobnames[$jobnamelist[$i][$j]][$key1][2][] = $filelist[$file_key];
                    echo '<br>FOUND##### '.$tmpfile.'<br>';
                    $matchcount++;
                 }
              }
              if($matchcount < $filearray[1])
              {
                 $check2 = 0;
              }
           }
           if($check2 == 1)
             $jobnamecolor[$i][$j] = "good";
           else
             $check1 = 0;
     }
     if($check1 == 1)
        $jobcolor[$i][$j] = "good";
   }

===============================================
this is the first iteration I have marked the inserted array with &&&&&&&&&&&&&&

Array
(
    [job_accounting_reference_tables] => Array
        (
            [0] => Array
                (
                    [0] => sku_retail_prices*.csv
                    [1] => 1
                    [2] => Array
                        (
                            [0] => Array
                                (
                                    [0] => sku_retail_prices_20061031.csv      &&&&&&&&&&&&&&&&&&
                                    [1] => 1905                                            &&&&&&&&&&&&&&&&&&
                                    [2] => 11/02/2006 16:31                         &&&&&&&&&&&&&&&&&&
                                )

                        )

                )

            [1] => Array
                (
                    [0] => pia_activation*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => pia_rate_plans*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => service_billing_sched_category_lookup*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => eom033*.txt
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => adj_code_category_lookup*.csv
                    [1] => 1
                )

            [6] => Array
                (
                    [0] => market_lookup*.csv
                    [1] => 1
                )

        )

    [job_apl_revenue_sharing] => Array
        (
            [0] => Array
                (
                    [0] => apl_revenue_sharing_ace*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => apl_revenue_sharing_qpay*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => apl_revenue_sharing_tio*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => apl_revenue_sharing_infotouch_retail*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => apl_revenue_sharing_firstech*.csv
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => apl_revenue_sharing_infotouch_nonretail*.csv
                    [1] => 1
                )

        )

    [job_biapia_daily_payments] => Array
        (
            [0] => Array
                (
                    [0] => BIAPIA_SERVICE_REVENUE_*.csv
                    [1] => 31
                )

        )

    [job_brightpoint_tables] => Array
        (
            [0] => Array
                (
                    [0] => handset_allocation*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => market_state_city_lookup*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => bp_ar_rollforward_cm*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => bp_ar_rollforward_sales*.csv
                    [1] => 1
                )

        )

    [job_cost_layer_load] => Array
        (
            [0] => Array
                (
                    [0] => cost_layer*.csv
                    [1] => 1
                )

        )

    [job_daily_holoads] => Array
        (
            [0] => Array
                (
                    [0] => bil001*.csv
                    [1] => 28
                )

            [1] => Array
                (
                    [0] => cyc002*.csv
                    [1] => 28
                )

            [2] => Array
                (
                    [0] => ivprch*.csv
                    [1] => 31
                )

            [3] => Array
                (
                    [0] => cyc01515*.csv
                    [1] => 28
                )

        )

    [job_indirect_dealer_compensation] => Array
        (
            [0] => Array
                (
                    [0] => 3001_actuals*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => 3006_commissions_accrual*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => 3008_adjustments*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => 3005_coop_accrual*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => 3003_expirations*.csv
                    [1] => 1
                )

        )

    [job_inv_dependent_files] => Array
        (
            [0] => Array
                (
                    [0] => discount_sku_lookup*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => period_defaults*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => master_sku_list*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => location_lookup*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => locator_lookup*.csv
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => indirect_dealer_cost*.csv
                    [1] => 1
                )

            [6] => Array
                (
                    [0] => sku_rollup*.csv
                    [1] => 1
                )

        )

    [job_monthly_holoads] => Array
        (
            [0] => Array
                (
                    [0] => msc225*.txt
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => eom02a*.txt
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => ema001*.txt
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => ema003*.txt
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => eom003*.txt
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => eom002*.txt
                    [1] => 1
                )

            [6] => Array
                (
                    [0] => eom001*.txt
                    [1] => 1
                )

            [7] => Array
                (
                    [0] => eom004*.txt
                    [1] => 1
                )

            [8] => Array
                (
                    [0] => eom016*.txt
                    [1] => 1
                )

            [9] => Array
                (
                    [0] => eom10x*.txt
                    [1] => 1
                )

            [10] => Array
                (
                    [0] => eom010*.txt
                    [1] => 1
                )

        )

    [job_net_biapia_deferred_rev] => Array
        (
            [0] => Array
                (
                    [0] => net_biapia_def_revenue_adj*.csv
                    [1] => 1
                )

        )

)



==========================================================
Here is the second one again marked with &&&&&&&&&&&&& to show the inserted array.

Notice how the previous inserted array is gone from the array.

FOUND##### pia_rate_plans.*\.csv

Array
(
    [job_accounting_reference_tables] => Array
        (
            [0] => Array
                (
                    [0] => sku_retail_prices*.csv
                    [1] => 1
                    [2] => Array
                        (
                        )

                )

            [1] => Array
                (
                    [0] => pia_activation*.csv
                    [1] => 1
                    [2] => Array
                        (
                        )

                )

            [2] => Array
                (
                    [0] => pia_rate_plans*.csv
                    [1] => 1
                    [2] => Array
                        (
                            [0] => Array
                                (
                                    [0] => pia_rate_plans_20061031.csv      &&&&&&&&&&&&&&&
                                    [1] => 144                                           &&&&&&&&&&&&&&&
                                    [2] => 11/01/2006 16:27                       &&&&&&&&&&&&&&&
                                )

                        )

                )

            [3] => Array
                (
                    [0] => service_billing_sched_category_lookup*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => eom033*.txt
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => adj_code_category_lookup*.csv
                    [1] => 1
                )

            [6] => Array
                (
                    [0] => market_lookup*.csv
                    [1] => 1
                )

        )

    [job_apl_revenue_sharing] => Array
        (
            [0] => Array
                (
                    [0] => apl_revenue_sharing_ace*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => apl_revenue_sharing_qpay*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => apl_revenue_sharing_tio*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => apl_revenue_sharing_infotouch_retail*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => apl_revenue_sharing_firstech*.csv
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => apl_revenue_sharing_infotouch_nonretail*.csv
                    [1] => 1
                )

        )

    [job_biapia_daily_payments] => Array
        (
            [0] => Array
                (
                    [0] => BIAPIA_SERVICE_REVENUE_*.csv
                    [1] => 31
                )

        )

    [job_brightpoint_tables] => Array
        (
            [0] => Array
                (
                    [0] => handset_allocation*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => market_state_city_lookup*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => bp_ar_rollforward_cm*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => bp_ar_rollforward_sales*.csv
                    [1] => 1
                )

        )

    [job_cost_layer_load] => Array
        (
            [0] => Array
                (
                    [0] => cost_layer*.csv
                    [1] => 1
                )

        )

    [job_daily_holoads] => Array
        (
            [0] => Array
                (
                    [0] => bil001*.csv
                    [1] => 28
                )

            [1] => Array
                (
                    [0] => cyc002*.csv
                    [1] => 28
                )

            [2] => Array
                (
                    [0] => ivprch*.csv
                    [1] => 31
                )

            [3] => Array
                (
                    [0] => cyc01515*.csv
                    [1] => 28
                )

        )

    [job_indirect_dealer_compensation] => Array
        (
            [0] => Array
                (
                    [0] => 3001_actuals*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => 3006_commissions_accrual*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => 3008_adjustments*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => 3005_coop_accrual*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => 3003_expirations*.csv
                    [1] => 1
                )

        )

    [job_inv_dependent_files] => Array
        (
            [0] => Array
                (
                    [0] => discount_sku_lookup*.csv
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => period_defaults*.csv
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => master_sku_list*.csv
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => location_lookup*.csv
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => locator_lookup*.csv
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => indirect_dealer_cost*.csv
                    [1] => 1
                )

            [6] => Array
                (
                    [0] => sku_rollup*.csv
                    [1] => 1
                )

        )

    [job_monthly_holoads] => Array
        (
            [0] => Array
                (
                    [0] => msc225*.txt
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => eom02a*.txt
                    [1] => 1
                )

            [2] => Array
                (
                    [0] => ema001*.txt
                    [1] => 1
                )

            [3] => Array
                (
                    [0] => ema003*.txt
                    [1] => 1
                )

            [4] => Array
                (
                    [0] => eom003*.txt
                    [1] => 1
                )

            [5] => Array
                (
                    [0] => eom002*.txt
                    [1] => 1
                )

            [6] => Array
                (
                    [0] => eom001*.txt
                    [1] => 1
                )

            [7] => Array
                (
                    [0] => eom004*.txt
                    [1] => 1
                )

            [8] => Array
                (
                    [0] => eom016*.txt
                    [1] => 1
                )

            [9] => Array
                (
                    [0] => eom10x*.txt
                    [1] => 1
                )

            [10] => Array
                (
                    [0] => eom010*.txt
                    [1] => 1
                )

        )

    [job_net_biapia_deferred_rev] => Array
        (
            [0] => Array
                (
                    [0] => net_biapia_def_revenue_adj*.csv
                    [1] => 1
                )

        )

)

Thanks.
0
Comment
Question by:hapaboy2059
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 3
  • 2
5 Comments
 
LVL 19

Expert Comment

by:VoteyDisciple
ID: 17919424
This part is what doesn't make sense to me:
                 $jobnames[$jobnamelist[$i][$j]][$key1][2] =  array();
                 $tmpfile = str_replace( "*", ".*", str_replace(".", "\\.", $filearray[0]));
                 if(preg_match("/" . $tmpfile . "/", $filestats[0]))
                 {
                    $jobnames[$jobnamelist[$i][$j]][$key1][2][] = $filelist[$file_key];
                    echo '<br>FOUND##### '.$tmpfile.'<br>';
                    $matchcount++;
                 }


You're reusing [2] every time, so you'll never get new items.

Let me suggest this instead:

                 $temp =  array();
                 $tmpfile = str_replace( "*", ".*", str_replace(".", "\\.", $filearray[0]));
                 if(preg_match("/" . $tmpfile . "/", $filestats[0]))
                 {
                    $temp[] = $filelist[$file_key];
                    echo '<br>FOUND##### '.$tmpfile.'<br>';
                    $matchcount++;
                 }

Then after the loop do:
$jobnames[$jobnamelist[$i][$j]][$key1][] = $temp
0
 
LVL 19

Expert Comment

by:VoteyDisciple
ID: 17919425
And by "after the loop" I mean after the inner foreach loop.
0
 

Author Comment

by:hapaboy2059
ID: 17931755
The hard coded two is becuase the 0 and 1 elements have already been defined as the filemask and the daily_fl so I insert the new array at index 2.

0
 
LVL 19

Accepted Solution

by:
VoteyDisciple earned 500 total points
ID: 17931937
That clearly works fine the first time, but then the next element doesn't belong at [2] anymore, it belongs at [3].  If you put it back at [2] you're replacing what you already put there, and you get the results you're seeing now.  Since that's not what you want, use [] without any explicit index to mean "insert at the end of the array" (regardless of how many things are already in it).

That's precisely what the code I've provided does.  In particular the following line uses the [] syntax:

$jobnames[$jobnamelist[$i][$j]][$key1][] = $temp
0
 

Author Comment

by:hapaboy2059
ID: 17932503
This is the code that works now:

Thanks!


// loops over job numbers
   for($i = 0; $i < count($joblist); $i++)
   {
     $check1 = 1;
     // loops over job names for the job number
     for($j = 0; $j < count($jobnamelist[$i]); $j++)
     {
        // loops over each
           $check2 = 1;
           foreach ($jobnames[$jobnamelist[$i][$j]] as $key1=>$filearray)
           {
              $matchcount = 0;
              $countin =0;
              foreach ($filelist as $file_key=>$filestats)
              {
                // $jobnames[$jobnamelist[$i][$j]][$key1][2] =  array();
                 $tmpfile = str_replace( "*", ".*", str_replace(".", "\\.", $filearray[0]));
                 if(preg_match("/" . $tmpfile . "/", $filestats[0]))
                 {
                    $array = $filelist[$file_key];
                    $jobnames[$jobnamelist[$i][$j]][$key1][2][$countin][0] = $array[0];
                    $jobnames[$jobnamelist[$i][$j]][$key1][2][$countin][1] = $array[1];
                    $jobnames[$jobnamelist[$i][$j]][$key1][2][$countin][2] = $array[2];
                    $matchcount++;
                    $countin++;
                 }
              }
              if($matchcount < $filearray[1])
              {
                 $check2 = 0;
              }
              if($matchcount == 0)
              {
                 $jobnames[$jobnamelist[$i][$j]][$key1][2][$countin][0] = 'missing';
                 $jobnames[$jobnamelist[$i][$j]][$key1][2][$countin][1] = 'missing';
                 $jobnames[$jobnamelist[$i][$j]][$key1][2][$countin][2] = 'missing';
              }
           }
           if($check2 == 1)
             $jobnamecolor[$i][$j] = "good";
           else
             $check1 = 0;
     }
     if($check1 == 1)
        $jobcolor[$i][$j] = "good";
   }
0

Featured Post

Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

3 proven steps to speed up Magento powered sites. The article focus is on optimizing time to first byte (TTFB), full page caching and configuring server for optimal performance.
Many old projects have bad code, but the budget doesn't exist to rewrite the codebase. You can update this code to be safer by introducing contemporary input validation, sanitation, and safer database queries.
The viewer will learn how to dynamically set the form action using jQuery.
This tutorial will teach you the core code needed to finalize the addition of a watermark to your image. The viewer will use a small PHP class to learn and create a watermark.

734 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question