BMI テキストボックスを小数点第一位まで入力を可能にする （小数点第二位からは入力させない）

Question

答えのあったページからコピペをし、変数などを揃えてビルドしても小数点が制御できません。
エラーもないのでどこがわるいのかわからなく困っています。

コード****

        // 数値のみ小数点第一位まで入力が可能であること。
        //----------------------------------------------------------------------------------------




　　　　　　//身長のテキストボックス入力時
        private void textHeight_KeyPress(object sender, KeyPressEventArgs e)
        {

            //.KeyChar　=　押されたキーに対応する文字を取得または設定します。
            //数値と小数点とバックスペース以外の入力は不可
            if ((e.KeyChar < '0' || e.KeyChar > '9') && e.KeyChar != '\b' && e.KeyChar != '.')
            {

                //.Handled = 取得またはを示す値を設定するかどうか、 KeyPress イベントが処理します。
                e.Handled = true;
                //e.Handled = Trueと入れると、キャンセルできる。
            }
        }



        /*イベントが呼び出される順番が KeyDown→TextChanged→KeyUp なので
        　KeyDownイベントを使用します。*/


        //DeleteキーやBackSpaceが押されたかどうかを判断。
        bool isDelete = false;

        private void textHeight_KeyDown(object sender, KeyEventArgs e)
        {
            isDelete = (e.KeyCode == Keys.Delete || e.KeyCode == Keys.Back);
        }









        //整数部桁数
        private readonly int LENGTH_INT_PART = 7;
        //小数部桁数
        private readonly int LENGTH_DECIMAL_PLACES = 2;

        private void textHeight_TextChanged(object sender, EventArgs e)
        {
            var txtBox = sender as TextBox;

            txtBox.TextChanged -= textHeight_TextChanged;

            try
            {
                if (!string.IsNullOrWhiteSpace(txtBox.Text))
                {
                    var value = txtBox.Text;
                    //整数と小数を分割する
                    string[] values = value.Split('.');

                    int currentPoint = 0;
                    switch (values.Length)
                    {
                        case 1:





                            //------------------------------
                            //整数部のみで構成されている場合
                            //------------------------------
                            //整数部の最大桁数を超えた場合、今回の入力値を無効にする
                            if (value.Length > LENGTH_INT_PART)
                            {
                                //DELETEやBackSpaceによる削除で桁あふれが発生しているなら
                                //小数点の削除によるものなので、先頭桁から削除する
                                if (isDelete)
                                {
                                    txtBox.Text = value.Substring(value.Length - LENGTH_INT_PART);

                                    //カーソル位置を移動
                                    if (currentPoint - (values.Length - LENGTH_INT_PART) >= 0)
                                    {
                                        txtBox.SelectionStart = currentPoint -
                                            (values.Length - LENGTH_INT_PART);
                                    }
                                    else
                                    {
                                        txtBox.SelectionStart = 0;
                                    }
                                }
                                else
                                {
                                    //入力後の現カーソル位置を取得
                                    currentPoint = txtBox.SelectionStart;

                                    //カーソル位置の前の１文字が今回入力された文字、
                                    //よって、それを省いた文字列に編集する
                                    var left = value.Substring(0
                                        , currentPoint > LENGTH_INT_PART
                                            ? LENGTH_INT_PART : currentPoint - 1);
                                    var right = left.Length >= LENGTH_INT_PART
                                        ? "" : value.Substring(currentPoint);
                                    txtBox.Text = left + right;

                                    //カーソル位置を入力前の位置に戻す
                                    txtBox.SelectionStart = currentPoint - 1;
                                }
                            }
                            break;






                        //--------------------------------------------------------------------------------------------------------------------------------------
                        case 2:

                            //----------------------------------
                            //整数部＋小数部で構成されている場合
                            //----------------------------------
                            //入力後の現カーソル位置を取得
                            currentPoint = txtBox.SelectionStart;

                            //今回の入力値が"."の場合、小数点を基準点として桁あふれ分を除外する
                            if (value.Substring(currentPoint - 1, 1) == ".")
                            {
                                //=== 整数部の処理 ===
                                var intPart = values[0];
                                if (values[0].Length > LENGTH_INT_PART)
                                {
                                    intPart = values[0].Substring(LENGTH_INT_PART - values[0].Length);
                                }

                                //=== 小数部の処理 ===
                                var decimalPart = values[1];
                                if (values[1].Length > LENGTH_DECIMAL_PLACES)
                                {
                                    decimalPart = values[1].Substring(0, LENGTH_DECIMAL_PLACES);
                                }

                                //整数と小数を結合
                                if (values[0].Length > LENGTH_INT_PART || values[1].Length > LENGTH_DECIMAL_PLACES)
                                {
                                    txtBox.Text = string.Format("{0}.{1}", intPart, decimalPart);

                                    //小数点入力時なら小数点の後ろにカーソルをセット
                                    txtBox.SelectionStart = value.IndexOf(".") + 1;
                                }
                            }
                            else
                            {
                                //=== 整数部の処理 ===
                                var intPart = values[0];
                                if (values[0].Length > LENGTH_INT_PART)
                                {
                                    //カーソル位置の前の１文字が今回入力された文字、
                                    //よって、それを省いた文字列に編集する
                                    var left = values[0].Substring(0
                                        , currentPoint > LENGTH_INT_PART
                                            ? LENGTH_INT_PART : currentPoint - 1);
                                    var right = left.Length >= LENGTH_INT_PART
                                        ? "" : values[0].Substring(currentPoint);

                                    //桁数調整後の整数部文字列
                                    intPart = left + right;
                                }







                                //=== 小数部の処理 ===
                                var decimalPart = values[1];
                                if (values[1].Length > LENGTH_DECIMAL_PLACES)
                                {
                                    //整数部と小数点を除いたときのカーソル位置を算出
                                    var tempPoint = currentPoint - values[0].Length - 1;
                                    //カーソル位置の前の１文字を除外する
                                    var left = values[1].Substring(0
                                        , tempPoint > LENGTH_DECIMAL_PLACES
                                            ? LENGTH_DECIMAL_PLACES : tempPoint - 1);
                                    var right = left.Length >= LENGTH_DECIMAL_PLACES
                                            ? "" : values[1].Substring(tempPoint);

                                    //桁数調整後の小数部文字列
                                    decimalPart = left + right;
                                }

                                //整数と小数を結合
                                if (values[0].Length > LENGTH_INT_PART || values[1].Length > LENGTH_DECIMAL_PLACES)
                                {
                                    txtBox.Text = string.Format("{0}.{1}", intPart, decimalPart);

                                    //カーソル位置を入力前の位置に戻す
                                    txtBox.SelectionStart = currentPoint - 1;
                                }
                            }

                            break;





                        default:
                            //"."で文字列を分割したときに３以上になるのなら、
                            //既に"."が存在しているのに今回"."が入力されたことを示す
                            //よって、今回の入力を無効にしてしまう。
                            {
                                currentPoint = txtBox.SelectionStart;
                                var left = value.Substring(0, currentPoint - 1);
                                var right = value.Substring(currentPoint);
                                txtBox.Text = left + right;
                                txtBox.SelectionStart = currentPoint - 1;
                            }
                            break;
                    }




                    //先頭が小数点なら先頭に 0 を入れる
                    if (txtBox.Text.StartsWith("."))
                    {
                        currentPoint = txtBox.SelectionStart;
                        txtBox.Text = "0" + txtBox.Text;
                        txtBox.SelectionStart = currentPoint + 1;
                    }
                }
            }
            catch (Exception ex)
            {
                Console.Error.WriteLine(ex);
            }
            finally
            {
                txtBox.TextChanged += textHeight_TextChanged;
            }




        }

