当在日期时间列的两个时间之间获取数据时，它不会返回任何行 - ASP.NET / C#

Question

我从事一个 ASP.NET web forms 项目； 我无法在 datetime 列的两次（时间从和时间到）之间获取数据。

C#函数

public DataTable GetDataForSearch(string datelogged, string FromTime, string ToTime)
{
    string response = string.Empty;

    SqlCommand cmd = new SqlCommand();
    DataTable dt = new DataTable();

    try
    {
            conn.Open();
            cmd.Connection = conn;
            cmd.CommandText = "select datelogged AS EntredDatetime, Doc_type AS OrderType, Printer_name, BranchID AS BranchCode, Status, id from Print_Report where cast(datelogged as date)=@datelogged  and and FORMAT(CAST(datelogged AS DATETIME), 'HH:mm')>'@FromTime' AND FORMAT(CAST(datelogged AS DATETIME), 'HH:mm')<@ToTime";
            cmd.CommandType = CommandType.Text;
            cmd.CommandTimeout = 50000;

            cmd.Parameters.AddWithValue("@datelogged", datelogged);
            cmd.Parameters.AddWithValue("@FromTime", FromTime);
            cmd.Parameters.AddWithValue("@ToTime", ToTime);

            SqlDataAdapter sda = new SqlDataAdapter(cmd);
            sda.Fill(dt);
        }
        catch (Exception ex)
        {
            response = ex.Message;
        }
        finally
        {
            cmd.Dispose();
            conn.Close();
        }

        return dt;
}  

当我从 SQL 服务器尝试时，它返回 2 行：

select 
    datelogged as EntredDatetime, Doc_type as OrderType, 
    Printer_name, BranchID as BranchCode, Status, id 
from 
    Print_Report 
where 
    BranchID = '10207' 
    and cast(datelogged as date) = '2010-07-05' 
    and Doc_type = 'BP'  
    and format(cast(datelogged as DATETIME), 'HH:mm') > '13:30' 
    and format(cast(datelogged as DATETIME), 'HH:mm') < '14:00'

预期结果：

我将查询修改为如下：

  cmd.CommandText = "select datelogged AS EntredDatetime, Doc_type AS OrderType, Printer_name, BranchID AS BranchCode, Status, id from Print_Report where BranchID=@BranchCode and cast(datelogged as date)=@datelogged and Doc_type=@OrderType and  FORMAT(CAST(datelogged AS DATETIME), 'HH:mm')>='@FromTime' AND FORMAT(CAST(datelogged AS DATETIME), 'HH:mm')<='@ToTime'";

但仍然没有得到任何结果所以我做了什么来解决问题

Answer 1

所以，有几件事：

不要包装长行代码，它们很容易出错。

看看你的 SQL，你有这个：

  select datelogged AS EntredDatetime, Doc_type AS OrderType, Printer_name,
  BranchID AS BranchCode, Status, id from Print_Report 
  where cast(datelogged as date)=@datelogged
  and and FORMAT(CAST(datelogged AS DATETIME), 'HH:mm')>'@FromTime'
  AND FORMAT(CAST(datelogged AS DATETIME), 'HH:mm')<@ToTime"

请注意，现在我们在您的 SQL 中看到“and and”是多么容易。

并且不要在 SQL 中做那些转换，因为它很乱，但是这样的表达式不能被索引，而且它们运行起来会很慢。 （此外，它使 SQL 变得一团糟）。

下一步：考虑将内联 SQL 移出存储过程，但更好（并且更少的努力和工作）是使用视图，因此比存储过程“更少”工作，您可以使用查询生成器/设计师，甚至更好的是多个例程可以使用一个“视图”。 您仍然使用内联 SQL，但只说

 select * from vPrint_Report
 WHERE datelogged is BETWEEN @dtStart AND @dtEnd";

接下来：

不要使用 add with value - 使用 STRONG 类型的参数。

接下来：

虽然不推荐使用 Parmaters.Add("some parm", "some value")，因为第二个参数可能与 int 混淆为 dbtype？

使用带有“ADD”的类型参数不被弃用，!!，事实上我建议使用

 .ADD("@ParmName", sqlDbType.int).Value = 

同样，上面的格式没有被弃用，只有上面第二个参数的非 dbtype 的重载被弃用了！！！

接下来：

您不显示该连接 object 的创建位置。 不要尝试建立一些全局范围的连接 object。在“网络时代”之前，为了性能，是的，我们经常“坚持”连接 object，但是基于 web，有一个自动“连接”池，因此标准是每次都重新创建连接，让系统自动为您处理。 通过这样做，您可以利用基于 .net web 的系统中的“自动”连接系统。

