diff options
| author | LukePulverenti Luke Pulverenti luke pulverenti <LukePulverenti Luke Pulverenti luke.pulverenti@gmail.com> | 2012-09-11 10:35:04 -0400 |
|---|---|---|
| committer | LukePulverenti Luke Pulverenti luke pulverenti <LukePulverenti Luke Pulverenti luke.pulverenti@gmail.com> | 2012-09-11 10:35:04 -0400 |
| commit | 016590529f1dde1b2bcfa6c64aa07ea866f26b51 (patch) | |
| tree | 3a7108f80c474f3b29f5d3c96114318448c7335b | |
| parent | af7744a1fde91784937fb8c00d1c11868b968f0f (diff) | |
Removed some excess looping
| -rw-r--r-- | MediaBrowser.Controller/Entities/Folder.cs | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/MediaBrowser.Controller/Entities/Folder.cs b/MediaBrowser.Controller/Entities/Folder.cs index a9c92c1fa..c0f1604b1 100644 --- 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>
|