1

モンスターを開始/停止するのに助けが必要です。
次のように生成します。

for (int i = 0; i < 20; i++) {
    boolean notFree = true;
    int x = 0, y = 0;
    // while not free change
    // not determinstisch setting!
    while (notFree) {
        x = (int) (Math.random() * Config.XSIZE);
        y = (int) (Math.random() * Config.YSIZE);
        if (map.getMapPosition(x, y) == Config.EMPTYPOSITION) {
            notFree = false;
        }
    }
    switch ((int) Math.floor(Math.random() * 2)) {
    case 0:
        monsterVerktor.add(new Monster(context, x, y, this.map, 1,
                this.charac));
        break;
    case 1:
        monsterVerktor.add(new DeathReaper(context, x, y, this.map, 1,
                this.charac));
        break;
    default:
        break;
    }
}

そして、私は次のようにそれらを停止します:(開始はStartmovingと同じです...)

public void stopAllMonsters() {
    for (Monster monster : monsterVerktor) {
        monster.getControl().stopMovingThread();
    }
 }

thread.stopMovingThreadは次のように機能します。

public void stopMovingThread() {
    this.monster.setAlife(false);
    running = false;
    moveDone = true;
    boolean retry = true;
    while (retry) {
        try {
            thread.join();
            retry = false;
        } catch (InterruptedException e) {
            // try again shutting down the thread
        }
    }
}

run()へ:

public void startMovementThread() {
    running = true;
    thread = new Thread() {

        @Override
        public void run() {
            while (running) {
                Log.d(TAG, "run Thread");
                // get a little randomness in the movement ;)
                try {
                    // up to 0.5s sleeping till next moving
                    sleep(new Random().nextInt(Config.RANDOMSLEEP));
                } catch (InterruptedException e1) {}
                while (monster.isAlife()) {// check if alife
                    moveDone = false; // resett movement Done
                    noMove = false;
                    // Charakter in near if one is in near he
                    // will move to it.
                    if (!checkCharInAggroRange()) {
                        noMove = rndMove(); // if we have no move
                    }
                    while (!moveDone) {
                        timeBegin = System.currentTimeMillis();
                        // if hes not done with the move
                        if (monster.moveToX == monster.positionX
                            && monster.moveToY == monster.positionY) {
                            if (noMove) {
                                try {// sleep because no movement with a bit
                                        // randomness
                                    sleep(Config.MONSTERMOVINGTIME
                                        + (new Random()
                                                .nextInt(Config.RANDOMSLEEP)));
                                } catch (InterruptedException e) {
                                    Log.d(TAG,
                                        "An error occured during sleep because"
                                            + " of no Animatino");
                                }
                            }
                            // false after sleep
                            moveDone = false;
                            break; // already at the right position!
                            // movetoX is left of position
                        } else if (monster.moveToX > monster.positionX) {
                            monster.positionX++;
                            // movetoX is left of it
                        } else if (monster.moveToX < monster.positionX) {
                            monster.positionX--;
                            // movetoY is left of position
                        } else if (monster.moveToY > monster.positionY) {
                            monster.positionY++;
                            // movetoY is left of position
                        } else if (monster.moveToY < monster.positionY) {
                            monster.positionY--;
                        }
                        // sleep if hes moving to fast!
                        timeDiff = System.currentTimeMillis() - timeBegin;
                        sleepTimer = (int) (Config.MONSTERTIMING - timeDiff);
                        if (sleepTimer > 0) { // if >0 we are fast enough
                            // and can sleep a bit ;)
                            try {
                                sleep(sleepTimer);
                            } catch (InterruptedException e) {
                                Log.d(TAG, "Gameloop thread cant sleep");
                            }
                        }
                    }
                }
                try {
                    sleep(Config.RESPAWNTIMER);
                } catch (InterruptedException e) {
                    Log.d(TAG, "Monsterthread cant sleep");
                }
                // respawn it after a sleep :)
                respawn();
            }
        }
    };
    thread.start();
}

Androidを停止してこのように起動したい場合、Androidは動作を停止しますが、理由がわかりません。また、このようにレンダリングを停止/開始します。(まあ、それは実行可能であり、これはそうではありません)

モンスターisAlifeなど:

public class Monster extends Drawable {

