每次使用后都应关闭流吗？

Question

我目前正在为游戏开发UDP服务器。在这个服务器中，每个tick都使用ByteArrayInputStream和ObjectInputStream将序列化字节转换为对象。为流创建一个变量并在程序关闭时关闭它们是否更有效？

像这样：

class Main {
private static ByteArrayInputStream byteIn;
private static ObjectInputStream objectIn;

public static void main(String[] args) {
    while(true){
      receive();
    }
    //when program is done call close();
}

 public static void receive(){
     byteIn = new ByteArrayInputStream();
     objectIn = new ObjectInputStream(new BufferedInputStream(byteIn));
      //do something
  }

 public static void close(){
    objectIn.close();
    byteIn.close();
  }
}

或者每次创建和关闭新流更高效？

像这样：

class Main {

public static void main(String[] args) {
    while(true){
      receive();
    }
}

 public static void receive(){
    ByteArrayInputStream byteIn = new ByteArrayInputStream();
    ObjectInputStream objectIn = new ObjectInputStream(new BufferedInputStream(byteIn));
    //do something
    objectIn.close();
    byteIn.close();
  }
}

Answer 1

如果您打开一个流，则应该关闭它。问题中的代码不会这样做，它只是放弃以前的流并仅关闭它创建的最后一个流。

目前尚不清楚为什么这些流变量应该是static而不是receive中的本地变量。如果它们在receive中是本地的，您可以使用try-with-resources自动清理它们：

public static void receive(){
    try (
        ByteArrayInputStream byteIn = new ByteArrayInputStream();
        ObjectInputStream objectIn = new ObjectInputStream(new BufferedInputStream(byteIn));
    ) {
        //do something
    }
}

当控制传出try时，它们将自动关闭。

如果由于某种原因必须成为static类成员，只需关闭并释放您创建的每个成员（但是在代码中包含错误会更容易，这意味着它有执行路径无法执行此操作）。

// All done
objectIn.close();
objectIn = null;
byteIn.close();
byteIn = null;

---

旁注：使用这​​三种流类型，您可能不需要将byteIn作为单独的变量。更多在this question's answers。

Answer 2

是的，你需要关闭流。使用try-with-resources块。它将为您关闭流。