Як я можу збільшити змінну, не перевищуючи максимального значення?

Я працюю над простою програмою відеоігор для школи, і я створив метод, коли гравець отримує 15 балів здоров’я, якщо цей метод буде викликаний. Я повинен тримати здоров'я на максимум 100, і з моїми обмеженими можливостями програмування на даний момент я роблю щось подібне.

public void getHealed(){
    if(health <= 85)
        health += 15;
    else if(health == 86)
        health += 14;
    else if(health == 87)
    health += 13; 
}// this would continue so that I would never go over 100

Я розумію, що мій синтаксис про не ідеальний, але моє запитання полягає в тому, що може бути кращим способом це зробити, тому що мені також доводиться робити подібне з точками пошкодження і не опускатися нижче 0.

Це називається арифметикою насичення .

Я б просто зробив це. В основному потрібно мінімум від 100 (максимальне здоров’я) і те, яке здоров’я було б із 15 додатковими балами. Це гарантує, що здоров'я користувача не перевищує 100.

public void getHealed() {
    health = Math.min(health + 15, 100);
}

Для того, щоб переконатися , що хітпоінтов не опускається нижче нуля, ви можете використовувати аналогічну функцію: Math.max.

public void takeDamage(int damage) {
    if(damage > 0) {
        health = Math.max(health - damage, 0);
    }
}

просто додайте 15 до здоров'я, отже:

health += 15;
if(health > 100){
    health = 100;
}

Однак, як зауважив bland, іноді при багатопотоковості (декілька блоків коду, що виконуються одночасно) перевищення рівня здоров'я у будь-який момент може становити проблеми, і зміна властивості здоров'я кілька разів також може бути поганою. У такому випадку ви можете зробити це, як зазначено в інших відповідях.

if(health + 15 > 100) {
    health = 100;
} else {
    health += 15;
}

Вам не потрібен окремий випадок для кожного intвищезазначеного 85. Просто знайдіть його else, щоб, якщо стан здоров’я вже 86або вище, просто встановити його безпосередньо на 100.

if(health <= 85)
    health += 15;
else
    health = 100;

Я думаю , ідіоматичне, об'єктно - орієнтований спосіб зробити це , щоб мати setHealthна Characterкласі. Реалізація цього методу буде виглядати так:

public void setHealth(int newValue) {
    health = Math.max(0, Math.min(100, newValue))
}

Це запобігає зниженню рівня здоров’я нижче 0 або вище, незалежно від того, на що ви його встановили.

Ваша getHealed()реалізація може бути такою:

public void getHealed() {
    setHealth(getHealth() + 15);
}

Чи є сенс Characterмати getHealed()метод - це вправа, що залишається за читачем :)

Я просто запропоную шматок коду для багаторазового використання, його не найменший, але ви можете використовувати його з будь-якою кількістю, так що його все ще варто сказати

health += amountToHeal;
if (health >= 100) 
{ 
    health = 100;
}

Ви також можете змінити 100 на змінну maxHealth, якщо хочете додати статистику до гри, яку ви робите, тому весь метод може бути приблизно таким

private int maxHealth = 100;
public void heal(int amountToHeal)
{
    health += amountToHeal;
    if (health >= maxHealth) 
    { 
        health = maxHealth;
    }
}

РЕДАГУВАТИ

Для отримання додаткової інформації

Ви можете зробити те ж саме, коли гравець пошкоджений, але вам не знадобиться minHealth, оскільки це все одно буде 0. Роблячи це таким чином, ви зможете пошкодити та зцілити будь-яку суму за допомогою того ж коду.

health = health < 85 ? health + 15 : 100;

Я б зробив статичний метод у допоміжному класі. Таким чином, замість повторення коду для кожного значення, яке повинно входити в певні межі, ви можете використовувати один універсальний метод. Він приймає два значення, що визначають min і max, і третє значення, яке слід затиснути в цьому діапазоні.

class HelperClass
{
    // Some other methods

    public static int clamp( int min, int max, int value )
    {
        if( value > max )
            return max;
        else if( value < min )
            return min;
        else
            return value;
    }
}

Для Вашого випадку Ви б десь декларували свій мінімальний і максимальний стан здоров’я.

final int HealthMin = 0;
final int HealthMax = 100;

Потім зателефонуйте функції, що передає ваш мінімальний, максимальний та скоригований стан здоров'я.

health = HelperClass.clamp( HealthMin, HealthMax, health + 15 );

Я знаю, що це шкільний проект, але якщо ви хотіли пізніше розширити свою гру і мати можливість оновити свою цілющу силу, напишіть функцію так:

public void getHealed(healthPWR) {
    health = Math.min(health + healthPWR, 100);
}

і викликати функцію:

getHealed(15);
getHealed(25);

... тощо ...

Крім того, ви можете створити свій максимальний HP, створивши змінну, яка не є локальною для функції. Оскільки я не знаю, якою мовою ви користуєтесь, я не буду показувати приклад, оскільки він може мати неправильний синтаксис.

Може це?

public void getHealed()
{
  if (health <= 85)
  {
    health += 15;
  } else
  {
    health = 100;
  }
}

Якщо ви хочете бути зухвалим і вмістити свій код в один рядок, ви можете скористатися трійковим оператором :

health += (health <= 85) ? 15 : (100 - health);

Зверніть увагу, що деякі люди будуть ненавидіти цей синтаксис через (можливо, погану читабельність)!

Я вірю, що це вдасться

if (health >= 85) health = 100;
else health += 15;

Пояснення:

• Якщо розрив у лікуванні становить 15 або менше, здоров’я стане 100.

• В іншому випадку, якщо розрив більше 15, це додасть 15 здоров’ю.

Так, наприклад: якщо стан здоров’я становить 83, він стане 98, але не 100.

Якби я хотів бути потокобезпечним, я б зробив це таким чином, а не використовуючи синхронізований блок.

Atomic compareAndSet досягає того самого результату, що і синхронізований без накладних витрат.

AtomicInteger health = new AtomicInteger();

public void addHealth(int value)
{
    int original = 0;
    int newValue = 0;
    do
    {
        original = health.get();
        newValue = Math.min(100, original + value);
    }
    while (!health.compareAndSet(original, newValue));
}

Найпростіший спосіб використання оператора модуля.

здоров'я = (здоров'я + 50)% 100;

здоров'я ніколи не буде рівним або перевищуватиме 100.

   private int health;
    public void Heal()
    {
        if (health > 85)
            health = 100;
        else
            health += 15;
    }
    public void Damage()
    {
        if (health < 15)
            health = 0;
        else
            health -= 15;
    }