Android Studio-JAVA-TIC-TAC-TOE

我已经在android上制作了我的第一个井字游戏，但是我发现了一个bug，我不知道如何修复它

当玩家1在最后一个回合的某些组合中获胜时，如底部水平行或第一个垂直行，当有赢家时，它会弹出-这是一个平局

这里我定义了我的变量

public class MainActivity extends AppCompatActivity {
// 0 yellow ; 1 red;
int activePlayer = 0;

boolean gameIsActive = true;

// 2 unplayed;
int[] gameState = {2, 2, 2, 2, 2, 2, 2, 2, 2};
int[][] winningPositions = {
        {0,1,2}, {3,4,5}, {6,7,8}, //hor
        {0,3,6}, {1,4,7}, {2,5,8}, // ver
        {0,4,8}, {2,4,6}
};         // dia

这是我的逻辑。我知道需要在评论之后的某个地方进行修正：检查赢家！因为它不去槽，如果部分，但进入其他和弹出-这是一个平局

完整的JAVA代码：

public class MainActivity extends AppCompatActivity {
// 0 yellow ; 1 red;
int activePlayer = 0;
boolean gameIsActive = true;
// 2 unplayed;
int[] gameState = {2, 2, 2, 2, 2, 2, 2, 2, 2};
int[][] winningPositions = {
        {0,1,2}, {3,4,5}, {6,7,8}, //hor
        {0,3,6}, {1,4,7}, {2,5,8}, // ver
        {0,4,8}, {2,4,6}
};         // dia
public void dropIn (View view) {
    ImageView counter = (ImageView) view;
    int tappedCounter = Integer.parseInt(counter.getTag().toString());
    if (gameState[tappedCounter] == 2  && gameIsActive) {
        gameState[tappedCounter] = activePlayer;
        counter.setTranslationY(-1000f);
        if (activePlayer == 0) {
            counter.setImageResource(R.drawable.yellow);
            activePlayer = 1;
        } else {
            counter.setImageResource(R.drawable.red);
            activePlayer = 0;
        }
        counter.animate()
                .translationYBy(1000f)
                .rotation(360)
                .setDuration(500);
        //checking for the winner
        for (int[] winningPosition : winningPositions) {
            if (gameState[winningPosition[0]] ==      gameState[winningPosition[1]] &&
                    gameState[winningPosition[1]] == gameState[winningPosition[2]] &&
                    gameState[winningPosition[0]] !=2 ) {
                //Someone has won
                gameIsActive = false;
                String winner = "Red";
                if (gameState[winningPosition[0]] == 0){
                    winner = "Yellow";
                }
                TextView winnerMsg = (TextView)  findViewById(R.id.winnerMsg);
                winnerMsg.setText(winner + " has won!");
                LinearLayout layout = (LinearLayout)findViewById(R.id.playAgainLayout);
                layout.setVisibility(View.VISIBLE);
            } else {
                boolean gameIsOver = true;
                for (int counterState : gameState ) {
                    if (counterState == 2) {
                        gameIsOver = false;
                    }
                }
                if (gameIsOver) {
                    TextView winnerMsg = (TextView) findViewById(R.id.winnerMsg);
                    winnerMsg.setText("It is a DRAW!");
                    LinearLayout layout = (LinearLayout)findViewById(R.id.playAgainLayout);
                    layout.setVisibility(View.VISIBLE);

                }
            }
        }
    }
}
public void playAgain(View view) {
    gameIsActive = true;
    LinearLayout layout = (LinearLayout)findViewById(R.id.playAgainLayout);
    layout.setVisibility(View.INVISIBLE);
    activePlayer = 0;
    for (int i = 0; i < gameState.length; i++) {
        gameState[i] = 2;
    }
    GridLayout gridLayout = (GridLayout)findViewById(R.id.gridLayout);
    for (int i = 0; i < gridLayout.getChildCount(); i++) {
        ((ImageView) gridLayout.getChildAt(i)).setImageResource(0);
    }
}
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
}

public类MainActivity扩展了AppCompatActivity{
//0个黄色；1个红色；
int activePlayer=0；
布尔gameIsActive=true；
//2未播放；
int[]游戏状态={2,2,2,2,2,2,2,2,2}；
int[][]赢取位置={
{0,1,2}，{3,4,5}，{6,7,8}，//hor
{0,3,6}，{1,4,7}，{2,5,8}，//ver
{0,4,8}, {2,4,6}
}；//直径
公共void dropIn（视图）{
ImageView计数器=（ImageView）视图；
int-tappedCounter=Integer.parseInt（counter.getTag（）.toString（））；
如果（游戏状态[tappedCounter]==2&&gameIsActive）{
游戏状态[tappedCounter]=活动玩家；
计数器设置平移Y（-1000f）；
如果（activePlayer==0）{
计数器.setImageResource（R.drawable.yellow）；
activePlayer=1；
}否则{
计数器.setImageResource（R.drawable.red）；
activePlayer=0；
}
counter.animate（）
.平移比（1000f）
.轮换（360）
.设定持续时间（500）；
//检查赢家
对于（int[]winningPosition:winningPositions）{
如果（游戏状态[winningPosition[0]]==游戏状态[winningPosition[1]]&&
游戏状态[赢牌位置[1]]==游戏状态[赢牌位置[2]]&&
游戏状态[赢牌位置[0]！=2）{
//有人赢了
gameIsActive=false；
字符串winner=“红色”；
如果（游戏状态[winningPosition[0]]==0）{
winner=“黄色”；
}
TextView winnerMsg=（TextView）findViewById（R.id.winnerMsg）；
winnersg.setText（winner+“has wind！”）；
LinearLayout布局=（LinearLayout）findViewById（R.id.playAgainLayout）；
布局.设置可见性（视图.可见）；
}否则{
布尔gameIsOver=true；
for（int counterState:gameState）{
如果（反状态==2）{
gameIsOver=false；
}
}
如果（游戏版本）{
TextView winnerMsg=（TextView）findViewById（R.id.winnerMsg）；
setText（“这是一场平局！”）；
LinearLayout布局=（LinearLayout）findViewById（R.id.playAgainLayout）；
布局.设置可见性（视图.可见）；
}
}
}
}
}
再次播放公共void（视图）{
gameIsActive=true；
LinearLayout布局=（LinearLayout）findViewById（R.id.playAgainLayout）；
布局.设置可见性（视图.不可见）；
activePlayer=0；
对于（inti=0；i

和XML：

<GridLayout
    android:layout_width="match_parent"
    android:layout_height="360dp"
    android:layout_centerVertical="true"
    android:layout_centerHorizontal="true"
    android:columnCount="3"
    android:rowCount="3"
    android:background="@drawable/board"
    android:id="@+id/gridLayout">

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView"
        android:layout_row="0"
        android:layout_column="0"
        android:layout_margin="10dp"
        android:onClick="dropIn"
        android:tag="0"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView2"
        android:layout_row="0"
        android:layout_column="1"
        android:layout_marginTop="10dp"
        android:layout_marginLeft="15dp"
        android:onClick="dropIn"
        android:tag="1"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView3"
        android:layout_row="0"
        android:layout_column="2"
        android:layout_marginTop="10dp"
        android:layout_marginLeft="15dp"
        android:onClick="dropIn"
        android:tag="2"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView4"
        android:layout_row="1"
        android:layout_column="0"
        android:layout_marginLeft="10dp"
        android:layout_marginTop="25dp"
        android:onClick="dropIn"
        android:tag="3"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView5"
        android:layout_row="1"
        android:layout_column="1"
        android:layout_marginLeft="15dp"
        android:layout_marginTop="25dp"
        android:onClick="dropIn"
        android:tag="4"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView6"
        android:layout_row="1"
        android:layout_column="2"
        android:layout_marginLeft="15dp"
        android:layout_marginTop="25dp"
        android:onClick="dropIn"
        android:tag="5"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView7"
        android:layout_row="2"
        android:layout_column="0"
        android:layout_marginLeft="10dp"
        android:layout_marginTop="30dp"
        android:onClick="dropIn"
        android:tag="6"/>>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView8"
        android:layout_row="2"
        android:layout_column="1"
        android:layout_marginLeft="15dp"
        android:layout_marginTop="30dp"
        android:onClick="dropIn"
        android:tag="7"/>

    <ImageView
        android:layout_width="90dp"
        android:layout_height="90dp"
        android:id="@+id/imageView9"
        android:layout_row="2"
        android:layout_column="2"
        android:layout_marginLeft="15dp"
        android:layout_marginTop="30dp"
        android:onClick="dropIn"
        android:tag="8"/>
</GridLayout>

<LinearLayout
    android:orientation="vertical"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_centerVertical="true"
    android:layout_centerHorizontal="true"
    android:background="#deed17"
    android:padding="30dp"
    android:id="@+id/playAgainLayout"
    android:baselineAligned="false"
    android:visibility="invisible">

    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="New Text"
        android:id="@+id/winnerMsg"
        android:layout_gravity="center_horizontal"
        android:textSize="30sp" />

    <Button
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Play Again!"
        android:id="@+id/playBtn"
        android:layout_gravity="center_horizontal"
        android:onClick="playAgain" />
</LinearLayout>

>

---

在for循环中，您正在检查所有可能的赢条件，当它找到一个赢条件时，它将继续执行其余的赢条件。如果您只是在代码末尾加了一个中断

 for (int[] winningPosition : winningPositions) 
    {
        if (gameState[winningPosition[0]] == gameState[winningPosition[1]] &&
                gameState[winningPosition[1]] == gameState[winningPosition[2]] &&
                gameState[winningPosition[0]] !=2 ) 
        {
            //Someone has won
            gameIsActive = false;

            String winner = "Red";
            if (gameState[winningPosition[0]] == 0) 
            {
                winner = "Yellow";
            }
            TextView winnerMsg = (TextView) findViewById(R.id.winnerMsg);
            winnerMsg.setText(winner + " has won!");
            LinearLayout layout = (LinearLayout)findViewById(R.id.playAgainLayout);
            layout.setVisibility(View.VISIBLE);
            break;
        } 

---

这可能就够了。

在for循环中，您正在检查所有可能的赢条件，当它找到一个赢条件时，它将继续处理其余的赢条件。如果您只是在代码末尾加了一个中断符

 for (int[] winningPosition : winningPositions) 
    {
        if (gameState[winningPosition[0]] == gameState[winningPosition[1]] &&
                gameState[winningPosition[1]] == gameState[winningPosition[2]] &&
                gameState[winningPosition[0]] !=2 ) 
        {
            //Someone has won
            gameIsActive = false;

            String winner = "Red";
            if (gameState[winningPosition[0]] == 0) 
            {
                winner = "Yellow";
            }
            TextView winnerMsg = (TextView) findViewById(R.id.winnerMsg);
            winnerMsg.setText(winner + " has won!");
            LinearLayout layout = (LinearLayout)findViewById(R.id.playAgainLayout);
            layout.setVisibility(View.VISIBLE);
            break;
        } 

---

这也许可以做到。

您可以使用以下代码而不是上面的“如果条件”来明确概念：-

int player1=0，player2=0；
对于（int i=0；i

您可以使用以下代码而不是上面的“如果条件”来明确概念：-

int player1=0，player2=0；
对于（int i=0；i

请粘贴您的完整文件，因为它不清楚您在该文件的其余部分正在做什么。请粘贴您的完整文件，因为它不清楚您在该文件的其余部分正在做什么。很高兴听到它起作用。如果您喜欢这个答案，可以请您投票吗？谢谢。我需要更多的声誉来做到这一点，sry。很高兴听到它起作用。如果您喜欢an斯沃，你能帮我投票吗？谢谢。我需要更高的声誉来做到这一点，sry。