rinefica / split Goto Github PK

1. ... .append('\n'); Корректным ли форматом будет обладать создаваемый файл с точки зрения различных операционных систем?

2. Если статичное поле baseOutputFileName может быть тождественно нестатичному inputFileName - точно ли оно должно быть таковым? Если у нас действует несколько объектов, отвечающих за парсинг КС, то у них может быть только единый формат именования выходных файлов?

3. Информация об ошибках обычно выводится в System.err;

4. Нужно ли продолжать выполнение программы, например, в writeFiles, если уже было получено исключение при попытке создания объекта класса Reader? Upd. Still on - посмотрите, например, метод Split.writeFiles(..) - плохого в текущей реализации ничего не произойдёт, но обрабатывать последующие if-ы, при падении, например, в writeByCountFiles - явно не стоит. Кроме того, если не предполагается каких-либо отличий от базовой схемы обработки - исключения стоит просто прокидывать наверх (то же самое касается и тестов);

5. Метод writeFiles не использует поля класса и может быть объявлен как статический. Про представление методов в памяти JVM можно посмотреть здесь;

6. Для работы с ресурсами - например, при чтении файлов - возможно более комфортным решением является применение паттерна 'try-with-resources'.

7. Не вполне понятно, почему в классе ParamsSplit часть методов, общих по решаемым задачам с public, стала package-private.

1. Что произойдёт, если при разбиении файла по заданному количеству символов мы выйдем за пределы набора существующих постфиксов имён выходных файлов?

2. Не вполне понятно, почему полное количество двухсимвольных комбинаций a-z Вы вычисляете как countDivElements > 26*27 - 1. Upd. Still on;

3. По заданию, если не задано имя выходного файла, оно должно формироваться как "X" + постфикс;

4. Что произойдёт в ParamsSplit.isCorrectCommand(..), если пользователь хочет получить разбиение по большому количеству символов (а не по количеству файлов, превышающему заданное значение)?

1. В текущей реализации нет тестов для выходных файлов с именами с текстовым постфиксом/ равными именам входных файлов;

2. Не вполне понятно, зачем нужен System.out.flush, с учётом того, что используется System.err.