c＃、年齢範囲を指定してユーザーを選択するSQLを構築

Question

誕生日の列を持つすべてのユーザーを保持する才能テーブルがあります。指定された年齢層の中で才能が最も良い方法は何でしょうか。ここに私が持っているものがありますが、カップルの日でオフになっているようです。そこには良い方法がありますか？

// BUILD SQL FROM FORM DATA 
sqlString += "SELECT * from Talent"; 

if (minAge != 0 || maxAge != 120) 
{ 
    // The age criteria has been change, filter by age. 

    // select all talents that have birthdays between the following 2 dates. 
    DateTime startDate = (DateTime.Now - new TimeSpan((maxAge * 365), 0, 0, 0)); // maxAge * 365 = totalDays 
    DateTime endDate = (DateTime.Now - new TimeSpan((minAge * 365), 0, 0, 0)); 
    sqlString += " WHERE Birthdate BETWEEN '" + startDate.ToString() + "' AND '" + endDate.ToString() + "'"; 
} 

Answer 1

個人的に、私は、終了日の範囲で000点に注意してください（startDate.ToString("yy-MM-dd 00:00:00.000")とendDate.ToString("yy-MM-dd 23:59:59.000")作品のベストを使用して発見しました。何らかの理由で、私の経験では、SQLはおそらく丸め誤差のいくつかの種類に（オフになっていますそれは範囲である。余談として

に来るとき）、あなたは時間の計算のためにTimeSpanオブジェクトから静的メソッドを使用することができます。例えばTimeSpan.FromDays(...)

Answer 2

問題は日付だけでなく、時間を考慮した、DateTime.Nowに関連するかもしれません。 DateTime.Todayと交換してください。

Answer 3

なぜDateTime.AddYearsメソッドを使用しないでください。

DateTime startDate = (DateTime.Now - new TimeSpan((maxAge * 365), 0, 0, 0)); 

のinsated

DateTime startDate = DateTime.Now.AddYears(-maxAge); 

もう一つは、次のとおりです。代わりにStringBuilderを使用して、SQLクエリを構築するために文字列の間+演算子を使用しないでください。 SQL Serverを使用していると仮定すると、

Answer 4

...

using (var connection = new SqlConnection(connString)) 
    using (var command = connection.CreateCommand()) { 

     string tsql = @" 
      select * 
       from Talent 
       where DATEDIFF(YEAR, BirthDay, GETDATE()) BETWEEN @minAge AND @maxAge"; 

     command.CommandText = tsql; 
     command.CommandType = CommandType.Text; 

     int minAge = 1; 
     int maxAge = 120; 

     SqlParameter minAgeParam = command.CreateParameter(); 
     minAgeParam.Direction = ParameterDirection.Input; 
     minAgeParam.DbType = SqlDbType.TinyInt; 
     minAgeParam.ParameterName = "@minAge"; 
     minAgeParam.Value = minAge; 

     SqlParameter maxAgeParam = command.CreateParameter(); 
     maxAgeParam.Direction = ParameterDirection.Input; 
     maxAgeParam.DbType = SqlDbType.TinyInt; 
     maxAgeParam.ParameterName = "@maxAge"; 
     maxAgeParam.Value = maxAge; 

     // Just unsure here whether I must add the parameters to the command, 
     // or if they are already part of it since I used the 
     // SqlCommand.CreateParameter() method. 
     // Been too long since I haven't done any ADO.NET 
     command.Parameters.Add(minAgeParam); 
     command.Parameters.Add(maxAgeParam); 

     connection.Open(); 

     SqlDataReader reader = null; 

     try { 
      reader = command.ExecuteReader(); 
      // Process your records here... 
     } finally { 
      connection.Close() 
      command.Dispose(); 
      connection.Dispose(); 
      if (reader != null) { 
       reader.Dispose(); 
      } 
     } 
    } 

@minAgeと@maxAgeはあなたの年齢パラメータです。

また、DATEDIFF TSQL functionに日、月、時間、分、秒などの違いを考慮するように指示することもできます。したがって、それに応じてパラメータ値を変換する必要があります。

Answer 5

閏年の代わりに365を使用しているため、元のソリューションは数日でオフになります。