TextBox = SelectからSqlへの変換日時

Question

こんにちは、sqlの値を日付としてテキストボックスに取り込みたいと思います。私はこのコードを使用しています：

SqlCommand command = new SqlCommand("select top 1 expirationdate from incomes where memberid='0' or memberid = '" + textBox22.Text + "' order by expirationdate DESC", con); 
textBox17.Text = command.ExecuteScalar().ToString("d/M/yyyy"); 

私はエラーを取得する：メソッドの過負荷は、「ToStringメソッドは、」あなたが最初のDateTimeに変換する必要があり1つの引数

Answer 1

を取りません。また、SQLのパラメタを使用してSQLインジェクションを回避します。

SqlCommand command = new SqlCommand("select top 1 expirationdate from incomes where memberid='0' or memberid = @memberId order by expirationdate DESC", con); 
    command.Parameters.AddWithValue("memberId",textBox22.Text); 
    var result = command.ExecuteScalar().ToString(); 
    textBox17.Text = Convert.ToDateTime(result).ToString("d/M/yyyy"); 

Answer 2

まず：使用パラメータ

第二は：

int memberId = int.Parse(textBox22.Text); // or whatever 
DateTime expiry; 
using(var command = new SqlCommand(
    "select top 1 expirationdate from incomes where memberid='0' or memberid = @memberid order by expirationdate DESC", con)) 
{ 
    command.Parameters.AddWithValue("memberid", memberId); // again, about 20 ways to do this 
    expiry = (DateTime)command.ExecuteScalar(); 
} 
textBox17.Text = expiry.ToString("d/M/yyyy"); // or whatever 

または "Dapperの" のようなツールでそれをフォーマットしようとする前ExecuteScalarから値をキャスト：

int memberId = int.Parse(textBox22.Text); // or whatever 
var expiry = con.QuerySingle<DateTime>(
    "select top 1 expirationdate from incomes where memberid='0' or memberid = @memberid order by expirationdate DESC", 
    new { memberId }); 
textBox17.Text = expiry.ToString("d/M/yyyy"); // or whatever