2つのforループは、時間を短縮するために何らかの解決策を実行するのに時間がかかりますJAVA

Question

これはjavaでの私のデータです。処理が速く、応答が得られない場合、このコードをどのように高速化するために最適ですか？

List<EventLog> eventLogs = new ArrayList<EventLog>(); 
List<EventLog> eventLogData = get(currentUser, data); 
Map<String, Integer> map = new HashMap<String, Integer>(); 

for (EventLog rep : eventLogData) { 
    if (map.containsKey(rep.getEventType())) { 
    map.put(rep.getEventType(), map.get(rep.getEventType()) + 1); 
    } else { 
    map.put(rep.getEventType(), 1); 
    } 
} 

for (Map.Entry<String, Integer> entry : map.entrySet()) { 
    EventLog list = new EventLog(); 
    list.setEventType(entry.getKey()); 
    list.setCount(entry.getValue()); 
    eventLogs.add(list); 
} 

return eventLogs; 

Answer 1

list.size()を使用してリストのサイズを取得しますが、代わりにforeachループのために使用することができ、このような一つのループでコードを変更します。

for(int a=0, b=5 ; a<=5 ; a++,b--){ 
    // do your stuff here 
} 

Answer 2

を、あなたは、代替高性能コレクションライブラリを使用して考えがありますか？
Troveライブラリを使用して実装しました。それはプリミティブなintの代わりに整数ののを扱う

マップが更新され

1. ので、はるかに高速実行が元の実装に代わり2の単一のアクセス（adjustOrPutValue）が必要（のcontainsKeyおよび取得）を提供し、すべての回避しますボクシングおよびアンボクシング操作を行い、より少ないメモリを消費する。

。

List<EventLog> eventLogData = get(currentUser, data); 
TObjectIntMap<String> map = new TObjectIntHashMap<>(); 

for (EventLog rep : eventLogData) { 
    map.adjustOrPutValue(rep.getEventType(), 1, 1); 
} 

List<EventLog> eventLogs = new ArrayList<>(); 
map.forEachEntry(new TObjectIntProcedure<String>() { 
    @Override 
    public boolean execute(String key, int value) { 
     EventLog list = new EventLog(); 
     list.setEventType(key); 
     list.setCount(value); 
     eventLogs.add(list); 
     return true; 
    } 
}); 

return eventLogs;