Task_2_4_1#14
Conversation
|
Я не знаю почему билд не проходит и что такое GradleWrapperMain |
|
Зато я знаю, что код просто ужасен и заранее за него извиняюсь. |
|
С другой стороны и лаба не подарок, так и живем |
возможно, дело в этом: у тебя пропущен jar-файл в папке wrapperа |
galinabykova
left a comment
galinabykova
left a comment
There was a problem hiding this comment.
ещё объясни, пожалуйста, разделение по пакетам. Почему Core и Service?
|
|
||
| for (Check check : config.getChecks()) { | ||
| Student student = config.getStudent(check.studentNick()); | ||
| Task task = config.getTasks().get(check.taskId()); |
There was a problem hiding this comment.
я бы сделала в конфиге метод getTask, потому что ты всегда вызываешь getTasks(), чтобы получить из результата задачу по индексу
| boolean styleOk = false; | ||
|
|
||
| if (buildOk) { | ||
| System.err.println("[INFO] Generating doc: " + task.id()); |
There was a problem hiding this comment.
хорошо бы хотя бы свой кастомный логгер сделать
There was a problem hiding this comment.
потому что сильно много дублирования кода, пусть и маленькими кусочками, и поменять формат логов будет больно
| * @param args . | ||
| * @throws Exception . | ||
| */ | ||
| public static void main(String[] args) throws Exception { |
There was a problem hiding this comment.
кажется, этот метод делает слишком много
There was a problem hiding this comment.
выдели отдельные задачи (например, поиск папки репозитория с её возможным созданием) в отдельный класс
There was a problem hiding this comment.
ещё было бы хорошо, если бы ты подумал о том, как унифицировать обработку проверок. Может быть можно все проверки объединить общим интерфейсом, чтобы цепочку однотипных ifов заменить одним циклом?
There was a problem hiding this comment.
а почему BuildService, GitService имеют Service в имени, а DocGenerator и StyleChecker - нет?
There was a problem hiding this comment.
объясни, почему этот класс с геттерами и сеттерами, а, например, Group рекорд
| import java.util.List; | ||
|
|
||
| /** | ||
| * . |
There was a problem hiding this comment.
не забудь написать документацию)
2 participants
No description provided.