You didn't quite understand me..
For example:
+ if (command.startsWith("five"))
+ {
+ LeagueOfElo league = new LeagueOfElo();
+ league.vs5reg(player);
+ player.sendMessage("Registration Successful");
+ if (league.vs5.size() == 10)
+ {
+ league.pick5v5();
+ }
+ }
Here, it is unnecessary to make a new object of LeagueOfElo class, you can change pick5vs5() method and vs5reg() to static, and call them like this:
LeagueOfElo.pick5vs5();
LeagueOfElo.vs5reg();
As well for vs5 variable which is static.
But my point is that you have not coded it in a good way. There are unused stuff in there and more.
You should have created for example an abstract class Match, and other classes afterwards that extend this class, to handle the 5vs5 match, 2vs2 and such. Also in Match class you should have abstract methods that all subclasses should override(must). So instead of this ugly code you would have a nice organized way to handle matches, using inheritance.
More stuff:
+ ii.SetRankPts(ii.getRankPts() + 105);
+ if (ii.getRankPts() == 100)
+ {
+ ii.setRank("Ranked");
+ }
if (ii.getRankPts() == 100)
This line is incorrect, you should check if it's >= 100 and if his rank is not Ranked (!ii.getRank().equals("Ranked")).
Also, here:
ii.SetRankPts(ii.getRankPts() + 105);
I think you meant 15..
Unused/Unneeded stuff:
+ if (isTurret())
+ {
+
+ }
+ public class Start implements Runnable
+ {
+ @Override
+ public void run()
+ {
+
+ }
+ }
+
+ public void createTeam(L2PcInstance a)
+ {
+
+ }