4

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

像这样：

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();
  }
}
4

2 回答 2

2

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

目前尚不清楚为什么这些流变量应该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作为单独的变量。更多在这个问题的答案中。

于 2019-03-25T13:17:52.103 回答
2

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

于 2019-03-25T13:17:56.563 回答