ItemRoulette/ItemRoulette.php

@@ -3,12 +3,7 @@ declare(strict_types=1);
 
 class ItemRoulette
 {
-    /**
-     * @var array Массив шанса выпадения предметов. Ключом выступает имя предмета, значением выступает шанс выпадения.
-     * Шанс задаётся в виде десятичной дроби (25% = 0.25).
-     * ['prize1' => 0.25, 'prize2' => 0.06, 'prize3' => 0.1,]
-     */
-    public array $items = [];
+    private array $items;
 
     /**
      * Рассчёт верхнего значения диапазона, подаваемого на рандомизатор через
@@ -25,7 +20,7 @@ class ItemRoulette
         if ((int)$num == $num || !is_numeric($num)) {
             return 0;
         }
-        [, $a] = explode('.', strval($num));
+        list(, $a) = explode('.', strval($num));
         return $num * pow(10, strlen($a));
     }
 
@@ -36,19 +31,19 @@ class ItemRoulette
      * После этого на полученный множитель умножается каждое процентное значение и гененируются диапазоны значений для
      * каждого предмета. После этого берётся случайное число от 0 до максимального значения и сравнивается с
      * диапазонами. В какой диапазон попадает, тот предмет и выпал.
-     * @return string|null имя выпавшего предмета.
+     *
+     * @return string имя выпавшего предмета.
      */
-    private function roll(): ?string
+    private function roll(): string
     {
-        if (!is_array($this->items || empty($this->items))) {
-            return null;
+        $sumOfPercents = 0;
+        foreach ($this->items as $itemsPercent) {
+            $sumOfPercents += $itemsPercent;
         }
-        $itemKey = '';
-        $sumOfPercents = array_sum($this->items);
         $rand = mt_rand(1, $this->randMax($sumOfPercents));
         $rangeStart = 1;
         foreach ($this->items as $itemKey => $itemsPercent) {
-            $rangeFinish = $rangeStart + $itemsPercent;
+            $rangeFinish = $rangeStart + ($itemsPercent * $multiplier);
             if ($rand >= $rangeStart && $rand <= $rangeFinish) {
                 break;
             }
@@ -58,25 +53,50 @@ class ItemRoulette
     }
 
     /**
-     * Функция, запускающая расчёт определённое количество раз и возвращающая массив выпавших предметов
-     * либо NULL в случае ошибки.
+     * Функция, запускающая расчёт определённое количество раз и возвращающая массив выпавших предметов.
      *
-     * @param int $rolls сколько предметов надо получить на выходе.
-     *
-     * @return array|null
+     * @param $rolls
+     * @return array
      */
-    public function loot(int $rolls = 1): ?array
+    private function getMany($rolls): array
     {
-        if ($rolls < 1) {
-            return null;
-        }
         $result = [];
         for ($i = 0; $i < $rolls; $i++) {
-            if (is_null($this->roll())) {
-                continue;
-            }
             $result[] = $this->roll();
         }
         return $result;
     }
+
+    /**
+     * Функция, запускающая расчёт один раз и возвращающая один предмет.
+     *
+     * @return string
+     */
+    private function getOne(): string
+    {
+        return $this->roll();
+    }
+
+    /**
+     * Передача массива шанса выпадения предметов в класс.
+     *
+     * @param array $items
+     * @return void
+     */
+    public function setItems(array $items)
+    {
+        $this->items = $items;
+    }
+
+    /**
+     * Функция, запускающая расчёт определённое количество раз и, в зависимости от количества запусков
+     * возвращающая либо массив выпавших предметов либо одно строчное значение.
+     *
+     * @param int $rolls сколько предметов надо получить на выходе.
+     * @return array|string
+     */
+    public function loot(int $rolls = 1)
+    {
+        return $rolls > 1 ? $this->getMany($rolls) : $this->getOne();
+    }
 }

ItemRoulette/index.php

@@ -1,14 +1,12 @@
 <?php
 require_once 'ItemRoulette.php';
 $prize = new ItemRoulette;
-$prize->items = ['item-1' => 0.25, 'item-2' => 0.5, 'item-3' => 5, 'item-4' => 4.25333];
+$items = ['item-1' => 0.25, 'item-2' => 0.5, 'item-3' => 5, 'item-4' => 4.25333];
+$prize->setItems($items);
 var_dump($prize->loot());
 var_dump($prize->loot(5));
 
-/*array(1) {
-  [0]=>
-  string(6) "item-4"
-}
+/*string(6) "item-4"
 array(5) {
   [0]=>
   string(6) "item-3"

README.md

@@ -3,7 +3,6 @@
 Генерация списка случайного дропа по заданным параметрам.
 
 Функция на вход получает массив состоящий из служебных названий предметов и процент выпадения.
-
 ```php 
 [
     'item-1' => '0.25', // 25%
@@ -11,10 +10,7 @@
     'item-3' => '0.5', // 50%
 ]
 ```
-
-На выходе функция вернёт массив названий `['item-3', 'item-3', 'item-1']` в зависимости от выбора количества призовых
-предметов.
+На выходе функция вернёт или одно название выпавшего предмета `item-3`, или массив названий `['item-3', 'item-3', 'item-1']` в зависимости от того, с какими параметрами запущена функция.
 
 ## Требования
-
 `PHP 7.4`. По запросу можно переписать под версии вплоть до `PHP 5.3`.