Рефакторинг.

* Замена цикла `for` на функцию `pow()`;
* Функция `countDecimals()` заменена на `randMax()`, выполняющую более комплексную задачу благодаря использованию `pow()`;

ItemRoulette/ItemRoulette.php

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

README.md

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