Comments

[YAmaGNZ]

以前の質問で回答が得られてますが、何が理解できていないのでしょうか？

[xlostdjx]

小数点の指定がまだピンとこなく、、、回答をいただいたのを拝見してもどの部分が小数点を指定しているなどがわからなかったので　例文もしくはなにかしらコードで示していただけたらと思いまして。

[YAmaGNZ]

前回の質問でsaziさんの回答のリンク先にコードがありますが、こちらのコードを実装してデバッガで動作確認したりされましたか？そのコードに小数の桁制限の部分もありますので参考になりますよ。

[xlostdjx]

質問を編集したのでみてアドバイスいただきたいです。

Answer 1

厳しいようですが、デバッガの使い方を勉強して、自分のプログラムがどう動いているか、ステップ実行するなりして確かめてください。
プログラムのデバッグを行う時に必要なことです。
初級者向けVisualStudio便利機能集Vol.2(デバッグ編)などを参照してください。

ヒント：
if (b.Length > 1)
これがどのように判定されるか確認してください。

追記

下記のように、コードでイベントハンドラを登録するか

    textHeight.TextChanged += new System.EventHandler(textHeight_TextChanged);
    textHeight.KeyDown += new System.Windows.Forms.KeyEventHandler(textHeight_KeyDown);
    textHeight.KeyPress += new System.Windows.Forms.KeyPressEventHandler(textHeight_KeyPress);

下図のようにデザイナで登録するかしてください。

動作に関しては、関数入口にブレイクポイントを設定すれば、キー入力した時にそこで停止するはずです。
自分で書いたコードが動作しているのか、していないのか、動作している場合はなぜ思っている動きをしないのかを調査するために最低限必要なことです。
デバッガの使い方を勉強してください。動作を確認してください。としつこく言っているのはこういうことです。

Comments

[xlostdjx]

