用Java读取和处理文件的迭代器

Question

假设我有一个类< code>Point和一个函数来处理< code>Point实例

class Point { private final int x, y; ... } 
...
void handlePoints(Iterable<Point> points) { for (Point p: points) {...} }

现在我想从一个文件中读取＜code＞点

Point makePoint(String line) { ... }

我现在该怎么办？我可以编写一个函数将文件读取到< code>points列表中，并调用< code>handlePoints函数。

List<Point> readPoints(BufferedReader reader) {...} // use makePoint here

void handlePoints(BufferedReader reader) {
   List<Point> points = readPoints(reader); 
   handlePoints(points);
}

不幸的是，这个函数看起来并不特别优雅，因为它在内存中创建了一个不必要的点列表。

使用迭代器不是更好吗？

void handlePoints(Iterator<Point> points) {...}

Iterator<Point> readPoints(BufferedReader reader) {...} // use makePoint here

void handlePoints(BufferedReader reader) {
   Iterator<Point> points = readPoints(reader); 
   handlePoints(points);
}

有意义吗？在Java，这段代码不会太“吵”吗？

Answer 1

只需从文件中读取一次并将其保存在内存中，而不是每次都从文件中读取

List<Points> points ;

public List<Point> readPoints(BufferedReader reader) {
     if(points == null) {
        points = new ArrayList();
        // read from file and populate 
        points.add(point) ;
     }
 return points;
} 

Answer 2

从内存的角度来看，使用迭代器不会真正节省内存-我猜您将把所有的点都读入内存，所以它们都必须以某种方式存储。

迭代器不是不同的集合类型：它只是迭代集合的不同方式。例如，您可以通过list.iterator（）让迭代器遍历您的列表（或任何集合）。

选择使用哪个集合来保存内存中的所有点将影响内存（例如。数组列表与链接列表）。

Answer 3

如果您不需要将所有点都存储在内存中，请考虑以下内容：

while (reader.ready())
{
  String line = reader.readLine();
  Point point = makePoint(line);
  handlePoint(point);
}

如何使用迭代器和句柄点执行此操作：（用于处理要添加的异常的代码）

class PointIterator implements Iterator<Point>
{
  BufferedReader reader;
  PointIterator(BufferedReader myReader) { reader = myReader; };
  @Override
  public boolean hasNext() { return myReader.ready(); };
  @Override
  public Point next() { return makePoint(myReader.readLine()); };
  @Override
  public void remove()
  { throw new UnsupportedOperationException("Remove not supported!"); };
}

因为< code>handlePoints采用< code>Iterable:

class PointIterable implements Iterable<Point>
{
  BufferedReader reader;
  public PointIterable(BufferedReader myReader) { reader = myReader; };
  @Override
  public Iterator<Point> iterator() { return new PointIterator(reader); }
}

使用方法：

handlePoints(new PointIterable(reader));