Bug #5829: duplicate code и другие мелочи - MicroTESK - Open-Source Projects

Actions

Copy link

Bug #5829

closed

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

Added by Alexey Demakov about 9 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Andrei Tatarnikov

Category:

Target version:

Start date:

04/13/2015

Due date:

% Done:

Estimated time:

Detected in build:

svn

Platform:

Published in build:

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/

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