質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

4回答

1911閲覧

C# 演算子の使い方

maam

総合スコア55

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2019/01/16 08:53

三項演算子について
下記のコードを三項演算子を使って書き直す方法を教えてください

C#

1 2 if (year == "" && month == "" && day == "")  3 { 4 command.Parameters.Add(new SqlParameter("@birthday", DBNull.Value)); 5 } 6 else 7 { 8 command.Parameters.Add(new SqlParameter("@birthday", DateTime.Parse(year + "/" + month + "/" + day))); 9 } 10

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答4

0

正直自力で試した形跡のない丸投げっぽい感じなのですが……

年月日が図べ手空文字列の時以外ならDateTime.Parseやっちまうってのは例外が起きるので好ましくないかと
今のところstringにnullは代入可能だし、年月日のうち一つだけ"a"が入ってて他はstring.Emptyまたはnullだったら落ちる
なので日付としての整合性はDateTime.TryParseさんにお任せ
一応下記の通りでも年月日に今年の自分の誕生日を入れても期待通りの動きはした

C#

1 var year = string.Empty; 2 var month = string.Empty; 3 string day = null; 4 5 DateTime dy; 6 7 var sqlparam = new SqlParameter("@birthday", 8 DateTime.TryParse($"{year}/{month}/{day}", out dy) 9 ? (object)dy 10 : (object)DBNull.Value); 11

投稿2019/01/16 16:06

len_souko

総合スコア1337

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

C#

1DateTime birthday; 2command.Parameters.Add(new SqlParameter("@birthday", DateTime.TryParse(year + "/" + month + "/" + day,out BirthDay) ? (object)birthday : (object)DBNull.Value);

とか

投稿2019/01/16 16:18

YAmaGNZ

総合スコア10222

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maam

2019/01/17 01:02

ご回答ありがとうございます。 3行目の””Birthday”は””birthday”でよろしいでしょうか。”Birthday”だとエラーになってしまいました…
YAmaGNZ

2019/01/17 01:05

すいません。修正ミスですね”birthday”が正しいです。
guest

0

こうでしょうか。

command.Parameters.Add(new SqlParameter("@birthday", (year == "" && month == "" && day == "") ? DBNull.Value : DateTime.Parse(year + "/" + month + "/" + day)));

めっちゃ見づらい(^_^;)

投稿2019/01/16 09:00

takabosoft

総合スコア8356

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

papinianus

2019/01/16 09:08

このご指摘のとおり、右に長すぎて読み辛いです。可読性を犠牲にしてまで、自分では書けない(難解な)三項演算子を採用するのは疑問です
guest

0

ベストアンサー

csharp

1command.Parameters.Add(new SqlParameter("@birthday", (new [] {year, month, day}).All(string.IsNullOrEmpty) ? (object)DBNull.Value : (object 2)DateTime.Parse(year + "/" + month + "/" + day)));

投稿2019/01/16 09:06

編集2019/01/16 09:34
papinianus

総合スコア12705

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maam

2019/01/16 09:26

ご回答ありがとうございます System.DBNull と System.DateTime との間に 暗黙的な変換がないため条件式の型がわかりません というエラーがでてしまいました…。
papinianus

2019/01/16 09:34

追記しましたが、objectにキャストしてください
maam

2019/01/17 00:45

ご回答ありがとうございます。 すごく初歩的なことだと思うのですが、 (new [] {year, month, day} とは どういう意味なのかおしえていただけますでしょうか。
papinianus

2019/01/17 00:52 編集

string[] stringArray = new string[] { year, month, day}; の意味です。その場で文字の配列を作り、その全て(All)が空であること(IsNullOrEmpty)を確認しようとしています。 上のように宣言文を置いたら1行でかけないし、stringArrayはずっと使うものでもないので。こういう書きかたをしました。 (newと[]の間にstringがないのは、データからそれがstringだとコンパイラが判断できるからです) ところで、len_souko様やYAmaGNZ様の回答のほうが望ましい動作をしますので、(変更できるかどうか分かりませんが)ベストアンサーはそちらにしたほうがいいと思います。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問