时间戳

首页 > 解决方案 > SequentialTransition 重复超过 cylcecount

java - SequentialTransition 重复超过 cylcecount

问题描述

我会写出故障，但我知道我即将被带到学校学习我不知道存在的 Javafx 功能。

当我使用PauseTransition和的组合SequentialTransition来制作一个由按钮网格组成的连接四个游戏时，一个按钮一个按钮地向下浮动。在一个非常具体的用例下，我有一个非常意想不到的结果。

以下是运行和复制错误所需的所有代码，这些代码被编辑为独立且极简，同时仍保持基本游戏逻辑。 

import javafx.animation.PauseTransition;
import javafx.animation.SequentialTransition;
import javafx.application.Application;
import javafx.event.Event;
import javafx.event.EventHandler;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.layout.BorderPane;
import javafx.scene.layout.GridPane;
import javafx.stage.Stage;
import javafx.util.Duration;

public class ConnectFourApplication extends Application {
    public static final int NUM_COLUMNS = 8;
    public static final int NUM_ROWS = 8;
    private Button[][] buttons;
    private int row =0;
    private int column = 0;
    private PauseTransition buttonGravityPause;
    private Scene scene;
    private BorderPane border;
    private ConnectEnum turnColor=ConnectEnum.RED;

    public enum ConnectEnum{
        RED ("Red"), BLACK ("Black");
        private String turn;
        ConnectEnum( String value) {
            this.turn = value;
        }
    }

    @Override
    public void start(Stage primaryStage) {
        buttons = new Button[NUM_ROWS][NUM_COLUMNS];
        GridPane gridpane = new GridPane();
        Button turn = new Button("Take turn");
        for(int i=0; i<NUM_ROWS; i++){
            for(int j = 0; j<NUM_COLUMNS; j++){
                buttons[i][j] = new Button("Empty");
                buttons[i][j].setMinHeight(20);
                buttons[i][j].setMaxWidth(Double.MAX_VALUE);
                buttons[i][j].setOnAction(new ButtonHandler<>(i,j));
                gridpane.add(buttons[i][j], j, (NUM_ROWS-1)-i);
            }
        }

        buttonGravityPause = new PauseTransition(new Duration(700));
        buttonGravityPause.setOnFinished(event-> {
            if (this.row>0) {
                buttons[this.row - 1][this.column].setStyle("-fx-background-color:blue");
                buttons[this.row][this.column].setStyle("");
                this.row = this.row - 1;
            }else System.out.println("THis should never happen, row is: "+this.row);
        });

        SequentialTransition s = new SequentialTransition(buttonGravityPause);
        s.setOnFinished(floatDown-> {
            System.out.println("The button " +row+ " " +column+" has been chosen");
            buttons[this.row][this.column].setText(""+getTurn());
            buttons[this.row][this.column].setStyle("-fx-background-color:"+getTurn());
            changeTurn();
        });

        turn.setOnAction(e->{
            System.out.println("The button " +row+ " " +column+" has been chosen");
            int i =0;
            while(this.row-i>0 && buttons[this.row-(i+1)][this.column].getText().equals("Empty"))i++;
            s.setCycleCount(i);
            System.out.println("i is: "+i+"  but cycle count is: "+s.getCycleCount());
            if(i>0)s.play(); else{
                System.out.println("The button " +row+ " " +column+" has been chosen");
                buttons[this.row][this.column].setText(""+getTurn());
                buttons[this.row][this.column].setStyle("-fx-background-color:"+getTurn());
                changeTurn();
            }
        });

        border = new BorderPane();
        border.setCenter(gridpane);
        border.setBottom(turn);
        scene = new Scene(border, 510, 380);

        primaryStage.setTitle("Connect Four");
        primaryStage.setScene(scene);
        primaryStage.show();
    }
    private ConnectEnum getTurn(){return this.turnColor;}
    private void changeTurn(){this.turnColor = this.turnColor==ConnectEnum.RED? ConnectEnum.BLACK:ConnectEnum.RED;}
    private void setRowColumn(int row, int column){
        this.row=row;
        this.column=column;
    }
    private int getRow(){
        return this.row;
    }
    private int getColumn(){
        return this.column;
    }
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        launch(args);
    }

    class ButtonHandler<T extends Event> implements EventHandler<T> {
        private int row;
        private int col;

        ButtonHandler(int row, int column){
            this.row=row;
            this.col=column;
        }

        @Override
        public void handle(T event){
            if (buttons[getRow()][getColumn()].getText().equals("Empty")) buttons[getRow()][getColumn()].setStyle("");
            if (buttons[this.row][this.col].getText().equals("Empty")) {
                setRowColumn(this.row, this.col);
                buttons[row][col].setStyle("-fx-background-color:blue");
            }else System.out.println("This space is occupied");
        }
    }
}

这个功能在所有情况下都能正常工作，除了我 1. 选择一个紧挨着已经放置的按钮上方的按钮，然后 2. 下一个选择，如果它被放置在几个空白点的上方，将导致以下控制台输出（包括前几个表现良好和预期的陈述）：

已选择按钮 2 1
我是：2，但循环计数是：2
已选择按钮 0 1
错误的
已选择按钮 1 1
i 是：0 但循环计数是：0
错误的
已选择按钮 4 2
我是：4 但循环计数是：4
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
这不应该发生，行是：0
...
...
已选择按钮 0 2

要重现错误/竞争条件： 1. 选择任何按钮，按轮流。2. 立即选择现在彩色按钮上方的按钮，按轮流。3. 选择一个不在底部按钮行中的按钮，然后观看奇迹发生。

以前这会导致数组越界，但我添加了一个 if 语句，这样我就可以避免错误并尝试阐明正在发生的事情。

如果有人对 JavaFX 的这一领域有任何经验，我将非常感激，因为我不知所措。

标签： javajavafx-8race-condition

解决方案

单击SequentialTransition s 时创建一个字段并构造一个新字段（而不是重复使用相同的实例）turn，可以解决问题：

    turn.setOnAction(e->{
        int i = 0;
        while(row-i>0 && buttons[row-(i+1)][column].getText().equals("Empty")) {
            i++;
        }

        if(i>0) {
            s = new SequentialTransition(buttonGravityPause);
            s.setAutoReverse(false);
            s.setOnFinished(floatDown-> {
                buttons[row][column].setText(""+getTurn());
                buttons[row][column].setStyle("-fx-background-color:"+getTurn());
                changeTurn();
            });
            s.setCycleCount(i);
            s.play();
        } else{
            buttons[row][column].setText(""+getTurn());
            buttons[row][column].setStyle("-fx-background-color:"+getTurn());
            changeTurn();
        }
    });

我不确定为什么重用SequentialTransition实例会导致问题。
希望了解更多的人可以添加解释。
可以从此处复制工作演示代码

推荐阅读