최대 값을 초과하지 않고 어떻게 변수를 증가시킬 수 있습니까?

저는 학교를위한 간단한 비디오 게임 프로그램을 작업 중이며 해당 메서드가 호출되면 플레이어가 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

나는 내 구문이 완벽하지 않다는 것을 이해하지만 내 질문은 더 나은 방법이 될 수 있다는 것입니다.

이를 포화 산술 이라고 합니다.

그냥 할 것입니다. 기본적으로 최소 100 (최대 체력)과 추가 점수 15 점 사이의 체력이 필요합니다. 사용자의 건강이 100을 초과하지 않도록합니다.

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

히트 포인트가 0 아래로 떨어지지 않도록 유사한 함수를 사용할 수 있습니다 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가 언급했듯이 때때로 멀티 스레딩 (한 번에 여러 코드 블록 실행)을 사용하면 어떤 시점 에서든 상태가 100을 넘으면 문제가 발생할 수 있으며 상태 속성을 여러 번 변경하는 것도 좋지 않을 수 있습니다. 이 경우 다른 답변에서 언급했듯이 이렇게 할 수 있습니다.

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 미만 또는 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;
    }
}

편집하다

추가 정보

플레이어가 손상을 입었을 때도 똑같이 할 수 있지만 어쨌든 0이되기 때문에 minHealth가 필요하지 않습니다. 이렇게하면 동일한 코드로 모든 양을 손상시키고 치유 할 수 있습니다.

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

도우미 클래스에서 정적 메서드를 만들 것입니다. 이렇게하면 일부 경계에 맞아야하는 모든 값에 대해 코드를 반복하는 대신 하나의 다목적 방법을 사용할 수 있습니다. 최소값과 최대 값을 정의하는 두 개의 값과 해당 범위 내에서 고정되는 세 번째 값을 허용합니다.

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이면 100이 아닌 98이됩니다.

스레드 안전을 원하면 동기화 된 블록을 사용하는 대신이 방법을 사용합니다.

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