c# - キャストされたオブジェクトでクラス メソッドを実行する

Question

次のコードがあります。

DateTime dt = (DateTime)rdr["DateFromSQLDB"];
genericclass.classpropertystring = dt.ToShortDateString();

これを1行にまとめるにはどうすればよいですか？

これがクラスです...

public class genericclass
{
    public string classpropertystring { get; set; }
}

そのままでもうまく機能しますが、次のようなものを使用できるようにしたいです。

genericclass.classpropertystring = (DateTime)rdr["DateFromSQLDB"].ToShortDateString();

ありがとう！

Answer 1

U はちょうど 2 つの括弧を欠いています:

genericclass.classpropertystring = ((DateTime)rdr["DateFromSQLDB"]).ToShortDateString();

Answer 2

これはうまくいくはずです：

genericclass.classpropertystring = ((DateTime)rdr["DateFromSQLDB"]).ToShortDateString();

ただし、このようにするとコードが読みにくくなるため、質問にあるようにそのままにしておきます。

Answer 3

最初にオブジェクトを修飾することを確認する必要があります。

genericclass.classpropertystring = ((DateTime)rdr["DateFromSQLDB"]).ToShortDateString();

キャストの周りに括弧を追加することで、内部のすべてを型として扱うことができますDateTime。

私は@adrianbanksに同意しますが、最初の例の読みやすさを好みます。

Answer 4

DbDataReader.GetDateTime代わりに使用してください:

genericclass.classpropertystring = rdr.GetDateTime(rdr.GetOrdinal("DateFromSQLDB")).ToShortDateString();

Answer 5

なぜそれを凝縮したいのですか？読みにくくなります。また、返された rdr オブジェクトが null の場合はどうなるでしょうか? 爆破します。潜在的な悪い値を処理するために、もう少し堅牢なものを試してみます。より多くのコードが必要ですが、try catch ブロックをあちこちに配置しなくても、より多くの状況を処理できます。

public static void DoSomething()
    {
        using (var connection =new SqlConnection("constring"))
        {
            var command = new SqlCommand("myquery", connection);
            connection.Open();
            SqlDataReader rdr = command.ExecuteReader();
            while (rdr.Read())
            {
                var mc = new MyClass();
                var mydate = rdr["DateFromSQLDB"];
                if (mydate != null)
                {
                    DateTime date;
                    if (DateTime.TryParse(mydate.ToString(), out date))
                    {
                        mc.MyDateProperty = date.ToShortDateString();
                    }
                }
            }
        }
    }