Я хочу, чтобы минимум был 0, но вычитание по-прежнему отрицательное

Я начинаю в ООП с PHP, и я хочу, чтобы 2 противника сражались с функцией боя. Я хочу иметь переменную $degat, которая является результатом произвольного числа от 1 до силы атакующего - (минус) ловкости защитника. Таким образом, результат должен быть минимум 0.

У меня есть этот код, и когда я его var_dump, у меня все еще есть отрицательное значение. Я не вижу, что я делаю неправильно.

public function fight(Fighter $target)
{

    // $degat = (rand(1, $this->strength) - $target->dexterity) >= 0 ? (rand(1, $this->strength) - $target->dexterity) : 0;
    if ((rand(1, $this->strength) - $target->dexterity) > 0) {
        $degat = (rand(1, $this->strength) - $target->dexterity);
    } else {
        $degat = 0;
    };

    var_dump($degat);

    $target->life -= $degat;
}

Спасибо !

Ожидаете ли вы, что возвращаемое значение rand() будет детерминированным?

Siguza 28.03.2022 19:58
Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Symfony Station Communiqué - 7 июля 2023 г
Symfony Station Communiqué - 7 июля 2023 г
Это коммюнике первоначально появилось на Symfony Station .
Оживление вашего приложения Laravel: Понимание режима обслуживания
Оживление вашего приложения Laravel: Понимание режима обслуживания
Здравствуйте, разработчики! В сегодняшней статье мы рассмотрим важный аспект управления приложениями, который часто упускается из виду в суете...
Установка и настройка Nginx и PHP на Ubuntu-сервере
Установка и настройка Nginx и PHP на Ubuntu-сервере
В этот раз я сделаю руководство по установке и настройке nginx и php на Ubuntu OS.
Коллекции в Laravel более простым способом
Коллекции в Laravel более простым способом
Привет, читатели, сегодня мы узнаем о коллекциях. В Laravel коллекции - это способ манипулировать массивами и играть с массивами данных. Благодаря...
Как установить PHP на Mac
Как установить PHP на Mac
PHP - это популярный язык программирования, который используется для разработки веб-приложений. Если вы используете Mac и хотите разрабатывать...
0
1
42
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Ну, проблема здесь в том, что каждый раз, когда вы вызываете rand, результат разный. Итак, вы должны сохранить выражение с помощью rand, а затем использовать его повторно.

Кроме того, все это можно было бы сделать немного короче с использованием тернарного оператора.

$expr = (rand(1, $this->strength) - $target->dexterity);

$degat = $expr > 0 ? $expr : 0

Некоторые примечания:

  • rand не генерирует криптографически безопасные значения и также может зависеть от начального числа, вы также можете рассмотреть возможность использования random_int, я думаю, что это, безусловно, помогает рандомизировать вашу игру.

  • подробнее о тернарном операторе здесь: https://www.php.net/manual/en/language.operators.comparison.php

Офигенный ответ! Большое спасибо! Спасибо за все подробности.

trueChoan 28.03.2022 22:38

@trueChoan, если ответ решил проблему, не могли бы вы поставить галочку, пожалуйста?

nicael 28.03.2022 22:42
Ответ принят как подходящий

Вместо того, чтобы сохранять результат rand(1, $this->strength) - $target->dexterity) в переменной, вы также можете полностью опустить оператор if и сократить свой метод до:

public function fight(Fighter $target)
{
    // max(a,b) returns the higher number from a and b
    $target->life -= max(0, rand(1, $this->strength) - $target->dexterity);
}

Другие вопросы по теме