Blockify

Blockify 1.1.2

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

tox1cozZ

aka Agravaine
8,454
598
2,890
Не надо создавать каждый раз новый поток. Есть ExecutorService, почитай про него, он один раз создаёт поток(и), кладёт в пул и переиспользует. Это намного выгоднее, так как создание нового потока довольно тяжелая операция.
 

TheAndrey

Редактор учебника
1,071
72
372
Старый стиль работы с ресурсами, актуальный для времён Java 6. В сам try нужно прописывать создание ресурса.

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

tox1cozZ

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