我需要再次咨询你,因为我遇到了以下问题:
为了向我的所有项目提供日志记录功能,我使用自制库进行日志记录,其中包括从所有类和对象访问的静态方法,以及在退出时将日志内容转储到文件的功能。多年来这种方法运行良好,现在我需要扩展库以提供一个函数来使用几个“日志通道”,每个“日志通道”代表各个日志。每个日志通道都是“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
非常感谢任何帮助,因为我不知道如何解决这个问题。感谢您阅读那批文字! ;)
答案 0 :(得分:3)
您尚未在Object中正确覆盖equals()
和hashCode()
。请注意,方法的正确签名是
public boolean equals(Object o)
而不是
public boolean equals(Log o)
如果您打算覆盖/实现方法,则应始终在方法上使用@Override
注释。这样可以避免像你那样的错误。
另外,你正在重新发明轮子。为什么不使用真实的,经过实战考验的高效日志框架,如logback或log4j?
答案 1 :(得分:1)
您在Log
类中的equals方法具有错误的签名。它应该是
@Override
public boolean equals(Object compare){
if (!(compare instanceof Log))
return false;
Log olog = (Log) compare;
return getChannel().equals(olog.getChannel());
}
以覆盖Object
的{{1}}方法,并由equals
的{{1}}方法使用。
如果不覆盖List
,您将获得检查引用相等性的默认实现,因此您的虚拟实例不等于List中的任何实例。