issues
search
willstay
/
liga-internship
Задачи на занятиях
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
В каждом файле нужно использовать автоформатирование кода Ctrl+Alt+L
#13
boris-dev
closed
6 years ago
1
Логируем ошибки в logger.error(),
#12
boris-dev
closed
6 years ago
1
Тут можно написать тест со своими тестовыми данными, интерфейс класса довольно хороший, мы получаем на вход массив нот, их можно создать самим для каждого теста свой и соответственно быть уверенным в проверках, сейчас выглядит что тесты были подогнаны под файлик zombie.mid
#11
boris-dev
opened
6 years ago
0
Если этот клас будет использоваться из двух мест одновременно, то будут проблемы так как у него есть глобальное состояние. Этим очень опасна статика. Можно безболезненно переделать без статики. И более того сделать эту поле локальным
#10
boris-dev
closed
6 years ago
1
С полным путём до файла работает? C:\users\blablabla\song.mid
#9
boris-dev
closed
6 years ago
1
Логику по анализу параметров можно было бы хорошо отдельно протестировать, но для этого нужен рефакторинг
#8
boris-dev
opened
6 years ago
1
Нужна конкретная ошибка, иначе будет ощущение, что программа просто не работает
#7
boris-dev
closed
6 years ago
1
По сути это копия main-a, вторая точка входа которая делает всё
#6
boris-dev
opened
6 years ago
1
Проблемы с тестами в PrinterTest
#5
vladimirkoba
opened
6 years ago
1
Тесты для changeTempoTest очень многословные
#4
vladimirkoba
opened
6 years ago
1
Плохие название тестов
#3
vladimirkoba
opened
6 years ago
1
Напрашивается короткий приватный метод или переменная, уж очень многословно получается
#2
boris-dev
closed
6 years ago
1
Дубляж кода
#1
boris-dev
closed
6 years ago
1