    private static final String TAG = Monster.class.getSimpleName();
    private int fullLife; // full life
    private int curLife; // current life
    public Context context;
    private Character charac;
    // ..
    // Basic monsterstuff
    private boolean alife, moveDone;
    private int level;
    public Status status;
    private MonsterControl control;

    // ..
    public Monster(Context context, int mapPosX, int mapPosY, Map map,
            int level, Character cha) {
        this.context = context;
        this.map = map;
        this.setCharac(cha);
        this.mapPosition[0] = mapPosX;
        this.mapPosition[1] = mapPosY;
        this.status = Status.IDLE;
        // example for full life calculation
        this.fullLife = level * 100 + ((int) (Math.random() * 10 * level)); // Examples
        this.curLife = this.fullLife;
        this.map.setMapPosition(mapPosX, mapPosY, Config.MONSTERSTATE);
        // set monster position
        // ..
        // load the sprite bitmap
        // ...
        // Change this later!
        alife = true;
        Log.d(TAG, "Monster created");
        // Starting the Controler
        control = new MonsterControl(this, this.charac);
        control.startMovementThread();
        Log.d(TAG, "Monster start moving");
        // exemplarisch cut of the sprite
        this.monsterPicAnimation();
    }
}

ここでaLifeのゲッター/セッター

public boolean isAlife() {
    return alife;
}

public void setAlife(boolean alife) {
    this.alife = alife;
}
4

3 に答える 3

3

まず、「Androidが動作を停止します」というコメントは、クラッシュを意味していますか?LogCatトレースを取得しましたか?

それ以外は、この分析は正しいですか?

1.モンスターを作成します。

running = true;
isAlife = true; //or whatever monster.isAlife() checks

2.スレッドが開始します。

while (running) {
  ...
  while (monster.isAlife()) {
    ...
  }
}

3.(1)実行をfalseに設定し、(2)スレッドに参加することで、スレッドを停止しようとしますか?


それがすべて真実であると仮定します。while(monster.isAlife())モンスタースレッドは、ネストされたループから何かがキックされるまで実行されます。それが終了すると、while(running)はfalseと評価され、スレッドは終了して、コレクション内の次のモンスターに移動します。これが終了しないと、メインスレッドをでロックしている間、各モンスターが死ぬのを待つことになりますjoin()

于 2012-12-13T17:58:59.387 に答える
2

私が詳細に調べていないコードのロジックは別として、共有変数の周りの同期が不足しています。特に、現状では、コードは、スレッド(in startMovementThread)がメインスレッドからフラグに加えられた変更を監視できることを保証するものではありません。

最初のステップとして、これらの変数を揮発性にすることをお勧めします。

private volatile boolean alife, moveDone;

ただし、他の問題がある可能性があることに注意してください(要件によって異なります)。たとえば、これらのステートメントが次の場所で実行される場合startMovementThread

if (monster.moveToX == monster.positionX ...)
monster.positionX++;

そのスレッドで行われた変更は、メインスレッドからは見えない可能性があります。その逆も同様です。monster.positionX++それが問題になる可能性がある場合は、それらの変数へのアクセスも同期する必要があります(たとえば、アトミックではないため、変数を揮発性にするだけでは不十分です)。

于 2012-12-13T22:51:43.283 に答える
0

私はそれを「良い」アイデアで自分で解決しました。問題は、私が彼らをある時点で眠らせたことでした。彼らが眠っている場合、それはスレッドを停止/参加することができず、その「実行中」かどうかさえ監視しません。だから私はこのような小さな方法でそれを解決しました:

private void breakableSleep(int i, int intervall){
    int x = 0;
    while(running && x <= i){
        try {
            this.thread.sleep(intervall);
        } catch (InterruptedException e) {
            Log.d(TAG, "Monsterthread cant sleep");
        }
        x++;
    }
}

間隔ごとに停止できます。確かにバッテリーがいくらかかかりますが、モンスターが死んだ場合(リスポーン)に約3秒間スリープするため、モンスターを「高速」に停止する他の方法は見つかりませんでした。私の意見では見栄えがよくなかったため、startMovementThread全体を書き直しました。 。

よろしくお願いします

于 2012-12-16T16:28:46.097 に答える