fix(Reward): Simplified and better comments

Assets/Mirror/Examples/Room/Scripts/Reward.cs

@@ -24,34 +24,29 @@ protected override void OnValidate()
         [ServerCallback]
         void OnTriggerEnter(Collider other)
         {
-            if (other.gameObject.CompareTag("Player"))
-                ClaimPrize(other.gameObject);
-        }
-
-        [ServerCallback]
-        void ClaimPrize(GameObject player)
-        {
-            if (available)
-            {
-                // This is a fast switch to prevent two players claiming the prize in a bang-bang close contest for it.
-                // First hit turns it off, pending the object being destroyed a few frames later.
+            // Set up physics layers to prevent this from being called by non-players
+            // and eliminate the need for a tag check here.
+            if (!other.CompareTag("Player")) return;
+
+            // This is a fast switch to prevent two players claiming the prize in a bang-bang close contest for it.
+            // First to trigger turns it off, pending the object being destroyed a few frames later.
+            if (!available)
+                return;
+            else
                 available = false;
 
-                Color32 color = randomColor.color;
-
-                // calculate the points from the color ... lighter scores higher as the average approaches 255
-                // UnityEngine.Color RGB values are float fractions of 255
-                uint points = (uint)(((color.r) + (color.g) + (color.b)) / 3);
+            // Calculate the points from the color...lighter scores higher as the average approaches 255
+            // UnityEngine.Color RGB values are byte 0 to 255
+            uint points = (uint)((randomColor.color.r + randomColor.color.g + randomColor.color.b) / 3);
 
-                // award the points via SyncVar on the PlayerController
-                player.GetComponent<PlayerScore>().score += points;
+            // award the points via SyncVar on Player's PlayerScore
+            other.GetComponent<PlayerScore>().score += points;
 
-                // spawn a replacement
-                Spawner.SpawnReward();
+            // spawn a replacement
+            Spawner.SpawnReward();
 
-                // destroy this one
-                NetworkServer.Destroy(gameObject);
-            }
+            // destroy this one
+            NetworkServer.Destroy(gameObject);
         }
     }
 }