check for vault money sign

hey everyone,
i'm thinking of a way to make kinda like a sign shop thing xD
i just need to know how to check the amount of money that someone has
so i got this code:

Code:

@EventHandler
                         public void onPlayerInteract94(PlayerInteractEvent e) {
                         if (e.getAction() != Action.RIGHT_CLICK_BLOCK) return;
                         if ((e.getClickedBlock().getState() instanceof Sign)) {
                           Sign s = (Sign)e.getClickedBlock().getState();
                           if (s.getLine(1).equalsIgnoreCase("§6Rank 2")) {
                              
                               //check for amount of money
                               // take away the right amount of money
                               // do other stuff
                           }
                         }
                         }

By the way, the money is from vault.

 

@HenkDeKipGaming

• Why are you inverting the if statment, only then to return? Say if it's equal to RightCLICKBLOCK
• Look at Vault's Docs, Vault's API, and all the methods it has.
• Format your code.

 

thanks i'll try that

 

@Zombie_Striker Hey, i tried this:

Code:

public static Economy econ = null;
                       
                       //spawnpoint setting
                         @EventHandler
                         public void onPlayerInteract94(PlayerInteractEvent e) {
                         if (e.getAction() != Action.RIGHT_CLICK_BLOCK) return;
                         if ((e.getClickedBlock().getState() instanceof Sign)) {
                           Sign s = (Sign)e.getClickedBlock().getState();
                           if (s.getLine(1).equalsIgnoreCase("§6Rank 2")) {
                               
                             
                               
                               @SuppressWarnings("deprecation")
                                EconomyResponse r = econ.withdrawPlayer(e.getPlayer().getName(), 10);
                               if (r.transactionSuccess()) {
                                  Location Location = e.getPlayer().getLocation();
                                  e.getPlayer().setBedSpawnLocation(Location, true);
                                    e.getPlayer().sendMessage("Your spawnpoint has been set");
                                       
                               }
                               else {
                                       e.getPlayer().sendMessage(ChatColor.RED + "You can't afford this kit :(");
                                       
                               }
                             
                             
                             
                                 
                           }
                         }

and it doesn't do anything... what to do?

 

• Is econ Null?
• Did you register a Listener?
• Why is it still inverted?
• What part of this does not work?

 

I did not register a listener, and there are no errors.. but it does nothing when i click the sign

 

@HenkDeKipGaming
Please learn Bukkit (and possibly Java) before you continue. It is obvious you do not know what you're doing and there is no way to help you at this point. Please look at this link before you continue with your project.
http://bukkit.org/threads/plugin-dev-sticky-learning-java-where-to-learn.350132/

 

@HenkDeKipGaming It wont work if you did not register events. To register events, In your main class you have to add method in onEnable()

Code:java

 public void onEnable() {
Bukkit.getServer().getPluginManager().registerEvents(new [Event's class name](this), this);
// If you have single class you will do (this, this)

 

Nothing wrong with doing this really, it's a matter of personal preference. Personally I prefer to do this where it makes sense to in order to reduce the nested nature of it In my opinion the following:

Code:

@EventHandler
public void onInteract(PlayerInteractEvent e) {
    if(e.getAction() == Action.RIGHT_CLICK_BLOCK) {
        if((e.getClickedBlock().getState()) instanceof Sign) {
            Sign s = (Sign) e.getClickedBlock().getState();
            if(s.getLine(1).equalsIgnoreCase("whatever") {
                // Do stuff here
            }
        }
    }
}

Isn't quite as nice as the following, due to the excessive indentation:

Code:

@EventHander
public void onInteract(PlayerInteractEvent e) {
    if(e.getAction() != Action.RIGHT_CLICK_BLOCK)
        return;

    if(!(e.getClickedBlock().getState() instanceof Sign))
        return;

    Sign s = (Sign) e.getClickedBlock().getState();
    // etc.
}

@HenkDeKipGaming The above advice is correct either way: You need to learn more about Java, and then more about the API that you are using, before trying to make plugins, otherwise you'll just run into simple mistakes that you are unable to solve. I recommend the Oracle tutorials, or a good Java book

 

@AdamQpzm @Zombie_Striker yeah, you are right... i wantto do hard things before easy things xD

@CoolGamerXD thanks, i'll try that

 

I was the same way but you got to work one step at a time.

 

I'll do that I just started and i'm allready making Minigame plugins... that's not how it works haha.
i first gotta learn xD

 

im not sure about you but I had basic Java knowledge before starting and I kept learning as I went along. Just don't quit cause it will get frustrating at times.