Blockify

Blockify 1.1.2

Нет прав для скачивания

tox1cozZ

aka Agravaine
8,455
598
2,892
Не надо создавать каждый раз новый поток. Есть ExecutorService, почитай про него, он один раз создаёт поток(и), кладёт в пул и переиспользует. Это намного выгоднее, так как создание нового потока довольно тяжелая операция.
 
1,074
72
372
Старый стиль работы с ресурсами, актуальный для времён Java 6. В сам try нужно прописывать создание ресурса.

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

tox1cozZ

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