‫۱۱ اشتباه رایج در LINQ که حتی توسعه‌دهندگان باتجربه #C مرتکب می‌شوند (و راه‌حل آن‌ها)

// ❌ BAD
var result = GetUsers().Where(u => u.IsActive).ToList();
if (result.Any()) { /* عملیات دیگر */ }

// ✅ GOOD
var activeUsers = GetUsers().Where(u => u.IsActive);
if (activeUsers.Any()) { /* عملیات دیگر */ }

// ❌ BAD
if (orders.Count() > 0) { /* عملیات */ }

// ✅ GOOD
if (orders.Any()) { /* عملیات */ }

// ❌ BAD
foreach (var user in users.Where(u => u.IsBlocked))
    users.Remove(user); // تغییر منبع در حین پیمایش!

// ✅ BETTER
var toRemove = users.Where(u => u.IsBlocked).ToList(); // مادی‌سازی عناصر برای حذف
foreach (var u in toRemove)
      users.Remove(u);

// ❌ BAD
var query = GetItems().Where(x => x.IsReady);
Log(query.Count()); // پیمایش اول
return query.First(); // پیمایش دوم

// ✅ GOOD
var materializedItems = GetItems().Where(x => x.IsReady).ToList(); // مادی‌سازی تنها یک بار
Log(materializedItems.Count);
return materializedItems.First();

// ❌ BAD: فکر می‌کنید این کوئری اجرا شده است
var result = GetItems().Where(x => x.IsArchived);
// در اینجا هیچ عملیاتی روی داده‌ها انجام نشده است!

// ✅ GOOD
var materialized = result.ToList(); // این خط کوئری را مجبور به اجرا می‌کند
// حالا می‌توانید روی materialized کار کنید

// ❌ BAD
var result = users.Where(u => u.Orders.Any(o => o.Total > 100))
                  .Select(u => new {
                      u.Name,
                      Orders = u.Orders.Where(o => o.Total > 100)
                  });

// ✅ GOOD (تقسیم به مراحل)
var usersWithHighValueOrders = users.Where(u => u.Orders.Any(o => o.Total > 100));
var finalResult = usersWithHighValueOrders.Select(u => new {
    u.Name,
    HighValueOrders = u.Orders.Where(o => o.Total > 100)
});

// ✅ GOOD (استفاده از Query Syntax)
var resultQuerySyntax = from u in users
                        where u.Orders.Any(o => o.Total > 100)
                        select new {
                            u.Name,
                            Orders = u.Orders.Where(o => o.Total > 100)
                        };

// ❌ BAD: تمام کاربران از دیتابیس به حافظه کشیده می‌شوند، سپس فیلتر می‌شوند
db.Users.ToList().Where(u => u.IsActive);

// ✅ GOOD: فیلترینگ در دیتابیس انجام شده و تنها کاربران فعال به حافظه کشیده می‌شوند
db.Users.Where(u => u.IsActive).ToList();

// ❌ BAD
var result = users.First(); // اگر users خالی باشد، خطا می‌دهد
Console.WriteLine(result.Name);

// ✅ GOOD
var result = users.FirstOrDefault(); // اگر users خالی باشد، null برمی‌گرداند
if (result != null)
    Console.WriteLine(result.Name);
else
    Console.WriteLine("کاربری یافت نشد.");

// ❌ BAD: ایجاد یک مجموعه میانی غیرضروری
var active = users.Where(u => u.IsActive).First();

// ✅ GOOD: فیلترینگ مستقیم و بهینه
var active = users.First(u => u.IsActive);

// ❌ BAD
var results = bigList.Select(DoExpensiveWork).ToList(); // عملیات سریالی

// ✅ GOOD
var results = bigList.AsParallel().Select(DoExpensiveWork).ToList(); // عملیات موازی

// ❌ BAD
var anon = users.Select(u => new { u.Id, u.Name }).ToList();
// بعداً نیاز به آبجکت کامل کاربر است که با این نوع بی‌نام قابل دسترسی نیست.

// ✅ GOOD
public class UserDto { public int Id { get; set; } public string Name { get; set; } }
var projected = users.Select(u => new UserDto { Id = u.Id, Name = u.Name }).ToList();