Added comments to some functions, renamed GetClientByName to GetClientByNameOrId

Server/Services/ChatService.cs

@@ -37,6 +37,11 @@ namespace reallife_gamemode.Server.Services
             player.SendChatMessage("~r~[FEHLER]~s~ Die Aktion wurde nicht ausgeführt.");
         }
 
+        /// <summary>
+        /// Sendet eine Nachricht an eine Liste von Fraktionen
+        /// </summary>
+        /// <param name="message">Die Nachricht, die gesendet werden soll</param>
+        /// <param name="factions">Die Liste an Fraktionen, die diese Nachricht bekommen sollen</param>
         public static void BroadcastFaction(string message, List<Faction> factions)
         {
             foreach (Client c in NAPI.Pools.GetAllPlayers())
@@ -52,11 +57,21 @@ namespace reallife_gamemode.Server.Services
             }
         }
 
+        /// <summary>
+        /// Sendet eine Nachricht an eine Fraktion
+        /// </summary>
+        /// <param name="message">Die Nachricht, die gesendet werden soll</param>
+        /// <param name="f">Die Fraktion, die diese Nachricht bekommen soll</param>
         public static void BroadcastFaction(string message, Faction f)
         {
             BroadcastFaction(message, new List<Faction>() { f });
         }
 
+        /// <summary>
+        /// Sendet eine Nachricht an alle Spieler mit einem bestimmten Admin Level
+        /// </summary>
+        /// <param name="message">Die Nachricht, die gesendet werden soll</param>
+        /// <param name="minLevel">Das mindest Admin Level, das für das Erhalten dieser Nachricht benötigt wird</param>
         public static void BroadcastAdmin(string message, AdminLevel minLevel)
         {
             NAPI.Pools.GetAllPlayers().ForEach(p =>

Server/Services/ClientService.cs

@@ -13,29 +13,29 @@ namespace reallife_gamemode.Server.Services
 {
     class ClientService
     {
-        public static Client GetClientByName(string name)
+        /// <summary>
+        /// Gibt einen Client anhand seines Namens oder der ID zurück
+        /// </summary>
+        /// <param name="nameOrId">Die ID oder der Name, nach dem gesucht werden soll</param>
+        /// <returns></returns>
+        public static Client GetClientByNameOrId(string nameOrId)
         {
-            /* Funktionsaufbau: Prüft erst, ob ein Spieler mit exakt diesem Namen online ist
-             * Wenn Ja: Wird dieser zurückgegeben
-             * Wenn Nein: Wird der erste Spieler zurückgegeben, dessen Namen mit dem eingegebenen Parameter übereinstimmt
-             * Gibt "null" zurück, falls kein Client gefunden wurde
-             */
             Client toReturn = null;
-            name = name.ToLower();
+            nameOrId = nameOrId.ToLower();
 
             List<Client> playerList = NAPI.Pools.GetAllPlayers();
 
-            if(int.TryParse(name, out int id))
+            if(int.TryParse(nameOrId, out int id))
             {
                 toReturn = playerList.Find(p => p.Handle.Value == id);
                 return toReturn;
             }
             
-            toReturn = playerList.Find(p => p.Name.ToLower() == name);
+            toReturn = playerList.Find(p => p.Name.ToLower() == nameOrId);
 
             if(toReturn == null)
             {
-                toReturn = playerList.Find(p => p.Name.ToLower().StartsWith(name));
+                toReturn = playerList.Find(p => p.Name.ToLower().StartsWith(nameOrId));
             }
 
             return toReturn;