runekingdom

Theres nothing wrong with the conventions?

   @Override
   public void handle(final Player player, Packet packet) {//don't worry about it
      int id = packet.getLEShort() & 0xFFFF;//don't worry about it
      System.out.println(id);//don't worry about it
      if(id < 0 || id >= Constants.MAX_NPCS) {//don't worry about it
         return;
      }

should be:

   @Override
   public void handle(final Player player, Packet packet) {//don't worry about it
      int id = packet.getLEShort() & 0xFFFF;//don't worry about it
                //System.out.println(id);//don't worry about it
      if(id < 0 || id >= Constants.MAX_NPCS) {//don't worry about it
         return;
      }

Cba doing all but you get my drift.
Also you don't need to print the output on everything, thats why I commented it out.