Command Executor error

Discussion in 'Plugin Help/Development/Requests' started by Speaw, Oct 13, 2014.

Thread Status:
Not open for further replies.
  1. Offline

    Speaw

    [​IMG]

    Code:java
    1. package com.speaw.mobdefence.komutlar;
    2.  
    3. import org.bukkit.ChatColor;
    4. import org.bukkit.command.Command;
    5. import org.bukkit.command.CommandExecutor;
    6. import org.bukkit.command.CommandSender;
    7. import org.bukkit.entity.Player;
    8. import org.bukkit.event.EventHandler;
    9.  
    10. import com.speaw.mobdefence.Main;
    11.  
    12. public class Anakomut implements CommandExecutor{
    13.  
    14. @Override
    15. @EventHandler
    16. public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    17. {
    18. if(sender instanceof Player){
    19. if(args.length == 0)
    20. {
    21. sender.sendMessage("@Copyrigt ****");
    22. return true;
    23. }
    24. if(args[0].equalsIgnoreCase("izinver"))
    25. {
    26. if (sender.isOp()){
    27. if (Main.instance.izinVer.contains(sender)) {
    28. Main.instance.izinVer.remove(sender);
    29. sender.sendMessage(ChatColor.GREEN + "İzin kapatıldı, artık blok kırıp koyamazsın!");
    30. } else {
    31. Main.instance.izinVer.add((Player)sender);
    32. sender.sendMessage(ChatColor.GREEN + "İzin verildi, artık blok kırıp koyabilirsiniz!");
    33. }
    34. }
    35. return true;
    36. }
    37. if(args[0].equalsIgnoreCase("lobiata"))
    38. {
    39. if (sender.isOp()){
    40. Main.instance.getConfig().set("Lobi.X", Double.valueOf(((Player)sender).getLocation().getX()));
    41. Main.instance.getConfig().set("Lobi.Y", Double.valueOf(((Player)sender).getLocation().getY()));
    42. Main.instance.getConfig().set("Lobi.Z", Double.valueOf(((Player)sender).getLocation().getZ()));
    43. Main.instance.getConfig().set("Lobi.Yaw", Double.valueOf(((Player)sender).getLocation().getYaw()));
    44. Main.instance.getConfig().set("Lobi.Pitch", Double.valueOf(((Player)sender).getLocation().getPitch()));
    45. Main.instance.getConfig().set("Lobi.World", ((Player)sender).getLocation().getWorld().getName());
    46. Main.instance.saveConfig();
    47. sender.sendMessage("§3Lobi koordinatları ayarlandı.");
    48. }
    49. return true;
    50. }
    51. if(args[0].equalsIgnoreCase("kirmiziata"))
    52. {
    53. if (sender.isOp()){
    54. Main.instance.getConfig().set("kirmizi.X", Double.valueOf(((Player)sender).getLocation().getX()));
    55. Main.instance.getConfig().set("kirmizi.Y", Double.valueOf(((Player)sender).getLocation().getY()));
    56. Main.instance.getConfig().set("kirmizi.Z", Double.valueOf(((Player)sender).getLocation().getZ()));
    57. Main.instance.getConfig().set("kirmizi.Yaw", Double.valueOf(((Player)sender).getLocation().getYaw()));
    58. Main.instance.getConfig().set("kirmizi.Pitch", Double.valueOf(((Player)sender).getLocation().getPitch()));
    59. Main.instance.getConfig().set("kirmizi.World", ((Player)sender).getLocation().getWorld().getName());
    60. Main.instance.saveConfig();
    61. sender.sendMessage("§3Kirmizi takim koordinatları ayarlandı.");
    62. }
    63. return true;
    64. }
    65. if(args[0].equalsIgnoreCase("kirmiziboss"))
    66. {
    67. if (sender.isOp()){
    68. Main.instance.getConfig().set("kirmiziboss.X", Double.valueOf(((Player)sender).getLocation().getX()));
    69. Main.instance.getConfig().set("kirmiziboss.Y", Double.valueOf(((Player)sender).getLocation().getY()));
    70. Main.instance.getConfig().set("kirmiziboss.Z", Double.valueOf(((Player)sender).getLocation().getZ()));
    71. Main.instance.getConfig().set("kirmiziboss.Yaw", Double.valueOf(((Player)sender).getLocation().getYaw()));
    72. Main.instance.getConfig().set("kirmiziboss.Pitch", Double.valueOf(((Player)sender).getLocation().getPitch()));
    73. Main.instance.getConfig().set("kirmiziboss.World", ((Player)sender).getLocation().getWorld().getName());
    74. Main.instance.saveConfig();
    75. sender.sendMessage("§3Kirmizi boss koordinatları ayarlandı.");
    76. }
    77. return true;
    78. }
    79. if(args[0].equalsIgnoreCase("maviata"))
    80. {
    81. if (sender.isOp()){
    82. Main.instance.getConfig().set("mavi.X", Double.valueOf(((Player)sender).getLocation().getX()));
    83. Main.instance.getConfig().set("mavi.Y", Double.valueOf(((Player)sender).getLocation().getY()));
    84. Main.instance.getConfig().set("mavi.Z", Double.valueOf(((Player)sender).getLocation().getZ()));
    85. Main.instance.getConfig().set("mavi.Yaw", Double.valueOf(((Player)sender).getLocation().getYaw()));
    86. Main.instance.getConfig().set("mavi.Pitch", Double.valueOf(((Player)sender).getLocation().getPitch()));
    87. Main.instance.getConfig().set("mavi.World", ((Player)sender).getLocation().getWorld().getName());
    88. Main.instance.saveConfig();
    89. sender.sendMessage("§3Mavi takim koordinatları ayarlandı.");
    90. }
    91. return true;
    92. }
    93. if(args[0].equalsIgnoreCase("maviboss"))
    94. {
    95. if (sender.isOp()){
    96. Main.instance.getConfig().set("maviboss.X", Double.valueOf(((Player)sender).getLocation().getX()));
    97. Main.instance.getConfig().set("maviboss.Y", Double.valueOf(((Player)sender).getLocation().getY()));
    98. Main.instance.getConfig().set("maviboss.Z", Double.valueOf(((Player)sender).getLocation().getZ()));
    99. Main.instance.getConfig().set("maviboss.Yaw", Double.valueOf(((Player)sender).getLocation().getYaw()));
    100. Main.instance.getConfig().set("maviboss.Pitch", Double.valueOf(((Player)sender).getLocation().getPitch()));
    101. Main.instance.getConfig().set("maviboss.World", ((Player)sender).getLocation().getWorld().getName());
    102. Main.instance.saveConfig();
    103. sender.sendMessage("§3Mavi boss koordinatları ayarlandı.");
    104. }
    105. return true;
    106. }
    107. }
    108. return true;
    109. }
    110. }


    Code:java
    1. getCommand("md").setExecutor(new Anakomut());
     
  2. Offline

    drpk

    Speaw Why do you have an @EventHandler over your command?
     
  3. Offline

    teej107

    Stop using static. Learn to handle parameters in constructors.
     
  4. Offline

    mine-care

    Speaw Ask for help where you got your build. Bukit forums are for Bukkit.

    teej107 is right, also you could use a method instead of typing 4 times liturealy the same things (parameters in methods would help)
    Also an efficiency thingy, probably it would be a good idea to load the things from config in variable-s and instead of reading from config all time, read from memory.

    EDIT by Moderator: merged posts, please use the edit button instead of double posting.
     
    Last edited by a moderator: Jun 14, 2016
Thread Status:
Not open for further replies.

Share This Page