Skip to content

Commit ecd6f2d

Browse files
rxturrxtur
authored
Merge pull request BlogEngine#108 from ThisWillDoIt/master
Improved some LINQ expressions
2 parents 9bccf0e + 166a4ca commit ecd6f2d

30 files changed

+66
-68
lines changed

‎BlogEngine/BlogEngine.Core/Data/BlogRepository.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public Models.Blog FindById(Guid id)
5252
if (!(Blog.CurrentInstance.IsPrimary && Security.IsAdministrator))
5353
throw new UnauthorizedAccessException();
5454

55-
var blog = Blog.Blogs.Where(b => b.Id == id).FirstOrDefault();
55+
var blog = Blog.Blogs.FirstOrDefault(b => b.Id == id);
5656
return ToJson(blog);
5757
}
5858

@@ -92,7 +92,7 @@ public bool Update(Models.Blog blog)
9292
throw new UnauthorizedAccessException();
9393
try
9494
{
95-
var coreBlog = Blog.Blogs.Where(b => b.Id == blog.Id).FirstOrDefault();
95+
var coreBlog = Blog.Blogs.FirstOrDefault(b => b.Id == blog.Id);
9696
return Save(coreBlog, blog);
9797
}
9898
catch (Exception)
@@ -113,7 +113,7 @@ public bool Remove(Guid id)
113113
throw new UnauthorizedAccessException();
114114
try
115115
{
116-
var blog = Blog.Blogs.Where(b => b.Id == id).FirstOrDefault();
116+
var blog = Blog.Blogs.FirstOrDefault(b => b.Id == id);
117117
blog.Delete();
118118
blog.Save();
119119
return true;

‎BlogEngine/BlogEngine.Core/Data/CategoryRepository.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public IEnumerable<CategoryItem> Find(int take = 10, int skip = 0, string filter
4242
// add categories without posts
4343
foreach (var c in Category.Categories)
4444
{
45-
var x = items.Where(i => i.Id == c.Id).FirstOrDefault();
45+
var x = items.FirstOrDefault(i => i.Id == c.Id);
4646
if (x == null)
4747
items.Add(new CategoryItem { Id = c.Id, Parent = OptionById(c.Parent), Title = c.Title, Description = c.Description, Count = 0 });
4848
}
@@ -87,11 +87,11 @@ public CategoryItem FindById(Guid id)
8787
// add categories without posts
8888
foreach (var c in Category.Categories)
8989
{
90-
var x = items.Where(i => i.Id == c.Id).FirstOrDefault();
90+
var x = items.FirstOrDefault(i => i.Id == c.Id);
9191
if (x == null)
9292
items.Add(new CategoryItem { Id = c.Id, Parent = OptionById(c.Parent), Title = c.Title, Description = c.Description, Count = 0 });
9393
}
94-
return items.Where(c => c.Id == id).FirstOrDefault();
94+
return items.FirstOrDefault(c => c.Id == id);
9595
}
9696
/// <summary>
9797
/// Add new item

‎BlogEngine/BlogEngine.Core/Data/CommentsRepository.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public CommentItem Add(CommentDetail item)
8282
var c = new Comment();
8383
try
8484
{
85-
var post = Post.Posts.Where(p => p.Id == item.PostId).FirstOrDefault();
85+
var post = Post.Posts.FirstOrDefault(p => p.Id == item.PostId);
8686

8787
c.Id = Guid.NewGuid();
8888
c.ParentId = item.ParentId;
@@ -104,7 +104,7 @@ public CommentItem Add(CommentDetail item)
104104
post.AddComment(c);
105105
post.Save();
106106

107-
var newComm = post.Comments.Where(cm => cm.Content == c.Content).FirstOrDefault();
107+
var newComm = post.Comments.FirstOrDefault(cm => cm.Content == c.Content);
108108
return Json.GetComment(newComm, post.Comments);
109109
}
110110
catch (Exception ex)

‎BlogEngine/BlogEngine.Core/Data/CustomFieldRepository.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public CustomField FindById(string type, string id, string key)
4949
if (!Security.IsAuthorizedTo(Rights.AccessAdminPages))
5050
throw new UnauthorizedAccessException();
5151

52-
var cf = Find("").Where(f => f.CustomType == type && f.ObjectId == id && f.Key == key).FirstOrDefault();
52+
var cf = Find("").FirstOrDefault(f => f.CustomType == type && f.ObjectId == id && f.Key == key);
5353
return cf;
5454
}
5555

@@ -157,10 +157,10 @@ public void ClearCustomFields(string type, string id)
157157

