Помогите Упростить код

[ВладМен]

Код:

public class commands implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
      Player p = (Player)sender;
      if(!sender.hasPermission("nyr.use"))
          p.sendMessage("§cУ Вас недостаточно прав!");
      {
          {
      p.sendMessage("§eВы получили новогодний подарок! §a:)");
    if (p.hasPermission("nyr.use"))
      Bukkit.dispatchCommand(Bukkit.getConsoleSender(), "pex user " + p.getName() + " group set boss");
        return true;
}
          }
      }
}

Что нужно вписать чтобы не писать каждый раз if(p.hasPermission("nyr.user"))
?
Не агритесь на меня я новичок :c

 

[ImperatorBuzov]

string

 

[ВладМен]

string

в коде пожалуйста :C

 

[AtomicScience]

У тебя один перм?

А вообще, вставь при успешной проверке на отсутствие права return false;

 

[ВладМен]

У тебя один перм?

А вообще, вставь при успешной проверке на отсутствие права return false;

Да, 1

 

[AtomicScience]

Да, 1

Ну, значит вставь один большой IF на весь метод, и будет тебе счастье

 

[MrTrojan]

public class commands implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        Player p = (Player)sender;

        if(p.hasPermission("nyr.use")) {
            p.sendMessage("§eВы получили новогодний подарок! §a:)");
            Bukkit.dispatchCommand(Bukkit.getConsoleSender(), String.format("pex user %s group set boss", p.getName()));
        } else
            p.sendMessage("§cУ Вас недостаточно прав!");

        return true;
    }
}

 

[_MasterCapeXD_]

    @Override
    public void onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if(!(sender instanceof Player))
        {
            sender.sendMessage("Вы не игрок");
            return true;
        }
        
        Player player = (Player) sender;
        
        if(!player.hasPermission("nur.use"))
        {
            sender.sendMessage("Нет прав!");
            return true;
        }
        
        player.sendMessage("§eВы получили новогодний подарок! §a:)");
        Bukkit.dispatchCommand(Bukkit.getConsoleSender(), String.format("pex user %s group set boss", player.getName()));
        
        return true;
    }

 

[ВладМен]

public class commands implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        Player p = (Player)sender;

        if(p.hasPermission("nyr.use")) {
            p.sendMessage("§eВы получили новогодний подарок! §a:)");
            Bukkit.dispatchCommand(Bukkit.getConsoleSender(), String.format("pex user %s group set boss", p.getName()));
        } else
            p.sendMessage("§cУ Вас недостаточно прав!");

        return true;
    }
}

Спасибо

 

[ВладМен]

    @Override
    public void onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if(!(sender instanceof Player))
        {
            sender.sendMessage("Вы не игрок");
            return true;
        }
       
        Player player = (Player) sender;
       
        if(!player.hasPermission("nur.use"))
        {
            sender.sendMessage("Нет прав!");
            return true;
        }
       
        player.sendMessage("§eВы получили новогодний подарок! §a:)");
        Bukkit.dispatchCommand(Bukkit.getConsoleSender(), String.format("pex user %s group set boss", player.getName()));
       
        return true;
    }

Спасибо за проверку на игрока 🙂