java - 从具有日期比较的列表中删除

Question

我必须删除 param.getFromDateTime 之前和 params.getToDateTime 之后的所有对象。我是这样做的:

for (ListIterator<Doc> iterator = doc.listIterator(); iterator.hasNext(); ) {
    while(iterator.hasNext()){
            if(param.getFromDateTime() != null
              && doc.get(iterator.nextIndex()).getDate().before(params.getFromTime())) {
                iterator.remove();
            }
            if(params.getToDateTime() != null
              && doc.get(iterator.nextIndex()).getDate().after(params.getToDateTime())) {
                iterator.remove();
           }
    }
}

我可以做得更好更清楚吗？

Answer 1

你不需要两个循环，你也不需要 ListIterator。
请注意，这两个条件可能为真，但在任何情况下您都只想删除一个元素，因为调用两次 remove() 会引发异常。
您还可以直接从迭代的 Doc 对象中提取 date 以使其更清晰一些:

for (Iterator<Doc> iterator = doc.iterator(); iterator.hasNext(); ) {
      Date date = iterator.next().getDate();

      if( (param.getFromDateTime() != null && date.before(params.getFromTime())) 
          || (params.getToDateTime() != null && date.after(params.getToDateTime()))) {
        iterator.remove();
      }

}