引数ありのメソッドの作成方法

2019/01/17 00:48

ご指摘ありがとうございます。訂正しました

行動規範の内容に同意します

回答2件

ベストアンサー

多分悩むのは、背景色のほうだと思います。基底クラスの型で受けとる関数にすれば、comboboxもtextも代入できます。当然個数が違うので配列みたいなデータ形式にする。

csharp
1private void Reset(TextBox text, IEnumerable<Control> Controls)
2//private void Reset(TextBox text, Control[] Controls) こんなんでもいいです。もし分かりにくければ
3{
4  text.Text = string.Empty;
5  foreach(var ctrl in Controls)
6  {
7    ctrl.BackColor = Color.White
8  }
9}
10private void Raise(TextBox text, string message, IEnumerable<Control> Controls)
11{
12  text.Text = message;
13  foreach(var ctrl in Controls)
14  {
15    ctrl.BackColor = Color.MistyRose;
16  }
17}

投稿2019/01/16 08:20

編集2019/01/16 09:03

総合スコア12705

2019/01/16 08:50

ご回答ありがとうございます。 IEnumerable<Control> Controlsというものを初めて見ました。どういう役割があるのか詳しく教えていただけないでしょうか…

2019/01/16 09:02

IEnumerable<Control>は型です(stringとかintとかboolとかint[]とかと同じ)。Controlsは変数名なので省略オブジェクト指向のお話を思いだしてほしいのですが、例えば、comboboxも、textboxも共通の祖先であるControlを継承しています(動物→人間、動物→馬みたいに。人間も馬もどっちも動物)。これがここにControlと書いてある理由です。 IEnumerableのIは、インターフェイスをあらわしています。Interfaceはクラスの継承とは別で、特定の機能をもっている(実装している)ことを表現するものです(人間も、走れる、馬も走れる、動物でない車も走れる。動物でもペンギンは走れない。このとき、人間や車はIRunnableを実装していると言えます) IEnumerableは「かぞえあげることができる」という意味およびその実装をもっており、ここではforeachにかけることができる、という機能をもっているならList<T>でもT[]でも何でもいいよ、ということでこうしました。 IEnumerableに<>があるのはジェネリックインターフェイスであることを示しています。これは、配列みたいなIEnumerableに入っているものの型を指定する機能です。IEnumerable自身は配列っぽいものなので、それがintの配列(int[])なのか、stringの配列なのか(string[])を宣言時に決めると思います。なので、そこで言っていることをざっくりまとめると、「配列っぽいものをください、中身がControlのやつで」ということになります。

2019/01/17 00:55

とてもわかりやすいご説明をしてくださりありがとうございます。型にもたくさんの種類があるのですね…

2019/01/17 01:03

Control[] Controlsにするのであれば、params Control[] Controlsのほうが分かりやすそうな気がします。呼び出すとき Reset(TextBox1,new Control[]{label1,label2}); ではなく Reset(TextBox1,label1,label2); となりますし

2019/01/17 01:12

maam様へ(また疑問がわくのではないかと思い) paramsは可変長引数を使うためのキーワードであり、かつparamsは配列でしか使えない(配列なら使える)ため、このようなご提案をいただいています。可変長にすると、呼び出すときの2つ目に書いてあるように、最後の変数を何個でも(0個でも)すきにかけるようになります。配列であることを強制してしまうと、呼び出すとき、の1つ目のように事前に配列にして、渡さないといけない。利用するうえでは便利ですよね。

2019/01/17 01:18

YAmaGNZさん papinianusさん細かいことまで教えていただきありがとうございます＞＜ paramsも初めて目にするものです。配列でしか使うことができないんですね。確かに毎回配列として呼び出すよりも、最初に宣言しておけば便利ですね。初心者の自分にお時間をつかっていただき本当にありがとうございます。

2019/01/17 02:43

もうひとつ質問があります。呼び出し側ですが、 Reset(TextBox1,label1,label2);というのは TextBox１　→　色が変わる label1,label2　→　メッセージを表示する　箇所で合っていますか？

