JavaFX组合框< POJO >不更新值

Question

几个小时寻找答案后，我终于放弃了。我有一个包含以下组合框的FXML表单:

<ComboBox fx:id="cboTipo" disable="true" prefWidth="247.0" promptText="Tipo de obra..." GridPane.columnIndex="1" GridPane.rowIndex="2" />

它注入到 JavaFX 控制器类中：

    @FXML
    public ComboBox<Tobra> cboTipo;

该组合显示了使用Eclipselink 2.7和JPA 2.2从嵌入式H2数据库加载的Tobra(在西班牙语中代表Tipo de Obra)列表。

我不会向用户展示Tobra的价值。toString，相反，我在初始化中设置了一个转换器：

@Override
public void initialize(URL url, ResourceBundle rb) {
    ...
    Objects.requireNonNull(cboTipo, "cboTipo not injected");
    ...
    cboTipo.setConverter(new StringConverter<Tobra>() {
        @Override
        public String toString(Tobra object) {
            return object.getCod() + ": " + object.getNombre();
        }

        @Override
        public Tobra fromString(String string) {
            return new Tobra(string);
        }
    });
   ...
}

我有一个实现＜code＞Task的内部类

task.setOnSucceeded(evt ->
    cboTipo.setItems(FXCollections.observableArrayList(task.getValue()))
);

当然，当显示窗体时，我在线程中运行任务：

 new Thread(task).start();

在代码被测试之前，一切似乎都很好。不管我点击什么值，总是选择的值重置为第一个项目。我试图从代码中强制一些值，它显示在组合框中，但是当用户单击组合框选择另一个值时，选择的值再次被重置为“第一项”。只有在使用带有类型参数的ComboBox时，才会出现这种行为。当我创建没有类型参数的combobox，然后我添加字符串值，就像这样:

 cboTipo.getItems().clear();
 cboTipo.getItems().addAll(
  tobraList.stream().map(x 
       -> x.getCod() + ": " + x.getNombre())
  .toArray());

一切正常。

因此，我尝试在不映射到字符串的情况下对POJO Tobra执行同样的操作：

 cboTipo.getItems().clear();
 cboTipo.getItems().addAll(tobraList);

但是问题又出现了。我也尝试过在没有类型参数的情况下声明< code>ComboBox cboTipo,但是都不起作用。我的POJO Tobra以这种方式覆盖equals方法:

@Override
public boolean equals(Object object) {
    if (!(object instanceof Tobra)) {
        return false;
    }
      var other = (Tobra) object;
    return ((this.cod == null && other.cod != null)
            || (this.cod != null && !this.cod.equals(other.cod)));
}

我做错了什么？

PS：我还尝试按照以下建议设置自己的单元格工厂：带有自定义对象的Javafx组合框显示对象地址，尽管使用自定义单元格工厂

尝试并调试它，我意识到问题不在于组件的呈现，因为 ComboBox 的值属性在选择后永远不会更新。

Answer 1

我这样解决了改变equals方法的问题:

@Override
    public boolean equals(Object object) {
        if (object == null || !getClass().isAssignableFrom(object.getClass())) {
            return false;
        } else {
              var other = (Tobra) object;
            return Objects.equals(this.cod, other.cod);
        }
    }

教训：永远不要相信自动生成的代码。

Answer 2

您的equals（）方法很奇怪。

return ((this.cod == null && other.cod != null)
        || (this.cod != null && !this.cod.equals(other.cod)));

我们分手吧。第一部分:

(this.cod == null && other.cod != null)

如果此Tobra的cod为null并且另一个Tobra的cod不是null，则此部分为true。

现在，当发生这种情况时，整个表达式将返回 true，因为您有一个 || 运算符。

我们来看第二部分。

(this.cod != null && !this.cod.equals(other.cod))

如果此Tobra的cod不为 ，并且其他 的 cod不相等，则此部分为true

这看起来又是相反的。

很可能你需要！整个表达式。或者，您可以使用 Objects.equals（this.cod， other.cod），它会为您检查 null。

最后，确保您还重写了hashCode（），并返回一个正确的值，该值不会破坏equals（）的约定。阅读Javadoc了解更多详细信息。