C＃Winforms：この単純なカスタムテキストボックスコードを改善する方法はありますか？

Question

私は英数字のみを許可するカスタムTextBoxコントロールに取り組んでいます。 KeyPress、KeyUpイベントの文字を制限していないことに注意してください。コピー/ペーストやその他の操作を制限したくないので、一般的に許可する必要があります。私はペースト操作時に英数字以外の文字をトリミングしています。しかし、デスクトップアプリケーションの経験がほとんどないので、私が書いたコードが良いか悪いかはわかりません。

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

namespace SaintThomas.UserControls 
{ 
    class TextBoxForUserName : TextBox 
    { 
     protected override void OnTextChanged(EventArgs e) 
     { 
      base.OnTextChanged(e); 

      this.SuspendLayout(); 
      int startPos = this.SelectionStart; 
      if (Regex.IsMatch(this.Text, "[^0-9_A-Z]", RegexOptions.IgnoreCase)) 
      { 
       int reduceStartPos = this.Text.Length; 
       this.Text = Regex.Replace(this.Text, "[^0-9_A-Z]", "", RegexOptions.IgnoreCase); 
       startPos = (startPos <= 0) ? 0 : startPos - (reduceStartPos - this.Text.Length); 
       if (this.Text.Length < startPos) 
       { 
        startPos = this.Text.Length; 
       } 
       this.SelectionStart = startPos; 
      } 
      this.ResumeLayout(); 
     } 
    } 
} 

Answer 1

これを行う最良の方法は、MaskedTextBox controlを使用することです。

車輪を作り直す必要はありません。

Answer 2

一見すると、最初のifの状態は、それを残しておけば同じことをするので全く必要ないようです。