Bug #5829

duplicate code и другие мелочи

Added by Alexey Demakov about 2 years ago.

Status:NewStart date:04/13/2015
Priority:NormalDue date:
Assignee:Andrei Tatarnikov% Done:

0%

Category:-Spent time:-
Target version:-
Detected in build:svn Published in build:
Platform:
Code review: trunk/microtesk/src/main/java/core/ru/ispras/microtesk/utils/PrintingUtils.java@3490:line 35

Description

1. Добавлен класс
/microtesk/src/main/java/core/ru/ispras/microtesk/Logger.java
в котором метод header дублирует метод printHeader класса
/microtesk/src/main/java/core/ru/ispras/microtesk/utils/PrintingUtils.java

2. В методе header есть дополнительная проверка null == text, отсутствие которой в printHeader - потенциальная ошибка.

3. Для вывода "\r\n" лучше использовать платформенно-независимое: System.getProperty( "line.separator" ). Начиная с Java 7 есть System.lineSeparator().

P.S. В Jenkins есть возможность смотреть отчёты инструментов статического анализа по проекту MicroTESK: http://forge.ispras.ru:8080/view/SAR/job/SAR%20MicroTESK/

Аналогично буду добавлять другие проекты. Приоритеты - по запросам разработчиков.

Also available in: Atom PDF