Null checks for AbstractPlayerGroup isLeader method.
This commit is contained in:
parent
13b9f0c499
commit
ec9cdcced0
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -82,7 +82,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -82,7 +82,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -83,7 +83,18 @@ public abstract class AbstractPlayerGroup
|
||||
*/
|
||||
public boolean isLeader(PlayerInstance player)
|
||||
{
|
||||
return getLeader().getObjectId() == player.getObjectId();
|
||||
if (player == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
final PlayerInstance leader = getLeader();
|
||||
if (leader == null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
return leader.getObjectId() == player.getObjectId();
|
||||
}
|
||||
|
||||
/**
|
||||
|
Loading…
Reference in New Issue
Block a user