Sign multiply

Discussion in 'Plugin Development' started by DevManABCD, Aug 25, 2014.

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

    DevManABCD

    Code:
        @EventHandler
        public void onPlayerInteract(PlayerInteractEvent e) {
            Player p = (Player) e.getPlayer();
            if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
                if (e.getClickedBlock().getState() instanceof Sign) {
                    Sign s = (Sign) e.getClickedBlock().getState();
                    for(Signs sign : SignManager.getSigns()) {
                        if(s.getLine(0).equalsIgnoreCase(sign.getLines().get(0))) {
                            if(s.getLine(1).equalsIgnoreCase(sign.getLines().get(1))) {
                                if(s.getLine(2).equalsIgnoreCase(sign.getLines().get(2))) {
                                    if(s.getLine(3).equalsIgnoreCase(sign.getLines().get(3))) {
                                        for(String ln : sign.getLines()) {
                                            p.sendMessage("" + sign.getAdd().getType());
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    Someone knows why when i click on sign i receive 4 messages no one?
     
  2. Offline

    Konkz

    Because

    PHP:
    for(String ln sign.getLines()) {
                                            
    p.sendMessage("" sign.getAdd().getType());
                                        }
    What this does is for every line on the sign (there are four lines) it sends a message.
    4 lines = 4 messages.
     
  3. Offline

    DevManABCD

    How can i fix it?
     
  4. Offline

    TheMcScavenger

    Create your string in the for loop (if you want to include all the lines), and then send the message.
     
  5. Offline

    Konkz

    I can't work out what you are even trying to do to be honest...
     
Thread Status:
Not open for further replies.

Share This Page