「次へ」ボタンを押して値を一つずつ表示する

Question

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
基本情報技術者試験の単語を覚えるためのアプリを作成しています。「次へ」ボタンを用意して、クリックされるたびに単語を表示させようとしていますが、方法がわかりません。
現時点まで値を取得するDB接続・値取得クラスを用意、そして値を保持するメソッド（何度もDB接続をするのが面倒なため）を用意しましたが、どのように単語を次々表示させていくかがわからず悩んでおります。
初心者なのできれいなコードではありませんし、プログラムの流れもわかりづらくおかしいかもしれませんが、アドバイスを頂けたらと思っております。

###発生している問題・エラーメッセージ

###該当のソースコード

(ShowVocabForm)
//MainForm
namespace ShowVocabNotes
{
     private void NextBtnClick(object sender, EventArgs e)
        {
      //値保持メソッド呼び出し
            string[] vocabdatas = HoldDatas(datas);   
            {
                bool btnclick = true;

                if (btnclick)
                {
           
                    for(int i = 0; i < vocabdatas.Length; i++)
                    {
                        VocabRichTextBox.Text = vocabdatas[i];
                        
                    }
                }
            }
        }
}
//値を保持するメソッド
 public string[] HoldDatas(string[] vocdatas)
        {
            //値を保持したいから箱を用意
            Dictionary<string, string> datas = new Dictionary<string, string>();
            //DBUtilsクラスへ行って、DBから値を取得する
            datas = DBUtils.GetData(datas);
            //取得した値を配列にキャスト
            vocdatas = new string[datas.Keys.Count];
            datas.Keys.CopyTo(vocdatas, 0);

            return vocdatas;
        }

###試したこと
for文を使用してみましたが、流れとしてはおかしい挙動になりました。

###補足情報(言語/FW/ツール等のバージョンなど)
OS: Windows10
DB: PostgreSQL
開発環境：Visual Studio2015

Answer 1

1. ボタンのイベントハンドラーでなにをすべきか

ボタンが１回おされれたときにやるべき処理しか書いてはいけません。ボタンを１回おされたときにやるべきは、次の単語一つだけを選んで表示することだけです。では「次の単語って、何番目の単語のことなのか」をどうやって決めればいいのでしょうか？答えはフォームクラスにフィールドを定義してそこに「今何番目を表示しているか」を覚えておくことです。

1. ボタンのイベントハンドラーで何をすべきでないか

せっかくHoldDatasというメソッドを用意したのに、ボタンのイベントハンドラーでそれをよびだしてはダメです。一度しかやらない処理はこのフォームのロード時にやりましょう。HoldDatasをボタンのイベントハンドラーで呼び出してはいけないとなるとどうやってこのデータを参照すればよいのでしょうか？答えはFormのクラスにフィールドを定義しそこに覚えておくことです。

解決方法にはいろいろなやりかたがありますが、最も素朴な答えだけを回答してみました。他にもやりかたはいろいろありますがまずはフィールドを使ったプログラミングについて考えてみてください。

Comments

[Kaffy]

お返事ありがとうございます！今日フィールドについて調べてみました。
ただ、HoldDatasの扱い方がわからなくなってしまいました。
private string[] holddatas; とprivate int index; の二文Formクラスで定義して、そのあとが悩んでいます、、
初心者なので、知識不足ですが、アドバイスいただけたらと思います。

[KSwordOfHaste]

定義したフィールドの値は同じクラスのメソッドのどれからでも参照・設定ができます。これを踏まえて考えてみてください。

[Kaffy]

単語の表示できました！ありがとうございます。