PHP foreachの中にfor文を入れた際の連想配列のキーについて

Question

        $setPositionPercentage =['upper'=>21, 'middle'=>67, 'low'=>14];
           
        if(array_sum($setPositionPercentage) > 100){
            foreach($setPositionPercentage as $key => $val){
                for ($i=100; $i<array_sum($setPositionPercentage); $i++){
                    $setPositionPercentage[$key] += -1;
                }
            }
        }
        var_dump($setPositionPercentage);
        /*
          array:3 [▼
            "upper" => 20.0
            "middle" => 66.0
            "low" => 14.0
          ]
       */

上記のvardumpの結果から、何故 "upper","middle"のkeyに対して、-1されているのでしょうか？
上記のループ、foreachの入れ子のロジックでは、upperのkeyの値に対してのみ-1ずつされていき、100になったらループを抜けると解釈していたため、何故forのループ中に、$keyが"upper"から"middle"になるのかわかりません。
返り値として、下記を想定していました。
/*
array:3 [▼
"upper" => 19.0
"middle" => 67.0
"low" => 14.0
]
*/

Comments

[m.ts10806]

途中echoなどで値を追ってデバッグしてみては？

Answer 1

2週目でfor文の条件を満たさなくなりますね

foreach($setPositionPercentage as $key => $val){
    // (1週目)$i = 100 array_sum($setPositionPercentage) = 102  (2週目) $i=101  array_sum($setPositionPercentage) = 101
    for ($i=100; $i<array_sum($setPositionPercentage); $i++){ 
      $setPositionPercentage[$key] += -1; // (1週目)array_sum($setPositionPercentage) = 101 (2週目)通らない
    }
}

Answer 2

foreachの入れ子のロジックでは、upperのkeyの値に対してのみ-1ずつされていき、100になったらループを抜ける

の解釈が間違っています。

for ($i=100; $i<array_sum($setPositionPercentage); $i++){
 //中身は省略
}

は、

$iを100から初めて、$iがarray_sum($setPositionPercentage)よりも小さい間処理をする

としか書かれていないです。

array_sum($setPositionPercentage)の減り方と、
for文のループの仕方を照らし合わせれば、今の結果になるのも納得できるはず・・・。

---

今のコードを再利用して

upperのkeyの値に対してのみ-1ずつされていき、100になったらループを抜ける

を実現するなら、せめてこう・・・

foreach($setPositionPercentage as $key => $val){
    if($key != 'upper'){
        continue;
    }
    for ($i=array_sum($setPositionPercentage); $i>100; $i--){
        $setPositionPercentage[$key] += -1;
    }
}

---

そもそもの話をすると、更新したいキーが固定であるならば、
外側のforeach自体が不要で、もっと単純に書いた方が分かりやすいです。

for文

$key = 'upper';
for ($i=array_sum($setPositionPercentage); $i>100; $i--){
    $setPositionPercentage[$key] += -1;
}

whileの方がコードはちょっと短くなりますね。

$key = 'upper';
while(array_sum($setPositionPercentage) > 100){
    $setPositionPercentage[$key] += -1;
}

★おまけ★
仮に、キーがランダムで、「最初のキーだけ」-1していくようにしたいならば
キーの指定はこういうことも可能です。

//連想配列の最初のキーを取得
$key = key(reset($setPositionPercentage));