From 3e7d1d028bb9a6f73c6f1edcf17ff91131ce4055 Mon Sep 17 00:00:00 2001 From: Brettflan Date: Wed, 11 Jan 2012 19:11:31 -0600 Subject: [PATCH] /f list sorting method performance boost --- src/com/massivecraft/factions/cmd/CmdList.java | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/com/massivecraft/factions/cmd/CmdList.java b/src/com/massivecraft/factions/cmd/CmdList.java index bc4da1e0..4251db9a 100644 --- a/src/com/massivecraft/factions/cmd/CmdList.java +++ b/src/com/massivecraft/factions/cmd/CmdList.java @@ -46,9 +46,11 @@ public class CmdList extends FCommand Collections.sort(factionList, new Comparator(){ @Override public int compare(Faction f1, Faction f2) { - if (f1.getFPlayers().size() < f2.getFPlayers().size()) + int f1Size = f1.getFPlayers().size(); + int f2Size = f2.getFPlayers().size(); + if (f1Size < f2Size) return 1; - else if (f1.getFPlayers().size() > f2.getFPlayers().size()) + else if (f1Size > f2Size) return -1; return 0; } @@ -58,9 +60,11 @@ public class CmdList extends FCommand Collections.sort(factionList, new Comparator(){ @Override public int compare(Faction f1, Faction f2) { - if (f1.getFPlayersWhereOnline(true).size() < f2.getFPlayersWhereOnline(true).size()) + int f1Size = f1.getFPlayersWhereOnline(true).size(); + int f2Size = f2.getFPlayersWhereOnline(true).size(); + if (f1Size < f2Size) return 1; - else if (f1.getFPlayersWhereOnline(true).size() > f2.getFPlayersWhereOnline(true).size()) + else if (f1Size > f2Size) return -1; return 0; }