Removed some excess looping

12 years ago · 016590529f
parent af7744a1fd
commit 016590529f
1 changed files with 10 additions and 6 deletions
--- a/MediaBrowser.Controller/Entities/Folder.cs
+++ b/MediaBrowser.Controller/Entities/Folder.cs
@ -218,15 +218,14 @@ namespace MediaBrowser.Controller.Entities
        {
            itemSet = itemSet.Where(i => !(i.IsFolder));

-            if (!itemSet.Any())
-            {
-                return 0;
-            }
-
            decimal totalPercent = 0;

+            int count = 0;
+
            foreach (BaseItem item in itemSet)
            {
+                count++;
+                
                UserItemData data = item.GetUserData(user, false);

                if (data == null)
@ -246,7 +245,12 @@ namespace MediaBrowser.Controller.Entities
                }
            }

-            return totalPercent / itemSet.Count();
+            if (count == 0)
+            {
+                return 0;
+            }
+
+            return totalPercent / count;
        }

        /// <summary>