Java 清除Tic-Tac-Toe/Noughts中的动画&；安卓中的十字架

我试图通过按下我分配给这个游戏的按钮来清除一个动画（这是一个“线”中三个闪烁按钮的集合），该按钮通过下面显示的“clearGame”方法清除棋盘。然而，当玩游戏赢了它（还没有设置按钮，稍后会做）时，它会正确地显示三线闪烁（比如用按钮1、4、7），当我调用“clearGame”方法时，它会重置棋盘。然而，出于某种原因，如果我用另一组按钮（例如我用按钮3、6、9赢了）第二次赢了游戏，它会显示两组按钮（例如3、6、9和1、4、7）都在闪烁，而不仅仅是一组。以下是主要游戏的活动：

package com.example.noughtsandcrosses;

import android.os.Bundle;
import android.app.Activity;
import android.content.Intent;
import android.view.Menu;
import android.view.View;
import android.view.View.OnClickListener;
import android.view.animation.Animation;
import android.view.animation.AnimationUtils;
import android.widget.Button;
import android.widget.TextView;

public class NoughtsAndCrossesXActivity extends Activity implements OnClickListener  {
    Button[] buttons = new Button[10];
    int[] squares = new int[10];
    Button newGame;
    Button exitGame;
    TextView WinGame;
    Animation WinningAnim;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.noughts_and_croses);    

        buttons[1] = (Button) findViewById(R.id.one);
        buttons[2] = (Button) findViewById(R.id.two);
        buttons[3] = (Button) findViewById(R.id.three);
        buttons[4] = (Button) findViewById(R.id.four);
        buttons[5] = (Button) findViewById(R.id.five);
        buttons[6] = (Button) findViewById(R.id.six);
        buttons[7] = (Button) findViewById(R.id.seven);
        buttons[8] = (Button) findViewById(R.id.eight);
        buttons[9] = (Button) findViewById(R.id.nine);  
        newGame = (Button) findViewById(R.id.new_game);
        newGame.setOnClickListener(this);
        WinGame = (TextView) findViewById(R.id.win_game);
        exitGame = (Button) findViewById(R.id.exit_game);
        exitGame.setOnClickListener(this);

        for (int i = 1; i <= 9; i++){
            buttons[i].setOnClickListener(this);    

            clearGame();
        }

        WinningAnim = AnimationUtils.loadAnimation(getApplicationContext(), R.anim.winning_animation);
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.login_page, menu);
        return true;
    }

    public void clearGame(){
        for (int i = 1; i <= 9; i++){
            buttons[i].setText("");
            buttons[i].setEnabled(true);
            squares[i] = 0;
        }
        WinGame.setText("");
    }

    @Override
    public void onClick(View view) {
        switch (view.getId()) {
        case R.id.one:
            makeMove(1);
            respond();
            break;
        case R.id.two:
            makeMove(2);
            respond();
            break;
        case R.id.three:
            makeMove(3);
            respond();
            break;
        case R.id.four:
            makeMove(4);
            respond();
            break;
        case R.id.five:
            makeMove(5);
            respond();
            break;
        case R.id.six:
            makeMove(6);
            respond();
            break;
        case R.id.seven:
            makeMove(7);
            respond();
            break;
        case R.id.eight:
            makeMove(8);
            respond();
            break;
        case R.id.nine:
            makeMove(9);
            respond();
            break;
        case R.id.new_game:
            System.out.println("newgame");
            clearGame();
            break;
        case R.id.exit_game:
            System.out.println("exitgame");
            //exitGameToMenu(); 
            Intent a = new Intent (this, MenuActivity.class);
            startActivity(a);
            break;
        }
    }

    public void CheckIfPlayerWon (int i) {
        i = 1;
        if (squares[1] == i && squares[2] == i && squares[3] == i){
            WinGame.setText("You have won the game! 1");
            buttons[1].startAnimation(WinningAnim);
            buttons[2].startAnimation(WinningAnim);
            buttons[3].startAnimation(WinningAnim);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[4] == i && squares[5] == i && squares[6] == i){
            WinGame.setText("You have won the game! 2");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].startAnimation(WinningAnim);
            buttons[5].startAnimation(WinningAnim);
            buttons[6].startAnimation(WinningAnim);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[7] == i && squares[8]== i && squares[9] == i){
            WinGame.setText("You have won the game! 3");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].startAnimation(WinningAnim);
            buttons[8].startAnimation(WinningAnim);
            buttons[9].startAnimation(WinningAnim);
        }
        else if (squares[1]== i && squares[4] == i && squares[7] == i){
            WinGame.setText("You have won the game! 4");
            buttons[1].startAnimation(WinningAnim);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].startAnimation(WinningAnim);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].startAnimation(WinningAnim);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[2] == i && squares[5] == i && squares[8] == i){
            WinGame.setText("You have won the game! 5");
            buttons[1].setEnabled(false);
            buttons[2].startAnimation(WinningAnim);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].startAnimation(WinningAnim);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].startAnimation(WinningAnim);
            buttons[9].setEnabled(false);
        }
        else if (squares[3] == i && squares[6] == i && squares[9] == i){
            WinGame.setText("You have won the game! 6");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].startAnimation(WinningAnim);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].startAnimation(WinningAnim);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].startAnimation(WinningAnim);
        }
        else if (squares[1] == i && squares[5] == i && squares[9] == i){
            WinGame.setText("You have won the game! 7");
            buttons[1].startAnimation(WinningAnim);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].startAnimation(WinningAnim);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].startAnimation(WinningAnim);
        }
        else if (squares[3] == i && squares[5] == i && squares[7] == i){
            WinGame.setText("You have won the game! 8");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].startAnimation(WinningAnim);
            buttons[4].setEnabled(false);
            buttons[5].startAnimation(WinningAnim);
            buttons[6].setEnabled(false);
            buttons[7].startAnimation(WinningAnim);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
    }

    public void CheckIfPlayerLost (int i) {
        i = 2;
        if (squares[1] == i && squares[2] == i && squares[3] == i){
            WinGame.setText("You have lost the game! 1");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[4] == i && squares[5] == i && squares[6] == i){
            WinGame.setText("You have lost the game! 2");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[7] == i && squares[8]== i && squares[9] == i){
            WinGame.setText("You have lost the game! 3");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[1]== i && squares[4] == i && squares[7] == i){
            WinGame.setText("You have lost the game! 4");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[2] == i && squares[5] == i && squares[8] == i){
            WinGame.setText("You have lost the game! 5");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[3] == i && squares[6] == i && squares[9] == i){
            WinGame.setText("You have lost the game! 6");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[1] == i && squares[5] == i && squares[9] == i){
            WinGame.setText("You have lost the game! 7");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
        else if (squares[3] == i && squares[5] == i && squares[7] == i){
            WinGame.setText("You have lost the game! 8");
            buttons[1].setEnabled(false);
            buttons[2].setEnabled(false);
            buttons[3].setEnabled(false);
            buttons[4].setEnabled(false);
            buttons[5].setEnabled(false);
            buttons[6].setEnabled(false);
            buttons[7].setEnabled(false);
            buttons[8].setEnabled(false);
            buttons[9].setEnabled(false);
        }
    }

    public void makeMove(int i) {
        buttons[i].setText("X");
        buttons[i].setEnabled(false);
        squares[i] = 1;
        CheckIfPlayerWon(i);
    }


    public void respond() {
        for (int i = 1; i <= 9; i++) {
            if (buttons[i].isEnabled()) {
                buttons[i].setText("O");
                buttons[i].setEnabled(false);
                squares[i] = 2;
                CheckIfPlayerLost(i);
                break;
            }
        }
    }
}

