Why will the code not run after this?

Why will the code not run after the getting the string if statement?

Code:

    @EventHandler
    public void onPlayerChallenge(EntityDamageByEntityEvent evt) {
       
        Entity target = evt.getEntity();
        Entity selector = evt.getDamager();
        if(selector instanceof Player) {
            if(target instanceof Player) {
        Player t = (Player) target;
        Player s = (Player) selector;
        s.sendMessage(ChatColor.GREEN + "Challenged player to a match!");
        t.sendMessage(ChatColor.AQUA + "" + s.getName() + ChatColor.GREEN + " " + ChatColor.GREEN + "has challenged you to a match!");
        config.set("" + s.getName(), "" + t.getName());
        saveConfig();
        if(config.getString(t.getName()) == s.getName()) {

 

Use .equals() or .equalsIgnoreCase() to compare strings.

 

How do I check if it is equal to the player tho?

 

Check if it's instance of a player.

 

Still won't work.
LaxWasHere
chasechocolate
Here is what I have:

Code:

    @EventHandler
    public void onPlayerChallenge(EntityDamageByEntityEvent evt) {
     
        Entity target = evt.getEntity();
        Entity selector = evt.getDamager();
     
        if(selector instanceof Player) {
            if(target instanceof Player) {
        Player t = (Player) target;
        Player s = (Player) selector;
        if(Spawn.contains(s)) {
         
        } else {
            evt.setCancelled(true);
        }
     
        s.sendMessage(ChatColor.GREEN + "Challenged player to a match!");
        t.sendMessage(ChatColor.AQUA + "" + s.getName() + ChatColor.GREEN + " " + ChatColor.GREEN + "has challenged you to a match!");
        config.set(s.getName(),t.getName());
        saveConfig();
        if(config.equals(t.getName() == s.getName())) {

 

Code:java

if(config.getString(t.getName()).equals(s.getName())) {

 

Well I fixed that part but now I made it so that it would check what the item in hand is and if it was a blaze rod it would run the code but it wont... Why not?

Code:java

@EventHandler
public void onPlayerChallenge(EntityDamageByEntityEvent evt) {
 
Entity target = evt.getEntity();
Entity selector = evt.getDamager();
 
if(selector instanceof Player) {
 
ItemStack Blaze = new ItemStack(Material.BLAZE_ROD);
if(((Player) selector).getItemInHand() == Blaze) {
if(target instanceof Player) {
Player t = (Player) target;
Player s = (Player) selector;
if(Spawn.contains(s)) {
 
} else {
evt.setCancelled(true);
}
 
s.sendMessage(ChatColor.GREEN + "Challenged player to a match!");
t.sendMessage(ChatColor.AQUA + "" + s.getName() + ChatColor.GREEN + " " + ChatColor.GREEN + "has challenged you to a match!");
config.set(s.getName(),t.getName());
saveConfig();

FurmigaHumana LaxWasHere chasechocolate

Yah can someone please tell me why it's not working? Thanks

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

The same reason as last time. you can't use == to compare two objects. it will only be true if the variables point to the exact same instance in memory. (in your case they don't, ever).

 

Change this:

Code:java

ItemStack Blaze = new ItemStack(Material.BLAZE_ROD);
if(((Player) selector).getItemInHand() == Blaze) {

To this:

Code:

if(((Player) selector).getItemInHand().getType() == Material.BLAZE_ROD) {