『C#』 labelなどのテキストを、関数の戻り値で設定するときに、元のテキストのままで、変更せずにしたい

Question

『C#』 labelなどのテキストを、関数の戻り値で設定するときに、元のテキストのままで、変更せずにしたい

説明しずらいんですが、、例えば

Dictionary<string,string> b = new Dictionary<string,string>{{"abc","あ"},{"cba","い"}}
string a(string c)
{
   return b[c];
}

という関数を作って、
label1.Text = a("abc")
とすると、
label1.Textは"あ"になりますよね。
ですが
label1.Text = a("abcd")
とやると、
エラーが発生してしまいます！
ですが
もし、値が存在しなかったら、label1のテキストを変えないようにしたいです。
ですが、label1.Text = a("hoge") の前に、ifとかを書くのはちょっとコードの長さとかがあれなので、
関数側で判定して、存在しなかったら変更しない
たとえばもとはlabel1.Text = "あいうえお"の状態で、
label1.Text = a("abcd")
としても、
label1.Textは、"あいうえお”のままという感じにしたいです
それで、
label1.Text = a("cba")
だったら、label1.Textは"い"という感じにしたいです。

試したこと

Dictionary b = new Dictionary(){{"abc","あ"},{"cba","い"}}
string a(string c)
{
   return b[c];
}

を、

Dictionary b = new Dictionary(){{"abc","あ"},{"cba","い"}}
string a(string c,Control d)
{
   if(b.ContainsKey("c"))
   {
       return b[c];
   }
   else
   {
         return d.Text;
   }
}

という感じにしましたが、
もし、設定する対象が、コントロール以外だったら、できません！

label.Text = 〇〇; の前に、

try
{
label.Text = 〇〇;
}
catch
{
return;
}

という感じにしたら、そりゃ対処できますが、
コードが増えてしまいます！！

エラーメッセージ

存在しないKeyを指定したとき

ハンドルされてない例外	????	✖
System.Collections.Generic.KeyNotFoundException: '指定されたキーはディレクトリ内に存在しませんでした。'
この例外は、最初にこの呼び出し履歴でスローされました:
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

System.Collections.Generic.Dictionary<TKey, TValue>.this[TKey].get(TKey)
[名前空間_Test.値の設定メソッド()] の Setting.cs(navigate-to-context:1)
[名前空間_Test.Form1.Load(object, System.EventArgs)] の Setting.cs(navigate-to-context:2)
System.Windows.Forms.ListControl.OnSelectedValueChanged(System.EventArgs)
    System.Windows.Forms.Load(object sender, EventArgs e)
System.Windows.Forms.Control.ControlNativeWindow.OnMessage(ref System.Windows.Forms.Message)
System.Windows.Forms.Control.ControlNativeWindow.WndProc(ref System.Windows.Forms.Message)
...
[呼び出し履歴の切り捨て]|||

#使用ツールなど
IDE : Microsoft Visual Studio 2019
OS : Windows10 64bit
言語 : C#
種類 : Windows Forms
フレームワーク : .Net(DotNet) Frameworks

Answer 1

Dictionaryには、キーがあるかをチェックするContainsKeyというメソッドがあります（MSDN）ので、キーを参照する前にチェックを入れましょう。例外処理をするよりはコンパクトかと思います。

Comments

[maisumakun]

> コードが増えてしまいます！！

対応すべきことが増えるのであれば、コードが増えるのは必要なことです。

[JumpActionGames]

試したことにありますが、
>Dictionary b = new Dictionary(){{"abc","あ"},{"cba","い"}}
>string a(string c,Control d)
>{
> if(b.ContainsKey("c"))
> {
> return b[c];
> }
> else
> {
> return d.Text;
> }
>}
>という感じにしましたが、
>もし、設定する対象が、コントロール以外だったら、できません！

[JumpActionGames]

でも大量にあるので、もしできたとしても50行くらい増えちゃうんですよね、

[maisumakun]

えっと、「Dictionaryから引いてくる」部分だけを独立して関数にする必要はありますか？

[maisumakun]

「コントロールとキーを渡して、あればセットして戻る」という関数を作れば、「ないときに何を返すか」で悩む必要もないかと思うのですが。

[JumpActionGames]

関数で行っている実際のコードは
GameLanguages.langs[Tuple.Create(Tuple.Create(langs,Scenes),keys)];
ですから、
さすがに、、

[maisumakun]

簡略化しすぎて、重要な部分が伝わっていません。懸念点がわかるようにコード例を書き直したほうがいいかと思います。

[JumpActionGames]

ちょっと読み込むjsonをresource上において、存在しないときはresourceからとってくるという感じにします

Answer 2

メソッドを作るとき、想定できる例外で例外を吐いてはいけません。
ただし、続行不能によりアプリケーションを終了する場合を除きます。

この場合 aメソッドに問題があります。
Dictionary<string,string> b に該当するキーがなかった時に何を返すかを予め決めておきます。
ここでは単純にnullを返すことにしましょう。

string A(string c)
{
    return b.Keys.Contains(c) ? b[c] : null;
}

とすると
label.Text = A("abcd") ?? label.Text
こんな具合に書けます。

Answer 3

この問題の解決方法を変えました。

Answer 4

スマートではないですが、キーが一致する項目がディクショナリになければ元の値をそのまま代入するとか。

label1.Text = a("cba") ?? label1.Text;

Answer 5

エラーが出たときはlabelへの代入しないようにすればいいのです

Comments

[JumpActionGames]

それはどうするのですか？

[JumpActionGames]

label.Text = 〇〇; の前に、
try
{
label.Text = 〇〇;
}
catch
{
return;
}
とかは嫌です

[y_waiwai]

「C#　例外」でぐぐれば、エラーが出たときの解説が出てきます

[JumpActionGames]

label.Text = 〇〇; の前に、
try
{
label.Text = 〇〇;
}
catch
{
return;
}
とかは嫌です

[y_waiwai]

質問に書いてないことをあと付けでどーこー言わないようにｗ
あなたの好みなんか知ったこっちゃありません