issues
search
vasiliilantratov
/
liga-internship
Задачи на занятиях
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
SongChangeTest. Много логики, не относящейся к тесту
#11
vladimirkoba
opened
6 years ago
1
SongAnalysisTaskTest - плохие названия тестов
#10
vladimirkoba
opened
6 years ago
1
Почему метод публичный?
#9
boris-dev
closed
6 years ago
2
Т.е. SongChange также умеет сохранять файлы? Не слишком ли много ответственности для одного класса?
#8
boris-dev
closed
6 years ago
2
Не понял смысл этого интерфейса
#7
boris-dev
closed
6 years ago
1
Идея хорошая, но почему логгер статический?
#6
boris-dev
closed
6 years ago
1
Немного с именованием не понял, если он Write, а от него наследуются Writer-ы, write же глагол? Поясни логику
#5
boris-dev
closed
6 years ago
1
возможно writer не будет закрыт если получим исключение
#4
boris-dev
opened
6 years ago
2
Можно ли было найти более элегантный способ, чтобы не использовать эти try-catch и повысить читабельность кода?
#3
boris-dev
closed
6 years ago
1
Ошибки нужно логировать logger.error()
#2
boris-dev
closed
6 years ago
1
Если поля не меняются ставим final
#1
boris-dev
closed
6 years ago
1