0

誰かがエラーで私を助けてくれることを願っていIndexOutOfBoundsExceptionます。

領地内のユニットの数を設定でき、領地の所有者を設定できますが、防御機能が問題を引き起こしています。

痕跡：

Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.ArrayList.RangeCheck(ArrayList.java:547)
at java.util.ArrayList.get(ArrayList.java:322)
at stackOverflow.Territory.calculateLoses(Territory.java:136)
at stackOverflow.Territory.defend(Territory.java:95)
at stackOverflow.Territory.defend(Territory.java:40)
at stackOverflow.Territory.main(Territory.java:155)



import java.util.ArrayList;
import java.util.Collections;
import java.util.List;

public enum Territory {

// Europe
GREATBRITAIN("Great Britain"), ICELAND("Iceland");

private final String name;
private int numberOfUnits;
private Player player;
private int ad = 0, dd = 0;

private Territory(String name) {
    this.name = name;
    this.numberOfUnits = 0;
    this.player = null;
}

public void setOwner(Player p) {
    this.player = p;
}

public Player getOwner() {
    return this.player;
}

public int getNumberUnits() {
    return this.numberOfUnits;
}

public void setNumberUnits(int units) {
    this.numberOfUnits = units;
}

public boolean defend(Territory attacker) throws Exception {
    return defend(attacker, attacker.numberOfUnits - 1);
}

public boolean defend(Territory attacker, int attackingUnits)
        throws Exception {
    if (attackingUnits > (attacker.getNumberUnits() - 1)) {
        throw new Exception("Invalid number of units");
    }

    attacker.setNumberUnits(attacker.numberOfUnits - attackingUnits);
    Player defender = this.player;
    this.player = null;
    int defendingUnits = this.numberOfUnits;
    this.numberOfUnits = 0;

    if (this.numberOfUnits >= 3 && defendingUnits >= 2) {
        ad = 3;
        dd = 2;
    }
    if (this.numberOfUnits >= 3 && defendingUnits == 1) {
        ad = 3;
        dd = 1;
    }
    if (this.numberOfUnits == 2 && defendingUnits >= 2) {
        ad = 2;
        dd = 2;
    }
    if (this.numberOfUnits == 2 && defendingUnits == 1) {
        ad = 2;
        dd = 1;
    }
    if (this.numberOfUnits == 1 && defendingUnits >= 2) {
        ad = 1;
        dd = 2;
    }
    if (this.numberOfUnits == 1 && defendingUnits == 1) {
        ad = 1;
        dd = 1;
    }
    if (this.name == "Great Britan" || this.name == "Central America"
            || this.name == "Argentina" || this.name == "Egypt"
            || this.name == "Western Australia" || this.name == "India") {
        dd++;
    }

    List<Die> attackerDice = createDice(ad);
    List<Die> defenderDice = createDice(dd);
    System.out.printf("Attacker: %d \tDefender: %d\n", attackingUnits,
            defendingUnits);
    while (attackingUnits > 0 && defendingUnits > 0) {
        roll(attackerDice);
        System.out.println(attackerDice);
        roll(defenderDice);
        System.out.println(defenderDice);

        attackingUnits -= calculateLoses(attackerDice, defenderDice, false);
        defendingUnits -= calculateLoses(defenderDice, attackerDice, true);

        System.out.printf("Attacker: %d \tDefender: %d\n", attackingUnits,
                defendingUnits);
    }

    if (defendingUnits > 0) {
        this.player = defender;
        this.numberOfUnits = defendingUnits;
        return true;
    } else if (attackingUnits > 0) {
        this.numberOfUnits = attackingUnits;
        this.player = attacker.player;
        return false;
    } else {
        // No one owns the territory as all units died
        return false;
    }
}

private List<Die> createDice(int number) {
    List<Die> dice = new ArrayList<Die>();
    for (int i = 0; i < number; i++) {
        dice.add(new Die());
    }
    roll(dice);
    return dice;
}

private void roll(List<Die> dice) {
    for (Die d : dice) {
        d.roll();
    }
    Collections.sort(dice);
}

private int calculateLoses(List<Die> diceOne, List<Die> diceTwo,
        boolean defender) {
    int number = 0;
    for (int i = 0; i < 2; i++) {
        int comparison = diceOne.get(i).compareTo(diceTwo.get(i));
        if (comparison > 0 || (!defender && comparison == 0)) {
            number++;
        }
    }
    return number;
}

String units()
{
    return "" + numberOfUnits;
}

public static void main(String[] args) throws Exception
{
    Territory.GREATBRITAIN.setNumberUnits(5);
    System.out.println(Territory.GREATBRITAIN.getNumberUnits());
    Territory.ICELAND.setNumberUnits(5);
    System.out.println(Territory.ICELAND.getNumberUnits());
    Territory.GREATBRITAIN.defend(Territory.ICELAND);
}
}
4

2 に答える 2

6

どちらかad、ddまたは両方が 0 のようです。

このコードを考えると、それは理にかなっています:

this.numberOfUnits = 0;

// Lots of these, all requiring numberOfUnits to be greater than 0
if (this.numberOfUnits >= 3 && ...)
{
    ad = ...;
    dd = ...;
}

0ifに設定したばかりのときに、どのようにしてこれらのブロックのいずれかに入ることが期待できますか?numberOfUnits

これは、コードの問題の1 つにすぎません。他の人は、スタイルまたは正確さの他の側面を指摘しています. ここですべてのコードを修正しようとするつもりはありませんが、自分でこれをどのように診断できたかを分析する必要があります。たとえば、デバッガーでコードをステップ実行しようとしましたか?

于 2013-04-10T21:01:11.477 に答える
2

このコードは間違っています:

if (this.name == "Great Britan" || this.name == "Central America"
        || this.name == "Argentina" || this.name == "Egypt"
        || this.name == "Western Australia" || this.name == "India") {
    dd++;
}

this.name.equals("Great Britain")文字列がメモリ内の同じ場所に格納されているかどうかではなく、文字列に同じ文字が含まれているかどうかをテストするために使用します。

また、あなたの多くの個別のifステートメントは悪い習慣のように見えます。または何かを使用elseして、少なくとも 1 つのケースにヒットしたことを確認します。this.numberOfUnits >= [stuff]設定後にケースにヒットしていないことは明らかですthis.numberOfUnits = 0;。

于 2013-04-10T21:04:33.337 に答える