С одной стороны - идиотское задание
С другой стороны - желание сделать хорошую программу для пользователя
в целом получилось - смесь ежа и колючей проволоки
IMHO
Разбить на процедуры (в обратном порядке)
1. Ф-ция генерирующая одну операцию, возврашает String
1.
DateTimeFormatter formatDate = DateTimeFormatter.ofPattern("dd-MM-uuuu hh:mm:ss:ms");
форматтер требуется между запусками, т.ч. код лучше вынести в отдельный класс и глобальные (в рамках класса) переменные/константы, типа форматера, создавать один раз в конструкторе/методе инициализации
2. Процедура генерирующая один файл
3. Процедура генерирующая набор файлов по списку ArrayList<String>
4. соответственно - процедура преобразующая конец командой строки args в список ArrayList<String>
ну и дальше, по необходимости
==============
По коду:
1.
if (System.getProperty("os.name").toLowerCase().contains("windows")){
Как уже сказали:
https://docs.oracle.com/javase/7/docs/api/java/io/File.html
File classstatic String pathSeparator
The system-dependent path-separator character, represented as a string for convenience.
static char pathSeparatorChar
The system-dependent path-separator character.
static String separator
The system-dependent default name-separator character, represented as a string for convenience.
static char separatorChar
The system-dependent default name-separator character.
NOTE: Только сейчас заметил, что используется класс File
s , вместо нормального File. Похоже, поэтому, и NIO ошибка вылезает. Меньше знаний - меньше печали (что такое Files и как с ним работать - вот лично я не знаю)
1.
2.
3.
4.
5.
// проверка 1го параметра, существует ли файл, который указан в 1ом аргументе
// должен существовать в каталоге запуска программы
if (Files.notExists((Paths.get(FoolPathOfficesTxt)))) {
ErrCount = 1;
}
Вас это просили?
В задание про необходимость проверки ничего нет - в топку
Нет кода - нет ошибок
Ошибка и так будет, когда попытаемся открыть файл, которого нет.
Нечего переусложнять тестовая (б#$%) задание проверками и ф-ционалом о котором не просили. IMHO
1.
2.
3.
if (IsDigit <= 0) {
ErrCount = 12;
} else {
Ну отрицательное - ну и пофиг. Тогда ничего и не создастся.
Правила проверки параметров в задании были? не было? - в топку
1.
2.
3.
4.
5.
6.
if (Files.notExists((Paths.get(FoolOperPath)))) {// если не существует папка OPER пытаемся ее создать
try { // создаем папку для файлов операций OPER
Files.createDirectory(Path.of(FoolOperPath));
} catch (IOException e) { // тут непонятно - то IOException? то NIOException?
}
}
В задании ничего про папку OPER не увидел.
Про необходимость ее создавать, если ее нет - так же.
Туда же. Т.е. - в топку
1.
if (ErrCount == 0) { // если ошибок не обнаружено выполняем программу
if и простыня текста - за гранью добра и зла.
название переменной:
1. с большой буквы
2. с Count
за гранью добра и зла
1.
try { //начало читаем файл Offices.txt
Try знаем... это хорошо
1.
2.
} catch (IOException q) {
}
ошибки тупо игнорируем?
даже не хочу думать, почему так
но просто игнорировать ошибки - лучше бы ее вообще не обрабатывали
Кроме того, дальше по коду будет try for resource. Почему здесь тогда просто try?
Или Вы знаете / умеете try с ресурсами, тогда его всюду и впихиваете. Или не умеете или не любите - тогда нафиг он вообще где-то. Делаете одностильно. Пусть ужасно, но однообразно
Файл в случае ошибки НЕ закрывается. (а вот это уже не предирка, а реальный косяк)
Чтение PosArrayList - явно напрашивается на отдельную процедуру/ф-цию
Нужна ли внутри нее обработка ошибок, вообще под вопросом. Ну не обработали ошибку, ну пролезет exception на уровень выше. Если использовать try с ресурсами, то все нормально (но у Вас тут не так)
IMHO Если код можно не писать - лучше его не писать. Нет кода - нет ошибок программиста. IMHO чисто мое
1.
Charset DefCharsetOper = Charset.forName("windows-1251");
Нигде про кодировки в задании ничего не было. Почему windows-1251 ?
Туда же - в топку
1.
2.
if (Files.exists((Paths.get(FoolPathOperFiles)))) {
System.out.println("Измените параметр.Файл уже существует " + FoolPathOperFiles);
Или эту ситуацию мы считаем ошибкой - выдаем исключение и пошли все нафиг
Или это не ошибка. Вроде стандартное поведение - просто перезатираем файл. Чем оно не устраивает? Отличное от стандартного поведение в задание указано?
Туда же - в топку
Бл№;%.
Почему переменная
Is Digit ?
И с заглавной буквы.
Вроде же
try for resource
мне кажется в этом случае .close() лишний
поменять close на комментарий, что закрывать файл не нужно, т.к. try for resource
Note: проверить по документации! Я не уверен
1.
2.
3.
} catch (IOException x) {
x.printStackTrace();
}
Вывели простыню на экран.... и ПРОИГНОРИРОВАЛИ ошибку
Замечательно
Или ошибки НЕ обрабатываем и просто их отправляем автоматом на уровень выше. Или как-то нужно обрабатывать. Раз в задании ничего не сказано - можно ничего и не делать.
При нормальном течении процесса - пытались закрыть файл. При ошибке - закрытия файла делать не стали. Опять раздвоение личности.
Туда же - в топку
1.
2.
3.
WriterOper.flush();
String Progress = String.format("%d \r", i);
System.out.print(Progress);
Зачем flush?
System.out в рабочем коде? Вы серьезно?
Туда же - в топку
проще самому написать, чем пытаться чужой код перелопачивать.
половину делать не просили, не зачем было и делать.
IMHO & AFAIK
Обработка ошибок за гранью добра и зла. Лучше бы ее не было. Ну есть ошибка, ну выскочит стандарное сообщение в консоль. Обрабатывать ошибки в задании же не просили.
В main можно вообще было без всякого errCode обойтись. Отругаться в консоль и return.
IMHO
много знаний - много печали
не забывайте, что все "суета сует"
Проще надо быть. Особенно с тестовыми заданиями. Которые расчитаны на уровень студента/школьника.
IMHO
