Bir maksimum değeri aşmadan bir değişkeni nasıl artırabilirim?

Okul için basit bir video oyun programı üzerinde çalışıyorum ve bu yöntem çağrılırsa oyuncunun 15 sağlık puanı alacağı bir yöntem geliştirdim. Sağlığı maksimum 100'de tutmalıyım ve bu noktada sınırlı programlama yeteneğimle böyle bir şey yapıyorum.

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

Sözdizimimin mükemmel olmadığını anlıyorum ama sorum şu: Bunu yapmanın daha iyi bir yolu ne olabilir, çünkü hasar puanlarıyla da benzer bir şey yapmalı ve 0'ın altına düşmemeliyim.

Buna doygunluk aritmetiği denir .

Ben sadece bunu yapardım. Temel olarak 100 (maksimum sağlık) ile 15 ekstra puanla sağlığın ne olacağı arasındaki minimum değeri alır. Kullanıcının sağlığının 100'ü geçmemesini sağlar.

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

Hitpoints sıfırın altına düşmediği emin olmak için, benzer bir işlevi kullanabilirsiniz: Math.max.

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

sadece sağlığa 15 ekleyin, yani:

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

Bununla birlikte, mülayim bir şekilde belirttiği gibi, bazen çoklu iş parçacığı (aynı anda çalıştırılan birden fazla kod bloğu) ile sağlığın herhangi bir noktada 100'ün üzerine çıkması sorunlara neden olabilir ve sağlık özelliğini birden çok kez değiştirmek de kötü olabilir. Bu durumda, diğer cevaplarda da belirtildiği gibi bunu yapabilirsiniz.

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

intYukarıdakilerin her biri için ayrı bir vakaya ihtiyacınız yoktur 85. Sadece bir tane alın else, böylece sağlık zaten 86veya daha yüksekse, o zaman doğrudan olarak ayarlayın 100.

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

Bence bunu yapmanın deyimsel, nesne yönelimli bir yolu setHealth, Charactersınıfta bir fikir sahibi olmaktır . Bu yöntemin uygulanması şöyle görünecektir:

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

Bu, ne ayarladığınızdan bağımsız olarak, sağlığın 0'ın altına veya 100'ün üzerine çıkmasını önler.

Sizin getHealed()uygulaması sadece bu olabilir:

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

CharacterBir getHealed()yönteme sahip olmanın mantıklı olup olmadığı , okuyucuya bırakılmış bir alıştırmadır :)

Ben sadece daha yeniden kullanılabilir bir kod dilimi sunacağım, bu en küçük değil ama onu herhangi bir miktarda kullanabilirsiniz, bu yüzden hala söylenmeye değer

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

Yaptığınız oyuna istatistik eklemek istiyorsanız 100'ü bir maxHealth değişkenine de değiştirebilirsiniz, böylece tüm yöntem bunun gibi olabilir.

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

DÜZENLE

Ek bilgi için

Aynı şeyi oyuncu hasar aldığında da yapabilirsiniz, ancak minHealth'e ihtiyacınız olmaz çünkü bu zaten 0 olurdu. Bu şekilde yapmak, aynı kodla herhangi bir miktara zarar verebilir ve iyileştirebilirsiniz.

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

Bir yardımcı sınıfta statik bir yöntem yapardım. Bu şekilde, bazı sınırlara sığması gereken her değer için kodu tekrarlamak yerine, tek bir çok amaçlı yönteme sahip olabilirsiniz. Min ve maks tanımlayan iki değer ve bu aralık içinde kenetlenecek üçüncü bir değer kabul eder.

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

Durumunuz için minimum ve maksimum sağlığınızı bir yerde beyan edersiniz.

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

Ardından min, maks ve ayarlanmış sağlığınızda geçen işlevi çağırın.

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

Bunun bir okul projesi olduğunu biliyorum, ancak oyununuzu daha sonra genişletmek ve iyileştirme gücünüzü yükseltmek istiyorsanız, işlevi şu şekilde yazın:

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

ve işlevi çağırın:

getHealed(15);
getHealed(25);

...vb...

Ayrıca işleve yerel olmayan bir değişken oluşturarak maksimum HP'nizi oluşturabilirsiniz. Hangi dili kullandığınızı bilmediğim için, yanlış sözdizimine sahip olabileceğinden bir örnek göstermeyeceğim.

Belki bu?

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

Küstah olmak ve kodunuzu bir satıra sığdırmak istiyorsanız, üçlü bir operatör kullanabilirsiniz :

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

Bazı kişilerin bu sözdizimine (muhtemelen) kötü okunabilirlik nedeniyle kaşlarını çatacağını unutmayın!

Bunun işe yarayacağına inanıyorum

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

Açıklama:

• İyileşme için boşluk 15 veya daha az ise, sağlık 100 olacaktır.

• Aksi takdirde boşluk 15'ten büyükse sağlığa 15 ekler.

Örneğin, sağlık 83 ise 98 olur ama 100 olmaz.

İş parçacığı güvenli olmak istersem, senkronize bir blok kullanmak yerine bunu bu şekilde yapardım.

Atomic CompareAndSet, ek yük olmadan senkronize edilmiş olarak aynı sonucu elde eder.

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

Modül operatörünü kullanmanın en basit yolu.

sağlık = (sağlık + 50)% 100;

sağlık hiçbir zaman 100'e eşit veya 100'ü aşmayacaktır.

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