c#ランダム文字列の続き

Question

前回の質問のURL

イメージ的には

using System;
using System.Text;
using System.Windows.Forms;

namespace testviewer
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void wordbox_TextChanged(object sender, System.EventArgs e)
        {
            throw new System.NotImplementedException();
        }

        private void numbarbox_KeyDown(object sender, KeyEventArgs e)
        {

            if (e.KeyCode == System.Windows.Forms.Keys.Enter)
            {
                
                //パスワードに使用する文字
                private static readonly string passwordChars = "0123456789abcdefghijklmnopqrstuvwxyz";

                public string GeneratePassword(int length)
                {
                    StringBuilder sb = new StringBuilder(length);
                    Random r = new Random();

                    for (int i = 0; i < length; i++)
                    {
                        //文字の位置をランダムに選択
                        int pos = r.Next(passwordChars.Length);
                        //選択された位置の文字を取得
                        char c = passwordChars[pos];
                        //パスワードに追加
                        sb.Append(c);
                    }

                    return sb.ToString();
                }
                
            }
        }

    }
}

のようにしたいのですがうまく行きません理由はわかっているのですがそれも不確かな状態です
どう助けてください><

Comments

[Tak1wa]

numbarboxが何か、は無視して良いですか？たとえば回答をボタン押下などに置き換えても問題ありませんか。

[come25136]

すみません存在を忘れていましたnumbarboxはランダム文字列の結果を表示するためのものなのですがよければそれにランダム文字列作成結果を表示できるコードを書いていただけると嬉しいです

Answer 1

こうですね。
メソッドの中にメソッドは記述しません。

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }


    //パスワードに使用する文字
    private static readonly string passwordChars = "0123456789abcdefghijklmnopqrstuvwxyz";

    public string GeneratePassword(int length)
    {
        StringBuilder sb = new StringBuilder(length);
        Random r = new Random();

        for (int i = 0; i < length; i++)
        {
            //文字の位置をランダムに選択
            int pos = r.Next(passwordChars.Length);
            //選択された位置の文字を取得
            char c = passwordChars[pos];
            //パスワードに追加
            sb.Append(c);
        }

        return sb.ToString();
    }

    private void numberbox_KeyDown(object sender, KeyEventArgs e)
    {
        if (e.KeyCode == Keys.Enter)
        {
            wordbox.Text = GeneratePassword(10);    //10文字の場合
        }
    }
}

Comments

[come25136]

ありがとうございます！！解決しました
余談なのですが、情報の追加・修正の依頼をするに書いた
>numbarboxはランダム文字列の結果を表示するためのもの
のnumbarboxはwordboxの間違いでした

Answer 2

いや、すみません。全然検討違いな回答をしてました。
コンパイルエラーが発生して進めない、という質問で良いでしょうか？

Comments

[come25136]

そうです