Use contextmanagers to ensure files are closed on errors

[chatcannon]

          This should really be done with a contextmanager but the `if ...: close()` is better than nothing.

Originally posted by @chatcannon in #102 (comment)