classes/Battles/InventoryItem.php

@@ -27,7 +27,7 @@ class InventoryItem extends Item
 
     public function printInfo()
     {
-        echo $this->getAllInfo();
+        parent::printAllInfo();
         if ($this->present) {
             echo "<p style='color: maroon; font-style: italic'>Это подарок от $this->present. Вы не можете передать его кому-либо ещё.</p>";
         }
@@ -48,8 +48,9 @@ IMG;
         }
     }
 
-    public function printControls(){
-        // Для кнопок управления под картинкой.
+    public function printControls()
+    {
+        /* Тут будут кнопки под картинкой. */
     }
 
     private function dressStatsChecks(): ?string
@@ -129,9 +130,14 @@ IMG;
         return $error ?? true;
     }
 
-    public static function destroyItem($itemId)
+    /**
+     * @param $itemId
+     *
+     * @return bool
+     */
+    public static function destroyItem($itemId): bool
     {
-        DBPDO::INIT()->execute('delete from inventory where dressed_slot = 0 and owner_id = ? and item_id = ?', [$_SESSION['uid'], $itemId]);
+        return DBPDO::INIT()->execute('delete from inventory where dressed_slot = 0 and owner_id = ? and item_id = ?', [$_SESSION['uid'], $itemId]);
     }
 
     /** Надеюсь, временная заглушка, которая объединяет get_meshok() и другую выдачу одной строкой.

classes/Battles/Item.php

@@ -102,14 +102,6 @@ class Item
                 $this->typename = 'Хлам';
         }
 
-        $this->item_cost = $this->calculateItemCost();
-    }
-
-    /** Рассчёт стоимости предмета в зависимости от его характеристик.
-     * @return int
-     */
-    private function calculateItemCost(): int
-    {
         $sum_stats =
             $this->add_strength +
             $this->add_dexterity +
@@ -129,7 +121,8 @@ class Item
         $mods_cost_modifier = 2 + floor($sum_mods / 50);
         $damage_cost_modifier = 1 + floor($sum_damage / 100);
         $result = intval($sum_stats * $stats_cost_modifier + $sum_mods * $mods_cost_modifier + $sum_damage * $damage_cost_modifier);
-        return $result < 1 ? 1 : $result;
+
+        $this->item_cost = $result < 1 ? 1 : $result;
     }
 
     protected function wrap(int $number): string
@@ -141,51 +134,50 @@ class Item
         }
     }
 
-    public function getAllInfo(): string
+    protected function printAllInfo()
     {
         $needsLines = [
-            'сила' => $this->need_strength,
-            'ловкость' => $this->need_dexterity,
-            'интуиция' => $this->need_intuition,
-            'выносливость' => $this->need_endurance,
-            'интеллект' => $this->need_intelligence,
-            'мудрость' => $this->need_wisdom,
+            "сила" => $this->need_strength,
+            "ловкость" => $this->need_dexterity,
+            "интуиция" => $this->need_intuition,
+            "выносливость" => $this->need_endurance,
+            "интеллект" => $this->need_intelligence,
+            "мудрость" => $this->need_wisdom,
         ];
         $addsLines = [
-            'Сила' => $this->add_strength,
-            'Ловкость' => $this->add_dexterity,
-            'Интуиция' => $this->add_intuition,
-            'Выносливость' => $this->add_endurance,
-            'Интеллект' => $this->add_intelligence,
-            'Мудрость' => $this->add_wisdom,
-            'Точность' => $this->add_accuracy,
-            'Увёртливость' => $this->add_evasion,
-            'Шанс крита' => $this->add_criticals,
+            "Сила" => $this->add_strength,
+            "Ловкость" => $this->add_dexterity,
+            "Интуиция" => $this->add_intuition,
+            "Выносливость" => $this->add_endurance,
+            "Интеллект" => $this->add_intelligence,
+            "Мудрость" => $this->add_wisdom,
+            "Точность" => $this->add_accuracy,
+            "Увёртливость" => $this->add_evasion,
+            "Шанс крита" => $this->add_criticals,
         ];
-        $str = "<b>$this->name</b> (Масса: $this->weight)";
-        $str .= '<br> Стоимость: ' . $this->item_cost;
-        $str .= '<br> Долговечность: ' . $this->durability;
-        $str .= "<br><em>$this->typename</em><br>";
+        echo "<b>$this->name</b> (Масса: $this->weight)";
+        echo "<br> Стоимость: " . $this->item_cost;
+        echo "<br> Долговечность: " . $this->durability;
+        echo "<br><em>$this->typename</em><br>";
         foreach ($needsLines as $stat => $value) {
             if ($value > 0) {
-                $str .= "<br>Требуется $stat" . $this->wrap($value);
+                echo "<br>Требуется $stat" . $this->wrap($value);
             }
         }
         foreach ($addsLines as $stat => $value) {
             if ($value) {
-                $str .= "<br>$stat" . $this->wrap($value);
+                echo "<br>$stat" . $this->wrap($value);
             }
         }
         if ($this->add_min_physical_damage && !$this->add_max_physical_damage) {
-            $damage = $this->add_min_physical_damage . ' - ' . $this->add_min_physical_damage;
+            $damage = $this->add_min_physical_damage . " - " . $this->add_min_physical_damage;
         } elseif (!$this->add_min_physical_damage && $this->add_max_physical_damage) {
-            $damage = $this->add_max_physical_damage . ' - ' . $this->add_max_physical_damage;
+            $damage = $this->add_max_physical_damage . " - " . $this->add_max_physical_damage;
         } elseif ($this->add_min_physical_damage && $this->add_max_physical_damage) {
-            $damage = $this->add_min_physical_damage . ' - ' . $this->add_max_physical_damage;
+            $damage = $this->add_min_physical_damage . " - " . $this->add_max_physical_damage;
         }
         if (isset($damage)) {
-            $str .= '<br>Урон: ' . $damage;
+            echo "<br>Урон: " . $damage;
         }
-        return $str;
     }
 }

