windowsフォームのテキストボックス（金額を入力）3桁で区切りたい

Question

お世話になります。

windowsフォームでテキスト金額を入れる部分に3桁で区切りを入れたいのですが、色々なサイト等のコードを参考にしているのですが、うまくいきません。

該当のイベントのコードを貼りますので、修正点をご教示いただけますでしょうか。
よろしくお願いします。

        private void textsikyou_Validated(object sender, EventArgs e)
        {
            var str = textsikyou.Text;
            string.Format("{0:#,##0}", str);
            textsikyou.Text = str;
        }

Answer 1

こういう方法もあります。フォーム名を Form1 と仮定します。

まず次のクラスを作ります。

public class Form1Data
{
	public int Money { get; set; }
}

これを一度ビルドします。
次にフォームをデザイナで表示し、該当するテキストボックスを選択します。
プロパティグリッドで「データ」カテゴリの「(DataBindings)」を展開し、「(詳細)」を選択し、「…」をクリックします。

「フォーマットと詳細バインド」ダイアログが開かれます。
「プロパティ」で「Text」を選択し、「バインド」のドロップダウンを開きます。
「プロジェクトデータソースの追加」をクリックし、「オブジェクト」を選択して「次へ」
「どのオブジェクトにバインドしますか？」で「Form1Data」にチェックを入れて「完了」
出てきた選択肢から「Money」を選択。
「形式の種類」を「カスタム」にし、「カスタム書式」に「#,##0」を入力して「OK」

Form1.Load イベントで次のように記述します。

private void Form1_Load(object sender, EventArgs e)
{
	form1DataBindingSource.DataSource = new Form1Data();
}

以上です。

Comments

[Zuishin]

データバインディングという手法ですが、これを使うとユーザーインターフェース(Form1)と実際のデータ(Form1Data)が分離されるので、保守しやすくなるという利点があります。例えば Windows.Forms を WPF に変えても Form1Data はそのまま使えるので、そのデータを実際に使った演算部分には手を入れる必要がありません。

Answer 2

textsikyou.Textは文字列型のプロパティなので、var strは文字列型の変数になります。
{0:#,##0}は文字列型の書式指定ではなく、数値型の書式指定です。
{0:#,##0}の書式指定を使うのであれば、引数は数値型出なくてはなりません。Int32.Parse等の型変換命令を使用して数値型に変換して与える必要があります。

Srting.Formatは文字列型を返す関数ですが、帰ってきた値を変数などに格納せずに捨ててしまっています。適当な変数に保存するなり、返値をTextプロパティに代入するなりしてください。

以上を直すと次の様なコードになります。

        private void textsikyou_Validated(object sender, EventArgs e)
        {
            var str = textsikyou.Text;
            var a = string.Format("{0:#,##0}", Int32.Parse(str));
            textsikyou.Text = a;
        }

実際にはコレに加えて、textsikyou.Textが数値に変換できない場合のエラー処理などを記載する必要があるでしょう。