for each if break

Question

I need this code to not add the draft document to the list if the loged in user is not the documents author. Should it work? var listofdocuments = dbContext.Documents.ToList(); var filtered = new List () ; foreach (var document in listofdocuments) { if (document.IsDraft) { if (document.Author == User.FindFirst(ClaimTypes.NameIdentifier).Value) { filtered.Add(document); } else break ; } filtered.Add(document); }

Sachin Singh · Answer

don't create additional list when it is not that necessary, var loogedInUser=User.FindFirst(ClaimTypes.NameIdentifier).Value; var listofdocuments = dbContext.Documents.where(x=>x.IsDraft== true && x.Author==loogedInUser).ToList();

Jaimin Shethiya · Answer

Hello Steven, can you please try with below code. var filtered = new List (); var loggedInUser = User.FindFirst(ClaimTypes.NameIdentifier); if(loggedInUser != null) { filtered = dbContext.Documents.Where(x=> x.IsDraft == true && x.Author.Equals(loggedInUser.Value, StringComparison.InvariantCultureIgnoreCase)).ToList(); } let me know if you have any other doubts so will definetly help you. Thanks

Satya Karki · Answer

Simple way var filtered = new List () ; var loggedInUser= User.FindFirst(ClaimTypes.NameIdentifier).Value; if (loggedInUser != null ) filtered = dbContext.Documents.where(x=> x.IsDraft == true && x.Author == loggedInUser).toList()

Rijwan Ansari · Answer

Better way to do in this way. var filtered = new List () ; var currentUser = User.FindFirst(ClaimTypes.NameIdentifier).Value; if (currentUser != null ) filtered = dbContext.Documents.where(x=> x.IsDraft == true && x.Author == currentUser).toList()