コードの改善点を教えていただきたいです。

Question

以下の問題のプログラムを書きましたが、変数名や流れに関して、こうした方が良いというところを教えていただきたいです。
よろしくお願いいたします。

/**

• 製品A
• 部品A2個と部品B1個からできている。
• 製品B
• 部品C3個と部品D2個からできている。
• 製品C
• 部品B1個と部品D1個からできている。
• 製品Aと製品Bと製品Cをランダムで1個ずつ発注する。
• 　部品にはそれぞれ在庫がありそれがなくなるまで製造する。
• 最後に以下を出力します。
• 製造前の各部品の在庫数
• 製品の発注数
• 製造した製品の個数
• 製造後の各部品の在庫数

*/

$part_names = [
    'part1' => 'A',
    'part2' => 'B',
    'part3' => 'C',
    'part4' => 'D',
];

$part_stocks = [
    'part1' => 120,
    'part2' => 120,
    'part3' => 130,
    'part4' => 130,
];

$product_names = [
    'product1' => 'A',
    'product2' => 'B',
    'product3' => 'C',
];

$needed_parts_counts = [
    'product1' => [
        'part1' => 2,
        'part2' => 1,
    ],
    'product2' => [
        'part3' => 3,
        'part4' => 2,
    ],
    'product3' => [
        'part2' => 1,
        'part4' => 1,
    ],
];

$first_part_stocks = $part_stocks;
$product_counts    = [];
$order_count       = 0;

while (true) {
    $makeable_product_ids = [];

    foreach ($needed_parts_counts as $product_id => $needed_parts_count) {
        $can_make = true;

        foreach ($needed_parts_count as $part_id => $needed_part_count) {
            if ($part_stocks[$part_id] > $needed_part_count) {
                $can_make = false;
                break;
            }
        }

        if ($can_make) {
            $makeable_product_ids[] = $product_id;
        }
    }

    if (count($makeable_product_ids) === 0) {
        break;
    }

    $ordered_product_id = $makeable_product_ids[array_rand($makeable_product_ids)];
    $order_count++;

    if (isset($product_counts[$ordered_product_id])) {
        $product_counts[$ordered_product_id]++;
    } else {
        $product_counts[$ordered_product_id] = 1;
    }

    foreach ($needed_parts_counts[$ordered_product_id] as $part_id => $needed_part_count) {
        $part_stocks[$part_id] -= $needed_part_count;
    }
}

echo "製造前<br>";

foreach ($first_part_stocks as $part_id => $part_stock) {
    echo '部品' . $part_names[$part_id] . ' ' . $part_stock . "個<br>";
}

echo "<br>";
echo '発注 ' . $order_count . '回<br>';

foreach ($product_counts as $product_id => $product_count) {
    echo '製品' . $product_names[$product_id] . ' ' . $product_count . "個<br>";
}

echo "<br>";
echo "製造後<br>";

foreach ($part_stocks as $part_id => $part_stock) {
    echo '部品' . $part_names[$part_id] . ' ' . $part_stock . "個<br>";
}

Comments

[m.ts10806]

コードの前に質問の仕方をヘルプ熟読の上、改善してください。作業依頼に近しい内容です。

[KazuhiroHatano]

オブジェクト指向にしよう、注文管理のクラスを作り注文を注文商品のIDと個数の配列を引数にメソッド一つでできるようにしよう、注文データの入出力はデータのソースがJSON、CSV、XMLとかのファイルになっても、DBになっても同じメソッドを使って入出力できるよう簡単に切り替えできるようにしておくと後々取り回しがいい

Answer 1

ありがとうございました。

質問の仕方を読んでからまた質問したいと思います。