if (b.Length > 1)　は　ｂに　165.5が入っていると仮定して　.で区切られるので　
165　と　.5　配列が二つなので　ｂ　には2が入っていることがわかりました。　

ただ　
数値とバックスペース以外の入力は不可にする事が適用されていないのはなぜかわかりません。
お答えいただきたいです。

[YAmaGNZ]

最初からテキストボックスに"165.5"と入っているということでしょうか？
最初は「何も入力されていない」のでは？
仮定ではなく、実際に動作させた時の値を確認しましょう。

[xlostdjx]

実際に動かしても　下記のことがらは適用されませんでした。
数値とバックスペース以外の入力は不可にする事が適用されていないのはなぜかわかりません。

[YAmaGNZ]

if (b.Length > 1)
ここにブレークポイントを張って、b.Lengthの値を確認しましたか？
そこからステップ実行して、どのように実行されたか確認しましたか？

[xlostdjx]

はい　確認しやり方を変えました。
コードを編集しました、

[YAmaGNZ]

b.Lengthが1以下の場合、チェックが走りませんがよいのですか？
入力がなにもされていない場合や”1”の場合などだと1以下になりますよ？

[xlostdjx]

はじめから一度 小数点第一位までを入力を可能にするコード書こうとおもいます
これは理解するとかけるようになるなどあったら教えていただきたいです。
もしよろしければ正解のコードを教えていただきたいです。
ほぼコピペで組み立てていたので、。

[YAmaGNZ]

前回の質問時にコードのあるページを回答してもらってるじゃないですか。
適度にコメントもありますし、実際に動作させながら勉強すればよいかと思います。

[xlostdjx]

質問を編集したのでみてアドバイスいただきたいです。

[YAmaGNZ]

まず、「適用されません」というあいまいな表現はやめましょう。
状況をなるべく具体的に説明するのも質問する時に重要なことです。
今回の場合、「数字以外を入力できてしまう」とか書くべきです。

次に、「数字以外が入力できてしまう」ということについてですが、
参考にしたページにあったKeyPressイベントはどうして記述していないのでしょうか？

[xlostdjx]

すみません。コピーが抜け落ちてしまっていました。
KeyPressイベントをつけたしました。
「数字以外が入力できてしまう」というのは反映されていて制御できているのですが、
小数点の制限のところが反映されていないみたいです。

[YAmaGNZ]

こちらで提示されたコードを動作させましたが、小数は1桁しか入力できません。
小数点も１つだけ入力可能です。

もしかして、コードをコピーしただけで、イベント登録してないということはありませんよね？
デバッガで、イベント部分が呼び出されているか確認しましたか？

[xlostdjx]

サイトでみて勉強したのですがいまいちイベントを理解していないです、なのでイベント登録できていないかもしれないです、
イベント部分が呼び出されているかどうかの確認も、どれかわからないので困っています。

[xlostdjx]

あ　イベント意味の理解しました！！
そしたら動くようになりました！！お手数かけてすみません。

[xlostdjx]

あと
ComboBox（丸め区分）をコード上で値の設定を行う事についてなんですけど
おすすめのやり方をおしえていただきたいです。

[MMashiro]

それは複数質問を挙げることになってしまう為混乱を生んでしまうため本質問を解決済みにして新規で質問を立ち上げてください

Answer 2

入力させない、ではなく、入力は自由にさせといて、Validateで小数点１位で取り込み、表示させるというのではダメでしょうか。
取り込み時にエラーが出たら、その旨表示し再入力させる、と。

初心者のうちはそのほうが簡単だと思いますよ

---

KeyPressでごそごそするというのはシュミでないんでちと考えてみました
直前の値を覚えておくためのtextval が必要なのがちとアレですが
あくまで一例ということで

using System.Text.RegularExpressions;


        string textval;
        // TextBoxのTextChange イベント
        private void txTextChange(object sender, EventArgs e)
        {
            TextBox tx = sender as TextBox;
            if (Regex.IsMatch(tx.Text, @"^\d*[.]?\d?$"))
            {
                textval = tx.Text;
            }
            else
            {
                tx.Text = textval;
            }
        }

Comments

[xlostdjx]

身長と体重の、テキストボックス入力時
数値のみ小数点第一位まで入力が可能であること。
バックスペースにて入力値が削除可能であること。= 数値とバックスペース以外の入力は不可
という条件なのですが　.Split 等を使って表した方がいいですか？

[y_waiwai]

「c# 数値のみ」でぐぐって、これをアレンジしよう、ってことで

[xlostdjx]

改善したコードを張ったのでみてほしいです。 また反映されないところがでてきたので

[y_waiwai]

回答を更新しました

[xlostdjx]

ご回答ありがとうございます、
KeyPressを使用するのが条件でして、
KeyPressバージョンをかいていただくことは可能でしょうか？？

[xlostdjx]

質問を編集したのでみてアドバイスいただきたいです。