158158
bool AlreadyExists(CustomField item)
159159
{
160-
var field = CustomFieldsParser.CachedFields.Where(f => f.BlogId == item.BlogId
160+
var field = CustomFieldsParser.CachedFields.FirstOrDefault(f => f.BlogId == item.BlogId
161161
&& f.CustomType == item.CustomType
162162
&& f.ObjectId == item.ObjectId
163-
&& f.Key == item.Key).FirstOrDefault();
163+
&& f.Key == item.Key);
164164
return field != null;
165165
}
166166

‎BlogEngine/BlogEngine.Core/Data/PageRepository.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,8 +189,8 @@ static string GetUniqueSlug(string slug)
189189

190190
private static bool IsUniqueSlug(string slug)
191191
{
192-
return Page.Pages.Where(p => p.Slug != null && p.Slug.ToLower() == slug.ToLower())
193-
.FirstOrDefault() == null ? true : false;
192+
return Page.Pages
193+
.FirstOrDefault(p => p.Slug != null && p.Slug.ToLower() == slug.ToLower()) == null ? true : false;
194194
}
195195

196196
// if description not set, use first 100 chars in the post

‎BlogEngine/BlogEngine.Core/Data/PostRepository.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ static void UpdatePostCategories(Post post, List<CategoryItem> categories)
172172
foreach (var cat in categories)
173173
{
174174
// add if category does not exist
175-
var existingCat = Category.Categories.Where(c => c.Title == cat.Title).FirstOrDefault();
175+
var existingCat = Category.Categories.FirstOrDefault(c => c.Title == cat.Title);
176176
if (existingCat == null)
177177
{
178178
var repo = new CategoryRepository();
@@ -208,7 +208,7 @@ static List<TagItem> FilterTags(List<TagItem> tags)
208208

209209
foreach (var t in tags)
210210
{
211-
if (!uniqueTags.Any(u => u.TagName == t.TagName))
211+
if (uniqueTags.All(u => u.TagName != t.TagName))
212212
{
213213
uniqueTags.Add(t);
214214
}
@@ -233,8 +233,8 @@ static string GetUniqueSlug(string slug)
233233

234234
static bool IsUniqueSlug(string slug)
235235
{
236-
return Post.ApplicablePosts.Where(p => p.Slug != null && p.Slug.ToLower() == slug.ToLower())
237-
.FirstOrDefault() == null ? true : false;
236+
return Post.ApplicablePosts
237+
.FirstOrDefault(p => p.Slug != null && p.Slug.ToLower() == slug.ToLower()) == null ? true : false;
238238
}
239239

240240
// if description not set, use first 100 chars in the post

‎BlogEngine/BlogEngine.Core/Data/RolesRepository.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public RoleItem FindById(string id)
5353
var roles = new List<Data.Models.RoleItem>();
5454
roles.AddRange(System.Web.Security.Roles.GetAllRoles().Select(r => new Data.Models.RoleItem { RoleName = r, IsSystemRole = Security.IsSystemRole(r) }));
5555

56-
return roles.Where(r => r.RoleName.ToLower() == id.ToLower()).FirstOrDefault();
56+
return roles.FirstOrDefault(r => r.RoleName.ToLower() == id.ToLower());
5757
}
5858

5959
/// <summary>
@@ -101,7 +101,7 @@ public bool Update(RoleItem role, string oldRole)
101101
if (!Security.IsAuthorizedTo(Rights.EditRoles))
102102
throw new System.UnauthorizedAccessException();
103103

104-
var updateRole = System.Web.Security.Roles.GetAllRoles().Where(r => r.ToString() == oldRole).FirstOrDefault();
104+
var updateRole = Roles.GetAllRoles().FirstOrDefault(r => r.ToString() == oldRole);
105105

106106
if (updateRole == null)
107107
throw new ApplicationException("Role not found");
@@ -172,7 +172,7 @@ public IEnumerable<Group> GetRoleRights(string role)
172172

173173
var category = rightDetails == null ? RightCategory.General : rightDetails.Category;
174174

175-
var group = groups.Where(g => g.Title == category.ToString()).FirstOrDefault();
175+
var group = groups.FirstOrDefault(g => g.Title == category.ToString());
176176

177177
var prm = new Permission();
178178
var rt = Right.GetRightByName(right.ToString());

‎BlogEngine/BlogEngine.Core/Data/Services/CustomFieldsParser.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,17 +117,17 @@ static string ReplaceCustomFields(string html)
117117

118118
if (s[1] == "POST")
119119
{
120-
var cf = postFields.Where(f => f.Key.ToLower() == key.ToLower()
121-
&& f.ObjectId.ToLower() == id.ToLower()).FirstOrDefault();
120+
var cf = postFields.FirstOrDefault(f => f.Key.ToLower() == key.ToLower()
121+
&& f.ObjectId.ToLower() == id.ToLower());
122122

123123
if (cf != null)
124124
val = cf.Value;
125125
}
126126

127127
if (s[1] == "THEME")
128128
{
129-
var cf = themeFields.Where(f => f.Key.ToLower() == key.ToLower()
130-
&& f.ObjectId.ToLower() == id.ToLower()).FirstOrDefault();
129+
var cf = themeFields.FirstOrDefault(f => f.Key.ToLower() == key.ToLower()
130+
&& f.ObjectId.ToLower() == id.ToLower());
131131

132132
if (cf != null)
133133
val = cf.Value;

‎BlogEngine/BlogEngine.Core/Data/Services/Json.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ public static CommentItem GetComment(Comment c, List<Comment> postComments)
141141
jc.Title = c.Teaser.Length < 80 ? c.Teaser : c.Teaser.Substring(0, 80) + "...";
142142
jc.DateCreated = c.DateCreated.ToString("yyyy-MM-dd HH:mm", CultureInfo.InvariantCulture);
143143
jc.RelativeLink = c.RelativeLink;
144-
jc.HasChildren = postComments.Where(pc => pc.ParentId == c.Id).FirstOrDefault() != null;
144+
jc.HasChildren = postComments.FirstOrDefault(pc => pc.ParentId == c.Id) != null;
145145
jc.Avatar = Gravatar(c);
146146
return jc;
147147
}
@@ -223,7 +223,7 @@ static SelectOption ItemParent(Guid? id)
223223
if (id == null || id == Guid.Empty)
224224
return null;
225225

226-
var item = Category.Categories.Where(c => c.Id == id).FirstOrDefault();
226+
var item = Category.Categories.FirstOrDefault(c => c.Id == id);
227227
return new SelectOption { OptionName = item.Title, OptionValue = item.Id.ToString() };
228228
}
229229

‎BlogEngine/BlogEngine.Core/Data/StatsRepository.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@ public Stats Get()
2626
if (!Security.IsAuthorizedTo(Rights.EditOtherUsersPosts))
2727
postList = postList.Where(p => p.Author.ToLower() == Security.CurrentUser.Identity.Name.ToLower()).ToList();
2828

29-
stats.PublishedPostsCount = postList.Where(p => p.IsPublished == true).Count();
30-
stats.DraftPostsCount = postList.Where(p => p.IsPublished == false).Count();
29+
stats.PublishedPostsCount = postList.Count(p => p.IsPublished == true);
30+
stats.DraftPostsCount = postList.Count(p => p.IsPublished == false);
3131

32-
stats.PublishedPagesCount = Page.Pages.Where(p => p.IsPublished == true && p.IsDeleted == false).Count();
33-
stats.DraftPagesCount = Page.Pages.Where(p => p.IsPublished == false && p.IsDeleted == false).Count();
32+
stats.PublishedPagesCount = Page.Pages.Count(p => p.IsPublished == true && p.IsDeleted == false);
33+
stats.DraftPagesCount = Page.Pages.Count(p => p.IsPublished == false && p.IsDeleted == false);
3434

3535
CountComments(stats);
3636

@@ -70,9 +70,9 @@ void CountComments(Stats stats)
7070
if (post.Author.ToLower() != Security.CurrentUser.Identity.Name.ToLower())
7171
continue;
7272

73-
stats.PublishedCommentsCount += post.Comments.Where(c => c.IsPublished == true && c.IsDeleted == false).Count();
74-
stats.UnapprovedCommentsCount += post.Comments.Where(c => c.IsPublished == false && c.IsSpam == false && c.IsDeleted == false).Count();
75-
stats.SpamCommentsCount += post.Comments.Where(c => c.IsPublished == false && c.IsSpam == true && c.IsDeleted == false).Count();
73+
stats.PublishedCommentsCount += post.Comments.Count(c => c.IsPublished == true && c.IsDeleted == false);
74+
stats.UnapprovedCommentsCount += post.Comments.Count(c => c.IsPublished == false && c.IsSpam == false && c.IsDeleted == false);
75+
stats.SpamCommentsCount += post.Comments.Count(c => c.IsPublished == false && c.IsSpam == true && c.IsDeleted == false);
7676
}
7777
}
7878

0 commit comments

Comments
 (0)