Sehe ich 100% genau so. Die Erweiterung mit "catch (Exception e)" sehe ich für total überflüssig an!
Im übrigen versuche ich catch(Excep...) im code zu vermeiden, da es den Code unleserlicher macht und auch eine Fehlerquelle darstellen kann, falls nämlich eine Exception an dieser stelle versickert an die nicht gedacht wurde. Ich bin der Meinung man sollte bewusst einzelne Exeptions fangen und nicht einfach alle (Ausnahmen gibt es natürlich!).
Evtl. könnte dein Ausbilder auch einen fehlenden finally block vermissen. Da du ja hier einen IO zugriff machst sollte nicht vergessen werden diesen IO stream auch wieder zu schließen.
so in der Art:
try {
...
} catch (...) {
...
} finally {
io.close();
}
viele grüße,
nab.