Question

我有一些嗯...我认为一些不合逻辑的逻辑，我自己认为...... 现在我可以告诉你：

$new ['period_10th'] = $periodic_items [9] ;
            foreach ($new ['period_10th'] as $item) { $po_10th [] = $item -> po_id ; }
          $new ['uni_10'] = array_unique($po_10th) ;
          foreach ($new ['uni_10'] as $po_id) {
            $po = $this -> model_prcsys -> get_po_by_id (md5($po_id)) ;
            $pos_10 [] = $po ['po_id'] ;
            $currency10 [] = $po ['currency'] ;
          }
          $cur_pos_10 = array_unique($currency10) ;
          foreach ($cur_pos_10 as $currency) {
            $new ['po_arr_10'] = $this -> model_prcsys -> get_pos_with_curr ($pos_10,$currency) ;
              foreach ($new ['po_arr_10'] as $key) {
                $test [] = $key -> total_line_price;
              }
              print_r($test);echo "<br><br>";
              print_r(array_sum($test));echo "<br><br>";
          }

那将是值

Array ( [0] => 20700000.00 [1] => 10340000.00 [2] => 4160000.00 [3] => 8150000.00 [4] => 9065000.00 [5] => 3500000.00 [6] => 2530000.00 [7] => 650000.00 [8] => 4395000.00 [9] => 17100000.00 [10] => 11250000.00 [11] => 6900000.00 [12] => 300000.00 [13] => 15750000.00 )

114790000

Array ( [0] => 20700000.00 [1] => 10340000.00 [2] => 4160000.00 [3] => 8150000.00 [4] => 9065000.00 [5] => 3500000.00 [6] => 2530000.00 [7] => 650000.00 [8] => 4395000.00 [9] => 17100000.00 [10] => 11250000.00 [11] => 6900000.00 [12] => 300000.00 [13] => 15750000.00 [14] => 1200000.00 )

115990000

我需要计算第一个数组直到结束数组（尽可能多的$ cur_pos_10），但它总是重新计算整个数组的第二个值到结尾。请帮助我，我真的很抱歉我的英语不好。

我需要这个：第一个结果计算数组[0]到[13]; 第二个结果只计算数组[14];

Answer 1

很难理解，不清楚代码应该做什么，但看起来你只需要在foreach之前定义用于存储结果的变量，因为$test []会将值推送到数组$test并且变量$test在foreach内定义，并在多个循环中累积所有结果。所以试试这个：

$new ['period_10th'] = $periodic_items [9] ;
foreach ($new ['period_10th'] as $item) { $po_10th [] = $item -> po_id ; }
$new ['uni_10'] = array_unique($po_10th) ;

$pos_10 = []; // added
$currency10 = []; // added
foreach ($new ['uni_10'] as $po_id) {
  $po = $this -> model_prcsys -> get_po_by_id (md5($po_id)) ;
  $pos_10 [] = $po ['po_id'] ; // possibly similar problem if whole code is in loop or is executed multiple times
  $currency10 [] = $po ['currency'] ; // possibly similar problem if whole code is in loop or is executed multiple times
}

$cur_pos_10 = array_unique($currency10) ;
foreach ($cur_pos_10 as $currency) {
  /* in your code
      in first iteration variable $test is undefined
      in second iteration variable $test is defined and count($test) == 14
  */
  $new ['po_arr_10'] = $this -> model_prcsys -> get_pos_with_curr ($pos_10,$currency) ;
  $test = []; // added, variable $test is set to an empty array and erase previous results
  foreach ($new ['po_arr_10'] as $key) {
    $test [] = $key -> total_line_price;  // problem was here
  }
  print_r($test);echo "<br><br>";
  print_r(array_sum($test));echo "<br><br>";
}

Answer 2

良好的缩进和对齐代码很重要。我缩进并对齐了你的代码。由于@Kazz表示问题在于您没有重置$ test变量。代码中的一些注释也很有价值。

<?php

$new['period_10th'] = $periodic_items[9] ;
foreach ($new['period_10th'] as $item) {
    $po_10th[] = $item->po_id ;
}

$new['uni_10'] = array_unique($po_10th) ;
foreach ($new['uni_10'] as $po_id) {
    $po           = $this->model_prcsys->get_po_by_id (md5($po_id)) ;
    $pos_10[]     = $po ['po_id'] ;
    $currency10[] = $po ['currency'] ;
}

$cur_pos_10 = array_unique($currency10) ;
foreach ($cur_pos_10 as $currency) {
    $new['po_arr_10'] = $this->model_prcsys->get_pos_with_curr ($pos_10,$currency) ;

    $test = array(); // this was missing and probably causing problems
    foreach ($new['po_arr_10'] as $key) {
        $test[] = $key->total_line_price;
    }

    print_r($test);
    echo "<br><br>";
    print_r(array_sum($test));
    echo "<br><br>";
}

在foreach内部指定foreach

2 个答案: