Все работает, Валидатор ругается)

Звернул, все ошибки остались на месте(

Если не переопределять fos на new FileInputStream, а сделать 2 потока: fosSecond = new FileOutputStream(secondFile); fosThird = new FileOutputStream(thirdFile); и в конце fosSecond.close; fosThird.close; то решается вопрос с валидацией по последнему пункту, хотя по логике, все закрыто.

Почему бы и нет, они здесь как раз общие, да и static в любом try и for видно

System.in закрываю, так как необходимо после использования закрывать все потоки

Статические переменные нужны там, где должны быть объекты, общие для всех экземпляров класса. Здесь это явно ни к чему - может, мне понадобится создать несколько экземпляров Solution, каждый - для работы со своим файлом, а у тебя получится один поток на всех. Объявлять переменные желательно как можно ближе к месту их использования. У тебя они используются в main - вот там их и объявляй. Потоки FileInputStream и FileOutputStream должны быть закрыты. Про системный ничего не сказано. И кстати, поток записи в 3-й файл ты закрываешь. А во второй - нет.

С одной стороны, по поводу статических согласен, хотя в данном случае у меня все работало так как нужно), я сейчас перенес объявление всех переменных в main за исключением static FileInputStream fis; static FileOutputStream fosSecond; static FileOutputStream fosThird; так как без этого в блоке try { fileNameReader.close(); fis.close(); fosSecond.close(); fosThird.close(); } catch писали, что не видят инициализацию fosSecond и fosThird, так как она происходит в другом try catch. Валидацию полностью прошло, но работать перестало, и идея мне предложила поменять настройки файловой системы))