java.util.List不';找不到元素

java.util.List不';找不到元素,java,list,object,search,Java,List,Object,Search,我需要再次咨询您,因为我遇到了以下问题: 为了给我的所有项目提供日志功能,我使用了一个自制的日志库,其中包括用于从所有类和对象访问的静态方法,以及在退出时将日志内容转储到文件的功能。这已经运行了好几年了,现在我需要扩展这个库来提供一个函数来使用几个“日志通道”,每个通道代表单个日志。每个日志通道都是类“log”的实例(见下文),然后添加到列表中(如类ListTest3中所示,见下文)。但是,当试图通过创建具有相同通道名称的虚拟对象从列表中获取特定日志时,始终会返回-1,表示找不到任何对象。为什么

我需要再次咨询您,因为我遇到了以下问题:

为了给我的所有项目提供日志功能,我使用了一个自制的日志库,其中包括用于从所有类和对象访问的静态方法,以及在退出时将日志内容转储到文件的功能。这已经运行了好几年了,现在我需要扩展这个库来提供一个函数来使用几个“日志通道”,每个通道代表单个日志。每个日志通道都是类“log”的实例(见下文),然后添加到列表中(如类ListTest3中所示,见下文)。但是,当试图通过创建具有相同通道名称的虚拟对象从列表中获取特定日志时,始终会返回
-1
,表示找不到任何对象。为什么?

类日志:

public class Log {
  //...

  public String getChannel(){
    return channel;
  }

  //...

  public boolean equals(Log compare){
    // return getChannel().equals(compare.getChannel());
    return true;

    /**
     * used to contain a method to compare the channel names of the log
     * object itself and the provided object for comparison, now always
     * returns true for debugging purposes
     */
  }

  //...
}
为了进行调试,我创建了以下类ListTest3:

package test;

import java.util.List;
import java.util.LinkedList;

import logging.Log;

public class ListTest3 {
  public static void main(String[] args){
    List list = new LinkedList();
    Log logDefault = new Log();
    logDefault.setChannel("default");
    Log logAdvanced = new Log();
    logAdvanced.setChannel("advanced");
    Log logDebug = new Log();
    logDebug.setChannel("debug");

    list.add(logDefault);
    list.add(logAdvanced);
    list.add(logDebug);

    System.out.println("Index of logDefault: " + list.indexOf(logDefault));
    System.out.println("Index of logAdvanced: " + list.indexOf(logAdvanced));
    System.out.println("Index of logDebug: " + list.indexOf(logDebug));

    Log logDefaultDummy = new Log();
    logDefaultDummy.setChannel("default");
    System.out.println("Index of logDefaultDummy: " + list.indexOf(logDefaultDummy));

    Log logAdvancedDummy = new Log();
    logAdvancedDummy.setChannel("advanced");
    System.out.println("Index of logAdvancedDummy: " + list.indexOf(logAdvancedDummy));

    Log logDebugDummy = new Log();
    logDebugDummy.setChannel("debug");
    System.out.println("Index of logDebugDummy: " + list.indexOf(logDebugDummy));
  }
}
但是,在通过虚拟对象搜索列表时,不会返回三个日志对象的索引,而是始终返回
-1
,如下输出所示:

Index of logDefault: 0
Index of logAdvanced: 1
Index of logDebug: 2
Index of logDefaultDummy: -1
Index of logAdvancedDummy: -1
Index of logDebugDummy: -1

非常感谢您的帮助,因为我不知道如何解决这个问题。感谢您阅读这一批文本!;)

您在
Log
类中的equals方法具有错误的签名。应该是

@Override
public boolean equals(Object compare){
    if (!(compare instanceof Log))
        return false;
    Log olog = (Log) compare;
    return getChannel().equals(olog.getChannel());
}
以覆盖
对象的
等于
方法,并由
列表的
索引方法使用


如果不重写
equals
,您将获得检查引用相等性的默认实现,因此您的伪实例不等于列表中的任何实例。

您在
Log
类中的equals方法具有错误的签名。应该是

@Override
public boolean equals(Object compare){
    if (!(compare instanceof Log))
        return false;
    Log olog = (Log) compare;
    return getChannel().equals(olog.getChannel());
}
以覆盖
对象的
等于
方法,并由
列表的
索引方法使用


如果不重写
equals
,您将获得检查引用相等性的默认实现,因此您的虚拟实例不等于列表中的任何实例。

您没有正确重写对象中的
equals()
hashCode()
。请注意,方法的正确签名是

public boolean equals(Object o)
而不是

public boolean equals(Log o)
当您打算重写/实现某个方法时,应始终在该方法上使用
@override
注释。这样可以避免像你所做的那样的错误


而且,你正在重新发明轮子。为什么不使用一个真正的、经过战斗测试的、高效的日志框架,如logback或log4j?

您没有正确地覆盖对象中的
equals()
hashCode()
。请注意,方法的正确签名是

public boolean equals(Object o)
而不是

public boolean equals(Log o)
当您打算重写/实现某个方法时,应始终在该方法上使用
@override
注释。这样可以避免像你所做的那样的错误


而且,你正在重新发明轮子。为什么不使用一个真正的、经过战斗测试的、高效的日志框架,如logback或log4j?

可能重复的可能重复哦,谢谢,我完全没有意识到equals()需要一个对象,而不是任何类或相同类型的类。不知道我怎么会想到这一点:D非常感谢你的快速回复!哦,谢谢,我完全忽略了equals()需要对象,而不是任何类或相同类型的类。不知道我怎么会想到这一点:D非常感谢你的快速回复!非常感谢您的快速回复!我真的很感激stackoverflow的回答来得这么快:谢谢你的快速回复!我真的很感激stackoverflow:D的答案来得这么快