2017-07-28 6 views
0

私は9つのボタンのテキストをすべて ""に割り当てるための関数を作成しました(私は一般的に "" "を使用しようとしました。私が間違っていると指摘していただければ幸いです)。しかし、すべてのパラメータを空の文字列にきれいに割り当てることができたようです。C#すべてのメソッドパラメータに1つの値をきれいに割り当てよう

public void resetText(ref string buttonText1, ref string buttonText2, ref string buttonText3, ref string buttonText4, ref string buttonText5, ref string buttonText6, ref string buttonText7, ref string buttonText8, ref string buttonText9) 
    { 
     buttonText1 = ""; 
     buttonText2 = ""; 
     buttonText3 = ""; 
     buttonText4 = ""; 
     buttonText5 = ""; 
     buttonText6 = ""; 
     buttonText7 = ""; 
     buttonText8 = ""; 
     buttonText9 = ""; 
    } 

すべてのパラメータを再割り当てする方法がありますか?この関数で50個の変数に新しい値を割り当てなければならない場合、これは非常に悪く見えます(例としてボタンを使用しただけです)。

+0

これを行うよりクリーンな方法はないようです。しかし、おそらく、親関数は、テキスト値を配列に保持することで、これをよりうまく実行できます。 –

+0

したがって、すべてのパラメータをループして、それらにヌル値などを割り当てることがわかっている方法はありませんか? –

+0

次のように更新できます: 'buttonText1 = buttonText2 = buttonText3 = buttonText4 = buttonText5 = buttonText6 = buttonText7 = buttonText8 = buttonText9 = string.Empty; ' – zzT

答えて

0

これは、これを実行するための最良の方法です。シンプルで読みやすいです。

本当に文字数を減らしたい場合は、buttonText1 = buttonText2 = buttonText3 = ... = "";のようにすることができます。

コードサイズを減らす別の方法は、リフレクションによって行うことです。この方法は、コードをあまりにも複雑なものにしてしまうので、私はこの方法をお勧めしません。

関連する問題