Solved Healing isnt cool..

Discussion in 'Plugin Development' started by MCSwicts, Feb 26, 2016.

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

    MCSwicts

    Invalid 404 Error
     
    Last edited: Feb 27, 2016
  2. Offline

    Zombie_Striker

    If you mean you're only been coding in Java for one week, then I need to tell you to stop working on bukkit until you have fully learned Java. Knowing Java is a requirement for making bukkit plugins because it will make understanding bukkits incredibly clear. If you mean you have been learning Bukkit the last week, then I would highly recommend finding a new tutorial because the following shows the tutorial you're learning from is terrible.

    Either way, you should really find a tutorial here.
    https://bukkit.org/threads/plugin-dev-sticky-learning-java-where-to-learn.395662/


    1. Don't steal Minecraft's Logger! Use getLogger to get the logger
    2. Don't Abuse Static. There is no reason to have a static instance of a class inside a class in this situation.
    3. Use Constructors to pass the Main's instance between classes.

    Bukkit already logs your plugins for you. There is no reason to say it's enabled twice.

    After checking to make sure it's not a player, you then check if it is a a player. Please remove one of the checks.

    The "targer" may not even be online. Lets say I try to get player "NonExistandPlayer", which is not online. If the player is not online/does not exist, this will throw a NullPointerException.

    Check if (target != null) before doing anything with target.
     
    Last edited: Feb 26, 2016
  3. Offline

    MCSwicts

    Bumb
     
    Last edited: Feb 27, 2016
  4. Offline

    Evonoucono

    Please go to the plugin request forum if you want a plugin/code made for you.
     
Thread Status:
Not open for further replies.

Share This Page