尝试比较对象不能使用 IF 语句 JAVA

Question

我正在编写一种在 ArrayList 中查找对象的方法。如果我能找到对象，我会将它打印到屏幕上，否则我会打印一条错误消息，指出“找不到对象”。我遇到的问题是因为我的方法是对象“十二面体”，而不是布尔值，所以我不能使用 if 语句来比较对象是否存在于数组中。我还能怎么办？

这是我的主要方法中的代码。

    System.out.print("\tLabel: ");

    label = userInput.nextLine();

    if(myDList.findDodecahedron(label)) {

        System.out.print(myDList.findDodecahedron(label));

    }

    else {

        System.out.print("\t\"" + label + "\" not found");

    }

        System.out.print("\n\nEnter Code [R, P, S, A, D, F, E, or Q]: ");

    break;

这是我的方法。

public Dodecahedron findDodecahedron(String label1In) {

      String label = "";

      String color = "";

      double edge = 0;

      Dodecahedron result = new Dodecahedron(label, color, edge);

      int index = -1;

      for (Dodecahedron d : dList) {

         if (d.getLabel().equalsIgnoreCase(label1In)) { 

            index = dList.indexOf(d);

            break;

         }    

      }

      if (index >= 0) {

         dList.get(index);

         result = dList.get(index);

      }

      return result;

   }

这是我尝试编译时遇到的错误。

DodecahedronListAppMenu.java:99: error: incompatible types: Dodecahedron cannot be converted to boolean

               if(myDList.findDodecahedron(label)) {

Answer 1

检查返回值是否为空。if (myDList.findDodecahedron(label) != null)findDodecahedron()如果没有找到任何东西，而不是新对象，则需要更新以返回 null。更改的初始值result将做到这一点：Dodecahedron result = null;另外，你可以摆脱index和result如果你只是立即返回形状，当你找到它。无需保存其索引，然后在循环结束后查找索引。public Dodecahedron findDodecahedron(String label1In) {   for (Dodecahedron d : dList) {      if (d.getLabel().equalsIgnoreCase(label1In)) {          return d;      }       }   return null;}您还可以使用 Java 8 流进一步简化它：public Dodecahedron findDodecahedron(String label1In) {   return dList.stream()      .filter(d -> d.getLabel().equalsIgnoreCase(label1In))      .findAny()      .orElse(null);}

Answer 2

您实际上是在此处实现空对象设计模式。您可以在findDodecahedron方法中明确说明这一点（旁注 - 为了优雅，我用 Java 8 样式的流替换了实现，但这并不是真正必需的）：public static final Dodecahedron NULL_DODECAHEDRON = new Dodecahedron("", "", 0);public Dodecahedron findDodecahedron(String label1In) {      return dList.stream()                  .filter(d -> d.getLabel().equalsIgnoreCase(label1In))                  .findFirst()                  .orElse(NULL_DODECAHEDRON);}然后在 if 条件下使用它：if (!myDList.findDodecahedron(label).equals(NULL_DODECAHEDRON)) {    System.out.print(myDList.findDodecahedron(label));} else {    System.out.print("\t\"" + label + "\" not found");}

Answer 3

如果你的方法不返回一个布尔值而是一个特定的对象，你应该将它分配给一个变量来利用它。您不会实例化特定对象并将其返回以像布尔值一样对其进行测试。正因为如此，您必须重复方法调用以第二次检索结果，因为您需要在 std 输出中打印它。它是无奈和重复的代码/处理。这应该是这样的：Dodecahedron obj = myDList.findDodecahedron(label)if(obj != null) {    System.out.print(obj);}else {    System.out.print("\t\"" + label + "\" not found");}