回数処理

Question

n回このメソッドを呼べば別のメソッドを呼ぶというものを実装したいけど、うまくいかないので質問させていただきました。

ソースコードは以下の通りです。
private boolean canMove(Rect movedRect) {

int horizontalBlock = movedRect.centerX() / blockSize;

int verticalBlock = movedRect.centerY() / blockSize;

if (placeHorizontalBlock != horizontalBlock
|| placeVerticalBlock != verticalBlock) {

setTargetBlock(verticalBlock, horizontalBlock);
placeHorizontalBlock = horizontalBlock;
placeVerticalBlock = verticalBlock;
}

int yLength = targetBlock.length;

for (int y = 0; y < yLength; y++) {
for (int x = 0; x < xLength; x++) {

int clearCounter =0; //最初は0 ・・・(1)

Block block = targetBlock[y][x];

if (block == null) {

continue;
}

if (block.type == Block.TYPE_WALL && Rect.intersects(block.rect, movedRect)) {

return false;
}

//ゴールと重なったらonGoalを呼び出す ・・・(2)

else if (block.type == Block.TYPE_GOAL
&& block.rect.contains(movedRect.centerX(), movedRect.centerY())) {

clearCounter += 1; //呼び出される度にclearCounterに１足したい

if ( clearCounter < 3) { //3回未満ならonGoalを呼びたい

eventCallback.onGoal();
return true;

} else { //3回以上ならonChangeを呼びたい

eventCallback.onChange();
return true;
}

}
}

}
return true;

}

(2)のところで、中に条件分岐をいれて、場合分けで処理したいのですが、"clearCounter += 1;"という処理をいれても、呼ばれる度にリセットされるのではないか？という結論にいたりましたが、解決方法がわかりません。

初めから、(1)を"clearCounter = 5;" などにすると、onChangeが呼ばれることは確認しました。

ご回答のほうよろしくおねがいいたします。

Comments

[KSwordOfHaste]

質問を編集して「</>」を使ってマークアップしたほうがいいですよ。閲覧者のみなさんが読みにくいです。

[swordone]

コードをマークダウンしてください。

[udooon]

ご指摘ありがとうございます

[swordone]

マークダウンのやり方間違ってる

[carimatics]

バッククォート3つで囲んだ部分がコードブロックとして認識されます。teratailのヘルプを参考にしてください。

Answer 1

ローカル変数（メソッド内で宣言した変数）は、メソッドを呼び出す度に初期化されます。
この場合、canMove(Rect)内でint clearCounter =0;と記述しているので、メソッドが呼び出される度にclearCounterの値は0で初期化されます。

clearCounterをインスタンス変数として保持するか、もしくは呼び出し元から引数としてclearCounterを渡すなどの手段取れば解決するのではないでしょうか。

Comments

[udooon]

ありがとうございます。
挑戦してみます。