Fix player count issue

Author: Cem Yılmaz

SteamQueryNet/SteamQueryNet.Tests/ServerQueryTests.cs

@@ -6,16 +6,15 @@ namespace SteamQueryNet.Tests
 {
     public class ServerQueryTests
     {
-        private const string IP_ADDRESS = "127.0.0.1";
-        private const string HOST_NAME = "localhost";
+        private const string IP_ADDRESS = "209.222.101.220";
         private const int PORT = 27015;
 
         [Theory]
         [InlineData(IP_ADDRESS)]
-        [InlineData(HOST_NAME)]
         public void ShouldInitializeWithProperHost(string host)
         {
             var squery = new ServerQuery(host, PORT);
+            var t = squery.GetServerInfo();
         }
 
         [Theory]

SteamQueryNet/SteamQueryNet/Models/ServerInfo.cs

@@ -39,7 +39,20 @@ public sealed class ServerInfo
         /// <summary>
         /// Number of players on the server.
         /// </summary>
-        public byte Players { get; set; }
+
+        private byte _players;
+        public byte Players
+        {
+            get
+            {
+                // Some servers send bots as players. We don't want that here.
+                return (byte)(this._players - this.Bots);
+            }
+            set
+            {
+                this._players = value;
+            }
+        }
 
         /// <summary>
         /// Maximum number of players the server reports it can hold.

SteamQueryNet/SteamQueryNet/ServerQuery.cs

@@ -284,10 +284,13 @@ private List<TObject> ExtractListData<TObject>(byte[] rawSource)
             var objectList = new List<TObject>();
 
             // Skip the response headers.
-            IEnumerable<byte> dataSource = rawSource.Skip(RESPONSE_HEADER_COUNT);
+            byte player_count = rawSource[RESPONSE_CODE_INDEX];
+
+            // Skip +1 for player_count
+            IEnumerable<byte> dataSource = rawSource.Skip(RESPONSE_HEADER_COUNT + 1);
 
             // Iterate amount of times that the server said.
-            for (byte i = 0; i < rawSource[RESPONSE_CODE_INDEX]; i++)
+            for (byte i = 0; i < player_count; i++)
             {
                 // Activate a new instance of the object.
                 var objectInstance = Activator.CreateInstance<TObject>();
@@ -387,7 +390,7 @@ private IEnumerable<byte> ExtractData<TObject>(TObject objectRef, byte[] dataSou
                         : property.PropertyType;
 
                     // Extract the value and the size from the source.
-                    (object result, int size) = ExtractMarshalType(enumerableSource.SkipWhile(x => x == 0), typeOfProperty);
+                    (object result, int size) = ExtractMarshalType(enumerableSource, typeOfProperty);
 
                     /* If the property is an enum we should parse it first then assign its value,
                      * if not we can just give it to SetValue since it was converted by ExtractMarshalType already.*/