每次重新创建连接 object 不会影响性能，因为连接池将“查找”并使用缓存连接 - 它运行速度很快 - 每次都可以随意重新创建连接 object。

你如何让系统为你管理这个？ 当然，为什么您总是将该代码包装在 using 块中。 在您的示例中，您仅在代码错误时处理连接，而不是在成功时处理连接。

接下来：

如前所述，不要尝试解析或转换 SQL 中的日期和日期时间，而是为开始日期和结束日期（均带有时间）提供 STRONG 类型。 这样的代码不仅不那么混乱，而且代码中的“微小”工作也很少，但是 SQL 变得 VAST VAST 不那么混乱了。 因此，我们用“一点点”代码换取 nice clean SQL 的巨大奖励。

所以，让我们吸取以上所有教训，因此我们现在得到了：

public DataTable GetDataForSearch(string datelogged, string FromTime, string ToTime)
{

    DataTable dt = new DataTable();
    string strCon = Properties.Settings.Default.TEST4; // change to YOUR conneciton

    using (SqlConnection conn = new SqlConnection(strCon))
    {
        string strSQL =
            @"select datelogged AS EntredDatetime, Doc_type AS OrderType, 
            Printer_name, BranchID AS BranchCode, Status, id
            FROM Print_Report
            WHERE datelogged is BETWEEN @dtStart AND @dtEnd";

        DateTime dtDate = DateTime.Parse(datelogged);
        DateTime dtFromTime = DateTime.Parse(FromTime);
        DateTime dtToTime = DateTime.Parse(ToTime);

        DateTime dtStart = 
          dtDate.Add(new TimeSpan(dtFromTime.Hour, dtFromTime.Minute, dtFromTime.Second));

        DateTime dtEnd =
          dtDate.Add(new TimeSpan(dtToTime.Hour, dtToTime.Minute, dtToTime.Second)); 

            
        using (SqlCommand cmd = new SqlCommand(strSQL, conn))
        {
            cmd.Parameters.Add("@dtStart", SqlDbType.DateTime).Value = dtStart;
            cmd.Parameters.Add("@dtEnd", SqlDbType.DateTime).Value = dtEnd;

            try
            {
                conn.Open();
                dt.Load(cmd.ExecuteReader());
            }
        }
    }
    return dt;
}

因此，请注意我们如何让系统“关闭”连接并处理掉它。 即使代码出错，即使没有被困，连接和命令 object 也会为您正确处理和管理 - 在所有情况下！！！

另外，请注意在代码方面如何获得 STRONG TYPED 日期的开始和结束，从而使 SQL 部分的工作量大大减少，但更重要的还意味着我们使用 + 享受参数的 STRONG 类型值，并且我们喜欢使用高速索引。

Answer 2

始终声明您传递给 SQL 的参数类型。不要依赖 sql 来推断它。 还要养成将数据库连接包装在Using块中的习惯。

cmd.Parameters.Add("@datelogged", sqldbtype.datetime).value = datelogged;
cmd.Parameters.Add("@FromTime", sqldbtype.time).value = FromTime;
cmd.Parameters.Add("@ToTime", Sqldbtype.time).value = ToTime;

您还需要为@branch 和@ordertype 参数添加代码。

您的 SQL 如下所示。 在编写 SQL 时，大多数错误都可以通过在编写时始终如一地格式化 SQL 来解决。 当你像我上面那样声明参数的数据类型时，参数不需要用引号引起来。 它为您处理所有这些。 我没有单独比较日期，而是选择将时间作为时间数据类型传递，然后将它们转换为查询中的日期时间。 从那里，您可以将两个日期时间与+一起添加，然后以相同的方式进行比较。 如果您决定将您的传入/传出值作为日期时间传递，则只需删除顶部的声明行。

您可以在WHERE子句中使用BETWEEN ，但这由您自行决定。

DECLARE @To DATETIME = (@datelogged + CAST(@ToTime AS DATETIME)), @From DATETIME = (@datelogged + CAST(@FromDate AS DATETIME));

SELECT
    datelogged AS EntredDatetime, 
    Doc_type AS OrderType, 
    Printer_name, 
    BranchID AS BranchCode, 
    Status,
    ID
FROM Print_Report 
WHERE BranchID = @BranchCode AND 
    Doc_type = @OrderType AND 
    datelogged >= @From AND 
    datelogged <= @To