Inefficient use of keySet iterator instead of entrySet iterator
This method accesses the value of a Map entry, using a key that was retrieved from a keySet iterator. It is more efficient to use an iterator on the entrySet of the map, to avoid the Map.get(key) lookup.
Off-topic - Até votei num RFE para que não tenhamos de ficar digitando a toda hora "Map<Integer,Acomodacao>". Não seria para criar uma classe (como é feito hoje), mas para ter alguma construção que criasse um sinônimo (alias) para "Map<Integer,Acomodacao>".
Paulo_Silveira
oi thingol
eh isso mesmo, mas voce usando o put ta perdendo a vantagem de usar Map.Entry, voce deve usar o setValue do Entry atual para nao ter de acessar a estrutura interna do mapa e fazer isso direto.
legal o RFE, eh tipo um import AS, que ja pediram faz tempo… mas eu tb nao acho tao vital.
T
thingol
Nem tinha visto que tinha um “put” no meio do loop. É claro que é melhor usar setValue…
marcioa1
Será que é pedir demais para alguém refatorar meu código ? Estou tendo dificuldades com o setValue no lugar do put.