如何动态构建LINQ Where语句？解决用户ID为空时的查询异常问题

首先来说说你遇到的两个核心问题：

1. 初始代码里直接用Convert.ToInt32(txtUserId.Text)，当输入为空时会抛出FormatException，因为空字符串无法转成整数；
2. 你修改后的逻辑用了string.IsNullOrEmpty(txtUserId.Text) || w.ScreenName == dbscreenanme.Text，这会导致当用户ID为空时，整个条件始终为true，所有数据都会被返回，完全没起到筛选ScreenName的作用。

根据你说的“用户ID为空时仅显示对应provider的内容”，我猜你的真实需求应该是：

• 当用户ID不为空且合法时，筛选出同时满足Code匹配用户ID + ScreenName匹配指定值的记录（如果是“或”的关系可以随时调整逻辑）；
• 当用户ID为空时，只筛选出ScreenName匹配指定值的记录。

下面给你两种靠谱的解决方案：

方案一：分步拼接查询（可读性最高） #

这种方式可以逐步添加条件，逻辑清晰，也方便后续扩展更多筛选规则：

// 先获取基础数据集
var screenQuery = _security.GetScreenermissionsForSearch().AsQueryable();

// 处理用户ID筛选条件：只有当输入不为空且能转成整数时才添加
if (!string.IsNullOrEmpty(txtUserId.Text) && int.TryParse(txtUserId.Text, out int userId))
{
    screenQuery = screenQuery.Where(w => w.Code == userId);
}

// 始终添加ScreenName的筛选条件（符合你“用户ID为空时仅显示对应provider内容”的需求）
screenQuery = screenQuery.Where(w => w.ScreenName == dbscreenanme.Text);

// 绑定到控件
gridControl1.DataSource = screenQuery.ToList();
this.gridView1.Columns[0].Width = 50;
this.gridView1.Columns[1].Width = 100;

方案二：单Where子句内写完整逻辑（更紧凑） #

如果不想分步拼接，可以在Where里把所有逻辑写清楚，同时用int.TryParse避免转换异常：

var screenData = _security.GetScreenermissionsForSearch();
// 提前判断用户ID是否有效
bool hasValidUserId = int.TryParse(txtUserId.Text, out int userId);

gridControl1.DataSource = screenData.Where(w => 
    // 必须满足ScreenName匹配
    w.ScreenName == dbscreenanme.Text 
    // 同时：如果有有效用户ID，还要满足Code匹配；没有的话就跳过该条件
    && (!hasValidUserId || w.Code == userId)
).ToList();

this.gridView1.Columns[0].Width = 50;
this.gridView1.Columns[1].Width = 100;

额外说明 #

• 为什么不用Convert.ToInt32？因为它在处理空字符串、非数字字符串时会直接抛出异常，而int.TryParse会安全返回false，不会导致程序崩溃；
• 如果你的需求是“用户ID不为空时，筛选Code匹配或ScreenName匹配”，只需要把方案里的&&改成||就行，调整逻辑很灵活；
• 要是需要处理用户输入非数字的情况（比如输入了字母），可以在int.TryParse返回false时给用户一个提示，提升使用体验。

内容的提问来源于stack exchange，提问作者dotnetdevcsharp