Blockify

Blockify 1.1.2

Нет прав для скачивания
534
46
178
Прикольно, но зачем?
** тут должна быть картинка тролейбуса из буханки хлеба **
 

tox1cozZ

aka Agravaine
8,351
593
2,816
Не надо создавать каждый раз новый поток. Есть ExecutorService, почитай про него, он один раз создаёт поток(и), кладёт в пул и переиспользует. Это намного выгоднее, так как создание нового потока довольно тяжелая операция.
 
534
46
178
Старый стиль работы с ресурсами, актуальный для времён Java 6. В сам try нужно прописывать создание ресурса.

Также в чём смысл использования Logger, если ошибки всё равно высираются в stderr через printStackTrace() ? Если уж осилили Logger, то используйте его везде.
 

tox1cozZ

aka Agravaine
8,351
593
2,816
Не стоит использовать newCachedThreadPool, он порождает огромное количество потоков. В твоём случае достаточно выделить один поток (newSingleThreadExecutor) и в нем слать запросы.
 
Сверху