Da sehe ich mehrere Punkte:
a) Generell bitte Code nicht als Bildschirmfoto sondern per Copy & Paste in code Tags posten.
b) Bitte versuchen, genaue Fragen zu stellen. Die Frage "Wäre das so richtig?" würde ich prinzipiell mit einer Gegenfrage beantworten: Hast Du es ausprobiert? In b) werden ja Unit Tests erwähnt: Laufen die durch?
Dann zu dem bisschen Code fällt mir direkt auf:
c) Dein Code dürfte sich nicht übersetzen lassen. new FileReader kann IOException werfen und die fängst Du nicht und Deine Methode hat auch kein throws IOException.
d) Du nutzt Klassen die AutoClosable implementieren für lokale Variablen -> Da sollte (in der Regel) immer try with resources verwendet werden!
e) Namen! Sinnvolle Namen vergeben. Die Methoden sollten so benannt sein, dass klar ist, was sie machen. Dein Methode sagt "getFileName" aber sie gibt nicht nur einen FileName, die macht ganz viele andere Dinge ... Das sieht nicht bezüglich Clean Code nicht korrekt aus.
f) Auf Grund des kurzen Ausschnitts schwer zu sagen, aber vermutlich hast Du wie in Scripts üblich, die Variablen alle am Methodenanfang deklariert. Deklaration und Nutzung sollten dicht beisammen sein und auch von der Lebenszeit auf ihren Block begrenzt sein so möglich.
Das wäre so mein Feedback, das mir da einfallen würde ...