2019/01/17 02:45

逆っす。最初のやつが、メッセージをEmpty("")にするやつで、可変のやつは背景色をかえるやつです。

2019/01/17 02:49

private void Reset(TextBox text, params Control[] Controls) と宣言して Reset(TextBox1,label1,label2); と呼び出した場合、 textにTextBox1が入り、Controlsにlabel1、label2が入ります。

2019/01/17 03:51

private void Reset(TextBox text,Control[] Controls) { //labelを空白、BackColorを白にするメソッド定義 text.Text = string.Empty; foreach(var ctrl in Controls) { ctrl.BackColor = Color.White; } } private void Raise(TextBox text, string message,Control[] Controls) { //labelにメッセージ"必須入力項目です"、BackColorを赤にするメソッド定義 text.Text = "必須入力項目です。"; foreach(var ctrl in Controls) { ctrl.BackColor = Color.MistyRose; } } private void Error(TextBox text, string meesage,Control[] Controls) { //labelにメッセージ"生年月日が不正です"、BackColorを赤にするメソッド定義 text.Text = "生年月日が不正です。"; foreach (var ctrl in Controls) { ctrl.BackColor = Color.MistyRose; } } private void AddBt_Click(object sender, EventArgs e) { //必須項目の判定 if (sei == "" || mei == "") { //姓が空欄の時 if (sei == "") { //label9.Text = "必須入力項目です。"; //textSei.BackColor = Color.MistyRose; Raise(label9, new Control[]{textSei}); } else { //label9.Text = ""; //textSei.BackColor = Color.White; Reset(label9, new Control[]{ textSei }); } 現在、このように作成しているのですが、Raiseの部分に ”（TextBox,string,control[]）の必要な仮パラメーター’Controls’Controlsに対応する特定の引数がありません。” label9に”’’System.Windows.Forms.Label’から’System.Windows。Forms.TextBox’へ変換することはできません。” というエラーがでてしまいました…。

2019/01/17 03:56

Reset、Raiseともに第1引数はTextBoxと定義しています。ですので、LabelからTextBoxへ変換しようとしてできませんとエラーになります。第一引数がLabelでいいのでしたら、Labelで宣言してください。

2019/01/17 04:00

label9の”’’System.Windows.Forms.Label’から’System.Windows。Forms.TextBox’へ変換することはできません。” というエラーは消えました！もう一つのエラーの原因はなんでしょうか…。

2019/01/17 04:09

messageの使い方が違っていたみたいです。下記でうまく動きました！ありがとうございます＞＜ //姓が空欄の時 if (sei == "") { Raise(label9, "必須入力項目です", new Control[]{textSei}); } else { Reset(label9, new Control[]{ textSei }); }

2019/01/17 04:35

labelだったのですね。それは失礼しました。型の指定をControlにしてもいいのかもしれないです。

2019/01/17 04:39

説明不足ですみませんでした…。型の指定とはどの部分のことですか？

2019/01/17 04:45 編集

private void Raise(TextBox text, string message, IEnumerable<Control> Controls) この関数の引数の、^^^^^ ここです。これを > private void Raise(Control MessageTarget, string message, Control[] Controls) こういう感じ

2019/01/17 04:54

そういうことですね！先程のYAmaGNZさんのアドバイスを受けて、 private void Raise(Label MessageTarget, string message, Control[] Controls) 　　　　　　　　　^^^ にしましたが、いかがでしょうか。

2019/01/17 05:00

設計しだいじゃないですか？第一引数がLabelでいいのか、汎用的にControlのほうがいいのかなんて第三者には分からないでしょ。

2019/01/17 05:11

YAmaGNZ様のおっしゃるとおりです。質問から配列のほうはControlじゃないとそもそも実現できないのでそうしました。あとはこのパーツをどの程度の範囲で利用するのかです。個人的には(他の言語の経験から)型は具体的に絞っておくほうが(つまり、Controlじゃなく、LabelやTextBoxのほうが)好きですが、この関数この用途ならControlをすすめます。