Remove redundant check to string value
continuous-integration/drone/push Build is passing Details

pull/48/head
Alexis Drai 3 years ago
parent 2bf7918a6f
commit 9e4b92f560

@ -13,7 +13,7 @@ namespace Model
} }
internal set internal set
{ {
if (!String.IsNullOrWhiteSpace(value) && !String.IsNullOrEmpty(value)) if (!String.IsNullOrWhiteSpace(value))
{ {
name = value; name = value;
} }

@ -55,7 +55,7 @@ namespace Model
/// <returns>player with said name</returns> /// <returns>player with said name</returns>
public Player GetOneByName(string name) public Player GetOneByName(string name)
{ {
if (!String.IsNullOrWhiteSpace(name) && !String.IsNullOrEmpty(name)) if (!String.IsNullOrWhiteSpace(name))
{ {
Player result = players.FirstOrDefault(p => p.Name.Equals(name)); Player result = players.FirstOrDefault(p => p.Name.Equals(name));
return new Player(result); // THIS IS A COPY (using a copy constructor) return new Player(result); // THIS IS A COPY (using a copy constructor)

Loading…
Cancel
Save