Làm cách nào để tăng một biến mà không vượt quá giá trị tối đa?

Tôi đang làm một chương trình trò chơi điện tử đơn giản cho trường học và tôi đã tạo ra một phương pháp mà người chơi nhận được 15 điểm sức khỏe nếu phương pháp đó được gọi. Tôi phải giữ sức khỏe ở mức tối đa là 100 và với khả năng lập trình hạn chế của mình tại thời điểm này, tôi đang làm một việc như thế này.

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

Tôi hiểu cú pháp của tôi về không hoàn hảo nhưng câu hỏi của tôi là, có thể là cách tốt hơn để làm điều đó, bởi vì tôi cũng phải làm điều tương tự với các điểm thiệt hại và không được xuống dưới 0.

Đây được gọi là số học bão hòa .

Tôi sẽ chỉ làm điều này. Về cơ bản, nó cần tối thiểu giữa 100 (máu tối đa) và máu sẽ là bao nhiêu với 15 điểm cộng thêm. Nó đảm bảo rằng sức khỏe của người dùng không vượt quá 100.

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

Để đảm bảo rằng hitpoints không giảm xuống dưới mức zero, bạn có thể sử dụng một chức năng tương tự: Math.max.

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

chỉ cần thêm 15 vào sức khỏe, do đó:

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

Tuy nhiên, như nhạt nhẽo đã lưu ý, đôi khi với đa luồng (nhiều khối mã thực thi cùng một lúc) có sức khỏe vượt quá 100 tại bất kỳ thời điểm nào có thể gây ra vấn đề và việc thay đổi thuộc tính sức khỏe nhiều lần cũng có thể không tốt. Trong trường hợp đó, bạn có thể làm điều này, như đã đề cập trong các câu trả lời khác.

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

Bạn không cần một trường hợp riêng biệt cho từng trường hợp inttrên 85. Chỉ cần có một cái else, để nếu sức khỏe đã 86trở lên, thì bạn chỉ cần đặt trực tiếp thành 100.

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

Tôi nghĩ rằng một cách thành ngữ, hướng đối tượng để làm điều này là có một setHealthtrên Characterlớp. Việc triển khai phương thức đó sẽ giống như sau:

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

Điều này ngăn không cho sức khỏe xuống dưới 0 hoặc cao hơn 100, bất kể bạn đặt nó thành gì.

Việc getHealed()triển khai của bạn chỉ có thể là:

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

Liệu phương pháp Charactercó-một có ý nghĩa hay không getHealed()là một bài tập được để lại cho người đọc :)

Tôi chỉ sẽ cung cấp một đoạn mã có thể tái sử dụng nhiều hơn, nó không phải là đoạn mã nhỏ nhất nhưng bạn có thể sử dụng nó với bất kỳ số lượng nào nên nó vẫn đáng được nói

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

Bạn cũng có thể thay đổi 100 thành biến maxHealth nếu bạn muốn thêm số liệu thống kê vào trò chơi của bạn, vì vậy toàn bộ phương pháp có thể giống như thế này

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

BIÊN TẬP

Để biết thêm thông tin

Bạn có thể làm tương tự khi người chơi bị hư hại, nhưng bạn sẽ không cần minHealth vì dù sao thì đó cũng sẽ là 0. Làm theo cách này, bạn sẽ có thể gây sát thương và chữa lành bất kỳ số tiền nào với cùng một mã.

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

Tôi sẽ tạo một phương thức tĩnh trong một lớp trợ giúp. Bằng cách này, thay vì lặp lại mã cho mọi giá trị cần phù hợp với một số ranh giới, bạn có thể có một phương pháp đa năng. Nó sẽ chấp nhận hai giá trị xác định giá trị tối thiểu và tối đa, và giá trị thứ ba được kẹp trong phạm vi đó.

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

Đối với trường hợp của bạn, bạn sẽ khai báo sức khỏe tối thiểu và tối đa của mình ở đâu đó.

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

Sau đó, gọi hàm chuyển theo trạng thái tối thiểu, tối đa và điều chỉnh của bạn.

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

Tôi biết đây là một dự án trường học, nhưng nếu bạn muốn mở rộng trò chơi của mình sau này và có thể nâng cấp sức mạnh chữa bệnh của mình, hãy viết hàm như sau:

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

và gọi ra hàm:

getHealed(15);
getHealed(25);

...Vân vân...

Hơn nữa, bạn có thể tạo HP tối đa của mình bằng cách tạo một biến không phải là cục bộ của hàm. Vì tôi không biết bạn đang sử dụng ngôn ngữ nào nên tôi sẽ không hiển thị ví dụ vì nó có thể sai cú pháp.

Có lẽ điều này?

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

Nếu bạn muốn trở nên táo bạo và phù hợp với mã của mình trên một dòng, bạn có thể sử dụng toán tử bậc ba :

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

Lưu ý rằng một số người sẽ cau mày với cú pháp này do (được cho là) ​​khả năng đọc kém!

Tôi tin rằng điều này sẽ làm được

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

Giải trình:

• Nếu khoảng cách hồi máu là 15 hoặc ít hơn, máu sẽ trở thành 100.

• Ngược lại nếu khoảng cách lớn hơn 15, nó sẽ cộng thêm 15 vào máu.

Vì vậy, ví dụ: nếu sức khỏe là 83, nó sẽ trở thành 98 chứ không phải 100.

Nếu tôi muốn an toàn cho chuỗi, tôi sẽ làm theo cách này hơn là sử dụng một khối được đồng bộ hóa.

So sánh nguyên tử đạt được kết quả tương tự như được đồng bộ hóa mà không có chi phí.

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

Cách đơn giản nhất bằng cách sử dụng toán tử mô đun.

máu = (máu + 50)% 100;

sức khỏe sẽ không bao giờ bằng hoặc vượt quá 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;
    }