Solved Whats wrong here? Message showing two things.

Discussion in 'Plugin Development' started by mrsamcraft, Jan 6, 2014.

Thread Status:
Not open for further replies.
  1. So Im creating a simple level up message for my server, but I'm stumped here... The level up works fine, But i want to display a message if the player uses/losses levels... Its still showing the level up message when the player loses/uses some...

    Code:java
    1. @EventHandler
    2.  
    3. public void onPlayerLevelUpEvent(PlayerLevelChangeEvent event) {
    4.  
    5. Player player = event.getPlayer();
    6. String displayName = player.getDisplayName();
    7.  
    8.  
    9. //Message if player levels up
    10. if (event.getNewLevel() > event.getOldLevel())
    11. Bukkit.broadcastMessage( displayName + ChatColor.GOLD + " Leveled up!");
    12. player.sendMessage(ChatColor.AQUA + "Congratulations, you've leveled up!" + ChatColor.GOLD + " Level: " + ChatColor.GOLD + player.getLevel());{
    13.  
    14.  
    15.  
    16. //Message if lost or usese levels
    17. if (event.getOldLevel() < event.getNewLevel())
    18. player.sendMessage(ChatColor.RED + "You have lost/used some levels!" + ChatColor.GOLD + " Now level: " + ChatColor.LIGHT_PURPLE + player.getLevel());
    19.  
    20. }
    21. }
    22. }


    Sorry If this is easy... But I just can't work this out... ahah!

    Well you learn from mistakes!

    ~ Sam
     
  2. Offline

    Xephiro

    Only you need sort your code.

    :D
    Code:java
    1. @EventHandler
    2. public void onPlayerLevelUpEvent(PlayerLevelChangeEvent event) {
    3.  
    4. Player player = event.getPlayer();
    5. String displayName = player.getDisplayName();
    6.  
    7.  
    8. //Message if player levels up
    9. if (event.getNewLevel() > event.getOldLevel())
    10. {
    11. Bukkit.broadcastMessage( displayName + ChatColor.GOLD + " Leveled up!");
    12. player.sendMessage(ChatColor.AQUA + "Congratulations, you've leveled up!" + ChatColor.GOLD + " Level: " + ChatColor.GOLD + player.getLevel());
    13. }
    14. else if (event.getOldLevel() < event.getNewLevel())
    15. {
    16. player.sendMessage(ChatColor.RED + "You have lost/used some levels!" + ChatColor.GOLD + " Now level: " + ChatColor.LIGHT_PURPLE + player.getLevel());
    17. }
    18. }


    Good Luck
     
    mrsamcraft likes this.
  3. @Xephiro
    ahah! thanks, Still learning as I go along! and I need to only code when I'm awake... 0_o.

    Thanks once again!
    ~ Sam

    @Xephiro
    Just tested the code and It's not displaying anything when I use some levels, only displays text when I level up?

    ~ Sam

    EDIT: FIXED!

    Thanks!

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

Share This Page