classes/Battles/ShopItem.php

@@ -49,7 +49,6 @@ SQL;
     private $optype;
     private ?int $shop_item_quantity;
     private ?int $price;
-    public static string $status = '';
 
     public function __construct($row, $operationType = null)
     {
@@ -62,12 +61,11 @@ SQL;
         $this->item_id = $row->item_id ?? $row->id;
     }
 
-    public function printInfo(): string
+    public function printInfo()
     {
-        //$this->printAllInfo();
-        $str = $this->getAllInfo();
+        $this->printAllInfo();
         if ($this->optype === 'buyshop' && $this->shop_item_quantity > 0 && $this->shop_item_quantity < 20) {
-            $str .= "<div style='margin-top: 9px; font-style: italic;'>На складе осталось $this->shop_item_quantity единиц товара!</div>";
+            echo "<div style='margin-top: 9px; font-style: italic;'>На складе осталось $this->shop_item_quantity единиц товара!</div>";
         }
         if ($this->optype === 'sellshop') {
             if ($this->getSellPriceMean() < 50) {
@@ -81,37 +79,36 @@ SQL;
             } else {
                 $goods = 'это превосходное изделие';
             }
-            $str .= "<div style='margin-top: 9px; font-style: italic;'>В среднем за $goods можно выручить <span class='success'>{$this->getSellPriceMean()}</span> кр.</div>";
+            echo "<div style='margin-top: 9px; font-style: italic;'>В среднем за $goods можно выручить <span class='success'>{$this->getSellPriceMean()}</span> кр.</div>";
         }
-        return $str;
     }
 
-    public function printImage(): string
+    public function printImage()
     {
         if (!$this->image) {
             $this->image = 'noitem.png';
         }
-        return "<img src='/i/sh/$this->image' class='item-wrap-normal' alt=''>";
+        echo "<img src='/i/sh/$this->image' class='item-wrap-normal' alt=''>";
     }
 
     //todo наличка после покупки отображается с задержкой.
