Conversation
| @@ -1,50 +1,165 @@ | |||
| package ru.mipt.java2016.homework.g596.proskurina.task2; | |||
There was a problem hiding this comment.
Лучше перенеси новое в .task3, в .task2 оставь всё, как оно было во втором задании
There was a problem hiding this comment.
В java по умолчанию поля инициализируются именно так
There was a problem hiding this comment.
Вот такой Integer очень опасно использовать как лок
There was a problem hiding this comment.
Выноси такие константы как static final поля
There was a problem hiding this comment.
У тебя получается однопоточная реализация этого метода: одновременно только один поток может узнавать размер. С ReadWriteLock можно быстрее
There was a problem hiding this comment.
Не бросай закомментированный код в репозитории. Ему так грустно и одиноко от этого
There was a problem hiding this comment.
FileWorker же Closeable. Используй try-with-resources и тогда close() вызовется автоматически и при любых неожиданностях
There was a problem hiding this comment.
Повторяются строчки
deleteKeyFile.close();
keyPositionFile.close();тут и в rebuild. Почему бы их просто не вынести из if?
|
|
||
| @Override | ||
| public boolean exists(K key) { | ||
| wlock.lock(); |
There was a problem hiding this comment.
А почему тут wlock?
И в readKeys
2 participants
No description provided.