Как распределить остаток по индексу массива, от меньшего к большему?

Question

[thinksdifferent]

$amount = 5;
$wareh = 4;

$wareh_id =
array(
'3'=>3,
'2'=>2,
'1'=>1,
'0'=>0
);

//arsort($wareh_id);
$i= 1;
$used = 0;
$arr=[];
foreach($wareh_id as $key=>$value){

if($i <= $wareh){
$used += ($current = round($amount * $i / $wareh - $used));
$arr[$key] = $current;
}
$i++;
}

Получаю ответ :

Array
(
[3] => 1
[2] => 2
[1] => 1
[0] => 1
)

а нужно :

Array
(
[3] => 1
[2] => 1
[1] => 1
[0] => 2
)

Должен быть приоритет к индексу '0', далее по остаточному признаку.

вот для примера 10 штук :
Array
(
[3] => 3
[2] => 2
[1] => 3
[0] => 2
)
а должно быть
Array
(
[3] => 2
[2] => 2
[1] => 3
[0] => 3
)

Answer 1

[Rsa97]

Использовать не foreach, а классический for.
И вычислять значения для заполнения сразу, а не пересчитывать на каждой итерации.

$count = count($wareh_id);
$val = ceil($amount / $count);
$n = $amount % $count;
if ($n === 0) {
    $n = $count;
}
for ($i = 0; $i < $n; $i += 1) {
    $wareh_id[$i] = $val;
}
$val -= 1;
for (;$i < $count; $i += 1) {
    $wareh_id[$i] = $val;
}

Comments

[thinksdifferent]

$count = count($wareh_id);
$val = ceil($amount / $count);
$n = $amount % $count;
if ($n === 0) {
$n = $count;
}
for ($i = 0; $i < $n; $i += 1) {
$wareh_id[$i] = $val;
}
$val -= 1;
for ($i = 0; $i < $count; $i += 1) {
$wareh_id[$i] = $val;
}

Ваш код выдает :

Array
(
[3] => 2
[2] => 2
[1] => 2
[0] => 2
)

[Rsa97]

thinksdifferent, Потому что вы даже скопировать код правильно не смогли. Во втором цикле у меня нет инициализации $i.

[thinksdifferent]

Да потому что вы написали for (;$i < count; $i += 1) {
который ошибку выдает. Я тоже виноват в этом ?

[thinksdifferent]

мне этот вариант не подходит, так как нет переменной $wareh = 4;, и $wareh_id может приходить в

Array
(
[3] => 2
[2] => 2
[0] => 2
)

или так

Array
(
[1] => 2
[0] => 2
)

или так

Array
(

[2] => 2
[1] => 2

)

[Rsa97]

thinksdifferent, Ошибка там была в пропущенном знаке доллара у $count, поправил.
С остальным непонятно. Что такое этот ваш $wareh?

[thinksdifferent]

Rsa97, там две ошибки, пропущен первый параметр, должно быть $i; как я понимаю теперь.

Что такое этот ваш $wareh?

количество передаваемых складов на который надо распределить остаток. $wareh_id это id этих складов. И эти параметры всегда(почти всегда) разные.

[Rsa97]

thinksdifferent, Ну так и используйте этот $wareh вместо $count.

[thinksdifferent]

У складов индекс может идти в произвольном виде 0,2,3. как тогда for их будет перебирать ?

$amount = 10;
$wareh = 3;

$wareh_id =
array(
'3'=>3,
'2'=>2,
//'1'=>1,
'0'=>0
);

на выходе показывает

Array
(
[3] => 3
[2] => 3
[0] => 4
[1] => 3
)

[Rsa97]

thinksdifferent, Так вы определитесь. Если вам в $wareh_id пришло два склада, а в $wareh записано четыре, то что делать?

[thinksdifferent]

Короче я сделал все проще

$count = count($wareh_id);
$val = ceil($amount / $count);
$n = $amount % $count;
if ($n === 0) {
$n = $count;
}
for ($i = 0; $i < $n; $i += 1) {
$wareh_id[$i] = $val;
}
$val -= 1;
for ($i; $i < $count; $i += 1) {
$wareh_id[$i] = $val;
}

$arr = array_combine($wareh_id, $arr) ;

сформировал сначала остатки по количеству складов, потом подставил свои ключи, так меня полностью устроило.