-    public static function buyItem($id, User $buyer)
+    public static function buyItem($id, User $buyer): string
     {
         $db = new DBPDO();
         $check = $db->ofetch("select * from trade_offers where shop_item_id = ?", $id);
         if (empty($check->shop_item_quantity) || empty($check->shop_item_id)) {
-            self::$status = self::NO_ITEMS_IN_STOCK;
+            return self::NO_ITEMS_IN_STOCK;
         }
 
         // TODO БАРТЕР!
         if (isset($check->barter_item_list_json)) {
-            self::$status = "Работаем по бартеру!";
+            echo "Работаем по бартеру!";
         }
 
         $db->execute(self::BUY_QUERY, [$buyer->getId(), $check->shop_item_id]);
         $item = $db->ofetch("select item_id, name, price from inventory where item_id = ?", $db->lastInsertId());
         if (empty($item->item_id) || empty($item->name)) {
-            self::$status = 'Запрос в базу не прошёл.';
+            return 'Запрос в базу не прошёл.';
         } else {
             $user = new User($_SESSION['uid']);
             // Если не хватает налички, снимаем с банка с комиссией.
@@ -120,12 +117,14 @@ SQL;
                     $bank = new Bank($buyer->getId());
                     $bank->withdrawMoney($item->price);
                 } catch (GameException $e) {
-                    self::$status = 'Банковская ошибка! ' . self::NO_MONEY;
+                    echo 'Банковская ошибка!';
+                    return self::NO_MONEY;
                 }
             } else {
                 $user->setMoney($user->getMoney() - $item->price);
                 $user->saveMoney();
             }
+
         }
 
         if ($check->shop_item_quantity != -1) {
@@ -134,10 +133,10 @@ SQL;
 
         $deloText = $buyer->getLogin() . " купил товар «" . $item->name . "» id:(" . $item->item_id . ") в магазине за " . $item->price . ".";
         GameLogs::addUserLog($buyer->getId(), $deloText);
-        self::$status = "Предмет " . $item->name . " куплен за " . $item->price . ".";
+        return "Предмет " . $item->name . " куплен за " . $item->price . ".";
     }
 
-    public static function sellItem($id, User $seller, $bankTrade = 0)
+    public static function sellItem($id, User $seller, $bankTrade = 0): string
     {
         $db = new DBPDO();
         $item = $db->ofetch('select * from inventory where item_id = ?', $id);
@@ -155,10 +154,11 @@ SQL;
         $deloText = "{$seller->getLogin()} продал товар «{$sellingItemName}» id:($id) в магазине за $sellingPrice кр.";
         GameLogs::addUserLog($seller->getId(), $deloText);
         if ($sellingPrice == 0) {
-            self::$status = "После длительных и изнурительных торгов вы плюнули на всё и просто подарили ваш «{$sellingItemName}» торговцу.";
+            $status = "После длительных и изнурительных торгов вы плюнули на всё и просто подарили ваш «{$sellingItemName}» торговцу.";
         } else {
-            self::$status = "Вы продали «{$sellingItemName}» за $sellingPrice кр.";
+            $status = "Вы продали «{$sellingItemName}» за $sellingPrice кр.";
         }
+        return $status;
     }
 
     /** Подчсчёт средней суммы продажи.
@@ -177,19 +177,18 @@ SQL;
     /**
      * Для кнопок управления под картинкой предмета в зависимости от ситуации.
      */
-    public function printControls(): string
+    public function printControls()
     {
-        if (!in_array($this->optype, ['setmarket', 'unsetmarket', 'buymarket', 'sellshop', 'buyshop',])) {
-            return '';
-        }
-        $str = $this->optype == 'setmarket' ? '<input placeholder=" ' . $this->price . ' " name="cost">' : '';
-        $button_name = self::BUTTON[$this->optype];
-        return <<<FORM
+        if (in_array($this->optype, ['setmarket', 'unsetmarket', 'buymarket', 'sellshop', 'buyshop',])) {
+            $str = $this->optype == 'setmarket' ? '<input placeholder=" ' . $this->price . ' " name="cost">' : '';
+            $button_name = self::BUTTON[$this->optype];
+            echo <<<FORM
 <form method="post">$str
 <input type="hidden" name="itemId" value="$this->item_id">
 <br><input type="submit" name="$this->optype" value="$button_name">
 </form>
 FORM;
+        }
     }
 
     /**

config.php

@@ -9,11 +9,6 @@
 use Battles\Database\DBPDO;
 use Battles\User;
 
-include_once 'classes/Database/db.php';
-include_once 'classes/Database/Mysql.php';
-include_once 'classes/Database/Statement.php';
-include_once 'classes/Database/Exception.php';
-
 ini_set('display_errors', 'On');
 error_reporting(E_ALL);
 const GAMEDOMAIN = "battles.lan";