How should a closed resource behave in java?

You should throw an exception when trying to access the internal resource after it has been closed, an IllegalStateException would be fine. One thing worth noting is you can use your Closeable in a try-with-resources to further ensure that once the object falls out of scope it is closed.


What is the canonical Java way?

First, let's look at the javadoc of the close() method:

Closes this stream and releases any system resources associated with it. If the stream is already closed then invoking this method has no effect.

So the answer to your options is "none of the above" for the close() method. You don't throw an exception, and you don't let exceptions through from the closed resource. If it has been closed, the call must be a NOOP.

Now, let's have a look at some of the Closeable classes:

  • FileInputStream.read()

    Throws IOException if an I/O error occurs.

    That includes "file closed".

    That goes for all the InputStream/OutputStream/Reader/Writer I/O classes.

  • FileSystem.close()

    After a file system is closed then all subsequent access to the file system, either by methods defined by this class or on objects associated with this file system, throw ClosedFileSystemException. If the file system is already closed then invoking this method has no effect.

  • Formatter.close()

    Attempting to invoke any methods except ioException() in this formatter after it has been closed will result in a FormatterClosedException.

  • URLClassLoader.findClass(name)

    Throws ClassNotFoundException if the class could not be found, or if the loader is closed.

Conclusion: All the methods (except close()) throw exceptions, though not IllegalStateException.

You can of course use IllegalStateException if you want.


Just take a look at one of the probably most used interfaces with a close method, java.sql.Connection:

Here an exception is thrown:

SQLException if a database access error occurs or this method is called on a closed connection

This is basically what makes sense: If calling a close method, the developer willingly closes whatever the object is (Connection, Socket, Stream, ...) and therefore leaves it, as you already stated, in a defunct state. If the developer now tries to call some function on the closed object, he should get an error, thus an exception is the only proper thing to do here.