最大値を超えずに変数をインクリメントするにはどうすればよいですか？

私は学校向けの簡単なビデオゲームプログラムに取り組んでおり、そのメソッドが呼び出された場合にプレイヤーが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;
}

しかし、当たり障り述べたように、時々 （一度に実行されるコードの複数のブロック）をマルチスレッドとで100以上の健康行くを有する任意の点は問題を引き起こす可能性があり、および健康特性を変更する複数回も悪いことができます。その場合、他の回答で述べたように、これを行うことができます。

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

int上記のそれぞれに個別のケースは必要ありません85。ちょうど1つを持っている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未満または100を超えるのを防ぐことができます。

あなたの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;

ヘルパークラスで静的メソッドを作成します。このように、いくつかの境界内に収まる必要があるすべての値に対してコードを繰り返すのではなく、1つの多目的メソッドを持つことができます。最小値と最大値を定義する2つの値と、その範囲内にクランプされる3番目の値を受け入れます。

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;
  }
}

生意気でコードを1行に収めたい場合は、三項演算子を使用できます。

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

（おそらく）読みやすさが悪いために、一部の人々はこの構文に不快感を抱くでしょう。

これでうまくいくと思います

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

説明：

• ヒーリングのギャップが15以下の場合、ヘルスは100になります。

• それ以外の場合、ギャップが15より大きいと、ヘルスが15増加します。

たとえば、ヘルスが83の場合、98になりますが、100ではありません。

スレッドセーフにしたい場合は、同期ブロックを使用するのではなく、この方法を使用します。

アトミック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;
    }