Setting a sign line and getting the value

Hello,
I am wondering how I would get a specific line on a sign, I have

Code:

                                if (((Sign) event.getClickedBlock().getState()).getLine(0).equalsIgnoreCase("10$")) {
                                        player.sendMessage("+ 10");

but how would I get the value of the line without having to have it set to a exact value in the code? I want to be able to change the value of the price in game on the sign and send the player a message with that same value

- Thanks

 

Keep a list of sign locations, loop through them if the block is sign cast it and then do your changes on it
Don't forget to update it.

 

diamondcodes

You can just call getLine(LINE_NUMBER) and that returns the String of the line, then cut anything you don't need (leaving the price) and send it to the player

 

Thanks. 1 question, For sending the player money the Line is a string so it will not work with "EconomyResponse r = Main.econ.depositPlayer" so how would I give them the money with vault? (Giving them the amount of money from the last line on the sign)

 

diamondcodes

Strip the string of all text and anything non-numerical (or you will get and exception) and them use Integer.parseInt()

 

Watto Here is my code,

Code:java

    @EventHandler
    public void onClick(PlayerInteractEvent event) {
        Player player = event.getPlayer();
 
        if (event.getClickedBlock().getType() == Material.WALL_SIGN || event.getClickedBlock().getType() == Material.SIGN_POST) {
            if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
                if (((Sign) event.getClickedBlock().getState()).getLine(0).equalsIgnoreCase("[SellAll]")) {
                    if (((Sign) event.getClickedBlock().getState()).getLine(1).equalsIgnoreCase("SomeBlock")) {
                        if (((Sign) event.getClickedBlock().getState()).getLine(2).equalsIgnoreCase("10")) {
                            ((Sign) event.getClickedBlock().getState()).getLine(3);
                            EconomyResponse r = Main.econ.depositPlayer(player.getName(), ((Sign) event.getClickedBlock().getState()).getLine(3).replaceAll("$", "").length());
                            player.sendMessage("You have gained " + ((Sign) event.getClickedBlock().getState()).getLine(3) + " for selling!");
                            return;
 
                        }
                    }
                }
            }
        }
    }
}
 

Can you show me what you mean? Please, Thanks

(First time using Vault/Making a plugin that uses signs)

 

diamondcodes

Look up 'regex' and 'non-numeric regex' and it will give you your answer.
Also rather than using event.getClickedBlock().getState() everytime.
Just store the Block / BlockState. It looks better and it's easier to read.

 

String signLine = "Price: $10";
String price = signLine.split(": $")[1]; //yes, I went there
int price = Integer.parseInt(price);

 

Barinade

Regex is much better and safer in this case.. Slight changes could cause your method to break.

 

That's why you make sure there are no changes lol

 

Barinade

..That's just more pointless code -_-
Regex = 1 line.

 

It's not pointless code, it's code that is going to be done anyways.
You know what it's coded to say so just use that format to your advantage.