added capacity of ArrayList to constructor to avoid unnecessary array expansion
parent
3b00419cbb
commit
e326643d0e
|
@ -52,16 +52,18 @@ public abstract class MagicChoice {
|
||||||
|
|
||||||
/** Gets the choice results for AI. */
|
/** Gets the choice results for AI. */
|
||||||
public List<Object[]> getArtificialChoiceResults(
|
public List<Object[]> getArtificialChoiceResults(
|
||||||
final MagicGame game,final MagicEvent event,final MagicPlayer player,final MagicSource source) {
|
final MagicGame game,
|
||||||
|
final MagicEvent event,
|
||||||
|
final MagicPlayer player,
|
||||||
|
final MagicSource source) {
|
||||||
|
|
||||||
final Collection<Object> options=getArtificialOptions(game,event,player,source);
|
final Collection<Object> options=getArtificialOptions(game,event,player,source);
|
||||||
final int size=options.size();
|
final int size=options.size();
|
||||||
if (size==1) {
|
if (size==1) {
|
||||||
return Collections.singletonList(new Object[]{options.iterator().next()});
|
return Collections.singletonList(new Object[]{options.iterator().next()});
|
||||||
} else if (size>1) {
|
} else if (size>1) {
|
||||||
final List<Object[]> choiceResultsList=new ArrayList<Object[]>();
|
final List<Object[]> choiceResultsList=new ArrayList<Object[]>(size);
|
||||||
for (final Object option : options) {
|
for (final Object option : options) {
|
||||||
|
|
||||||
choiceResultsList.add(new Object[]{option});
|
choiceResultsList.add(new Object[]{option});
|
||||||
}
|
}
|
||||||
return choiceResultsList;
|
return choiceResultsList;
|
||||||
|
@ -82,4 +84,4 @@ public abstract class MagicChoice {
|
||||||
|
|
||||||
return NO_CHOICE==choiceResult;
|
return NO_CHOICE==choiceResult;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue