ASP.NET MVC按DateTime字段搜索返回数据与预期不符问题排查

提问 #

我开发了一个按Date类型字段（如DateTo、DateFrom）实现检索的功能，在Model中定义了DateTime类型的日期属性，代码如下：

[Required]
[Display(Name = "Date and Time")]
[DisplayFormat(DataFormatString = "{0:yyyy-MM-dd}", ApplyFormatInEditMode = true)]
[DataType(DataType.Date)]
public DateTime DateAndTime { get; set; }

随后我在Controller中编写了用于搜索的Action方法，代码如下：

public IActionResult Index(string search)
{
    IEnumerable<AdmissionPacients> admissionPatient = _db.AdmissionPacients
        .Include(u => u.Patient)
        .Include(d => d.Doctor);

    if(!string.IsNullOrEmpty(search))
    {
        if(DateTime.TryParse(search,out var dateTime))
        {
            admissionPatient = admissionPatient.Where
                (x => x.DateAndTime.ToShortDateString() == dateTime.ToShortDateString()).ToList();
        }
    }

    return View(admissionPatient);
}

当前存在的异常表现为：当我输入类似1/22/2021-01/31/2021的日期条件时，无法得到预期的返回结果：预期应返回3条匹配记录，实际仅返回2条。
我已经尝试过添加.FirstOrDefault()、使用Select()等多种方式排查，问题仍未解决，请问该问题的根因是什么，该如何修复？

回答 #

根因 #

1. 没有实现范围输入的解析逻辑
   你现在的代码是把整个search字符串直接丢给DateTime.TryParse处理，但1/22/2021-01/31/2021这种带连接符的范围字符串本身就不是合法的单日期格式，要么直接解析失败导致筛选逻辑不生效，要么在宽松解析规则下被错误识别成区间里的某一个单日期。你现在遇到的少返回1条的情况，大概率是后者：范围字符串被错误解析成了区间里的某一个单日期，所以只返回了该日期的2条记录，而非整个区间的3条。
2. 用字符串比较日期的写法本身就有问题
   就算单日期筛选做对了，用ToShortDateString()做等值比较的方式也有两个硬伤：
   • 强依赖服务器区域文化：ToShortDateString()的输出格式完全由服务器的系统区域配置决定，同一日期在不同环境下生成的字符串可能不一样，直接导致比较失配。
   • 无法利用数据库能力：如果是在IQueryable管道中写这个逻辑，EF Core没法把ToShortDateString()翻译成SQL语句，要么触发全表数据拉到内存里再处理，要么直接报错，效率和准确性都没保障。
3. 默认DateTime.TryParse的解析结果不可控
   你现在用的是没指定文化和格式的TryParse方法，传入的日期格式如果和服务器当前文化不匹配，很容易把日期解析错，比如某些区域配置下会把月、日的顺序搞反，筛选结果自然不对。

修复方案 #

1. 拆分范围字符串，分别解析起止日期，不要把整个范围字符串当单日期处理
2. 用DateTime原生的日期部分做范围比较，替代字符串转换，既避免区域问题，也能让EF Core正常翻译成SQL走数据库索引
3. 解析日期时显式指定约定的格式和文化，避免默认解析的不确定性

修复后的参考代码：

public IActionResult Index(string search)
{
    var admissionPatientQuery = _db.AdmissionPacients
        .Include(u => u.Patient)
        .Include(d => d.Doctor)
        .AsQueryable();

    if (!string.IsNullOrWhiteSpace(search))
    {
        // 按短横线拆分范围，最多拆成2部分，避免日期本身带短横线的歧义
        var dateRangeParts = search.Split('-', 2);
        // 显式指定你前端约定的日期格式和对应文化，避免区域影响
        var enUSCulture = CultureInfo.GetCultureInfo("en-US");

        if (dateRangeParts.Length == 1)
        {
            // 单日期查询逻辑
            if (DateTime.TryParseExact(
                dateRangeParts[0].Trim(), 
                "M/d/yyyy", 
                enUSCulture, 
                DateTimeStyles.None, 
                out var singleDate))
            {
                // 直接比较日期部分，不要转字符串
                admissionPatientQuery = admissionPatientQuery
                    .Where(x => x.DateAndTime.Date == singleDate.Date);
            }
        }
        else if (dateRangeParts.Length == 2)
        {
            // 范围查询逻辑
            var startValid = DateTime.TryParseExact(
                dateRangeParts[0].Trim(), 
                "M/d/yyyy", 
                enUSCulture, 
                DateTimeStyles.None, 
                out var startDate);
            var endValid = DateTime.TryParseExact(
                dateRangeParts[1].Trim(), 
                "M/d/yyyy", 
                enUSCulture, 
                DateTimeStyles.None, 
                out var endDate);
            
            if (startValid && endValid)
            {
                // 直接比较日期部分，自动包含结束日期当天的所有时间记录
                admissionPatientQuery = admissionPatientQuery.Where(x =>
                    x.DateAndTime.Date >= startDate.Date
                    && x.DateAndTime.Date <= endDate.Date);
            }
        }
    }

    return View(admissionPatientQuery.ToList());
}

注意：使用CultureInfo需要先引入System.Globalization命名空间。
优化建议：更稳妥的做法是直接把前端的日期范围拆成startDate、endDate两个独立参数传给后端，从参数设计上规避拆分字符串的格式歧义问题。

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