Спуск SwingWorker нить сна

Question

У меня есть Swing GUI, который должен периодически читать rss-канал (фид предоставляется приложением, которое я написал). Я реализовал это в SwingWorker и хочу спросить, является ли это звуковой дизайн.

Вот код, который я написал для фона метод:

@Override 
protected Void doInBackground() throws Exception { 

    comboBoxModel.removeAllElements(); 

    while(true) { 

     Channel feedChannel = webResource.path(role.role()) 
             .accept(MyMediaType.APPLICATION_RSS) 
             .get(Rss.class).getChannel(); 

     List<Item> itemList = feedChannel.getEntryList(); 

     for(Item item : itemList) { 

      String itemLink = item.getLink(); 

      if(!feedItemMap.containsKey(itemLink)) { 

       comboBoxModel.addElement(new Pair<>(itemLink, true)); 
       feedItemMap.put(itemLink, true); 
      } 
     } 

     Thread.sleep(10000); 
    } 
} 

Моя идея позволяя спать нить уменьшить количество GETs. Моя программа не должна обновляться каждые миллисекунды, поэтому для снижения нагрузки на сервер я думал, что это будет хорошей идеей.

После прочтения этого вопроса (java - thread.sleep() within SwingWorker) Я теперь смущен, если это была хорошая идея. Я собрал то, что вы обычно не пропускаете фоновый поток, потому что система позаботится об этом.

Однако в моем случае этот метод, по-видимому, полезен для меня.

Может ли кто-нибудь уточнить возможные проблемы с этой реализацией?

Любой ввод оценивается. Это касается конкретной темы, а также общего дизайна. Если я делаю что-то неправильно, я хочу это знать. :)

Привет

Answer 1

Используйте ScheduledExecutorService:

ScheduledExecutorService ses = Executors.newSingleScheduledExecutorService(); 
ses.scheduleAtFixedRate(new Runnable() { 
    @Override 
    public void run() { 
     doInBackground(); 
    } 
}, 10, TimeUnit.SECONDS); 

и избавиться от Thread.sleep и петли while(true):

protected Void doInBackground() throws Exception { 
    comboBoxModel.removeAllElements(); 
    Channel feedChannel = webResource.path(role.role()) 
            .accept(MyMediaType.APPLICATION_RSS) 
            .get(Rss.class).getChannel(); 
    List<Item> itemList = feedChannel.getEntryList(); 
    for(Item item : itemList) { 
     String itemLink = item.getLink(); 
     if(!feedItemMap.containsKey(itemLink)) { 
      comboBoxModel.addElement(new Pair<>(itemLink, true)); 
      feedItemMap.put(itemLink, true); 
     } 
    } 
} 

Answer 2

Размах исполнитель (на моей машине) отладки только пожары до 10 потоков для всех SwingWorkers. Это означает, что если 10 задач спят, больше никаких задач не может быть начато до тех пор, пока не вернется. однако вы можете отправить SwingWorker любому исполнителю для обработки вместо того, чтобы полагаться на исполнителя по умолчанию.

Существует еще одна проблема в вашем коде, где вы получаете доступ к объекту графический интерфейс из другого потока, вы должны вместо publish пары и использовать метод обработки, чтобы справиться с этим:

if(!feedItemMap.containsKey(itemLink)) { 
     publish(itemLink); 
    } 


public void process(List<String> strings){ 
    for(String str: strings){ 
     comboBoxModel.addElement(new Pair<>(str, true)); 
     feedItemMap.put(str, true); 
    } 
}