package com.example.noughtsandcross；
导入android.os.Bundle；
导入android.app.Activity；
导入android.content.Intent；
导入android.view.Menu；
导入android.view.view；
导入android.view.view.OnClickListener；
导入android.view.animation.animation；
导入android.view.animation.AnimationUtils；
导入android.widget.Button；
导入android.widget.TextView；
公共类NoughtsAndCrossXactivity扩展了活动实现OnClickListener{
按钮[]按钮=新按钮[10]；
整数[]平方=新整数[10]；
按钮新游戏；
按钮出口；
TextView WinGame；
动画WinningAnim；
@凌驾
创建时受保护的void（Bundle savedInstanceState）{
super.onCreate（savedInstanceState）；
setContentView（R.layout.noughts和croses）；
按钮[1]=（按钮）findViewById（R.id.one）；
按钮[2]=（按钮）findViewById（R.id.two）；
按钮[3]=（按钮）findViewById（R.id.three）；
按钮[4]=（按钮）findViewById（R.id.four）；
按钮[5]=（按钮）findViewById（R.id.five）；
按钮[6]=（按钮）findViewById（R.id.six）；
按钮[7]=（按钮）findViewById（R.id.seven）；
按钮[8]=（按钮）findViewById（R.id.8）；
按钮[9]=（按钮）findViewById（R.id.nine）；
newGame=（按钮）findViewById（R.id.new_game）；
newGame.setOnClickListener（这个）；
WinGame=（TextView）findViewById（R.id.win_游戏）；
exitGame=（按钮）findviewbyd（R.id.exit_游戏）；
ExitName.setOnClickListener（此）；
对于（int i=1；i我对Android不是很满意，但从我对文档的深入了解（当您遇到问题时，应该始终是您的第一站）可以看出，您在调用clearAnimation（）时使用了错误的对象。据我所知，类Animation（这就是WinningAnim）甚至没有clearAnimation（）方法。我想某种错误应该会让您知道这一点。另外，从loadAnimation（context，id）加载的动画是静态动画，因此您只想在静态上下文中处理它

您可能希望尝试在按钮上调用clearAnimation（），因为这些是视图的子类，它有一个名为clearAnimation（）的方法

此外，您应该真正重写大部分代码，您应该将代码整合到方法中和/或简化它

例如：


如果你有赢家，你就有输家，不需要同时检查输赢
用更优雅的东西（可能是一个循环？）替换那可怕的长链if语句

根据经验，如果看起来您多次执行完全相同的操作，但变化很小，通常可以用更优雅的代码来替换

祝你好运
谢谢，不过我需要在周一之前完成这项工作，并重新编写，以简化找出赢家和输家的方法。尽管对于清晰动画方法，我应该如何着手实施它？你可以在循环时将其插入到清晰游戏方法中，只需将此输入：'按钮[I].clearAnimation（）”。请尝试此操作。