diff --git a/Sources/Model/Player.cs b/Sources/Model/Player.cs index e5d2347..85464d4 100644 --- a/Sources/Model/Player.cs +++ b/Sources/Model/Player.cs @@ -13,7 +13,7 @@ namespace Model } internal set { - if (!String.IsNullOrWhiteSpace(value) && !value.Equals("")) + if (String.IsNullOrWhiteSpace(value) && !String.IsNullOrEmpty(value)) { name = value; } diff --git a/Sources/Model/PlayerManager.cs b/Sources/Model/PlayerManager.cs index bfe1b16..ceac71d 100644 --- a/Sources/Model/PlayerManager.cs +++ b/Sources/Model/PlayerManager.cs @@ -55,7 +55,7 @@ namespace Model /// player with said name public Player GetOneByName(string name) { - if (!String.IsNullOrWhiteSpace(name) && !name.Equals("")) + if (!String.IsNullOrWhiteSpace(name) && !String.IsNullOrEmpty(name)) { Player result = players.FirstOrDefault(p => p.Name.Equals(name)); return new Player(result); // THIS IS A COPY (using a copy constructor)