2010-12-07 3 views
2

次の手法を使用して複数の項目をDropDownListから重複なしのTextBoxに選択しましたが、最も適切な方法であるとは思われません。DropDownListから複数の項目をTextBoxに選択します。重複はありません

alt text

を選択したのと同じ値を再び押す

alt text

ここDDLと押しボタンから

alt text

を別の値を選択してください私のコード

です
Protected Sub Button2_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles Button2.Click 
    If TextBox2.Text.Contains(DropDownList1.SelectedItem.Text) Then 
     Dim m As New Label 
     m.Text = "duplicate !" 
     Me.form1.Controls.Add(m) 
     Exit Sub 

    End If 
    If TextBox2.Text = "" Then 
     TextBox2.Text = DropDownList1.SelectedItem.Text 
    Else 
     TextBox2.Text = TextBox2.Text + " , " + DropDownList1.SelectedItem.Text 
    End If 
End Sub 

答えて

2

ロジックは私にはかなり正しいように見えます。私は、構築物およびIf..else if...elseを行い、早期復帰を避けたい

Protected Sub Button2_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles Button2.Click 
    If TextBox2.Text.Contains(DropDownList1.SelectedItem.Text) Then 
    Dim m As New Label 
    m.Text = "duplicate !" 
    Me.form1.Controls.Add(m) 
    Else If TextBox2.Text = "" Then 
    TextBox2.Text = DropDownList1.SelectedItem.Text 
    Else 
    TextBox2.Text = TextBox2.Text + " , " + DropDownList1.SelectedItem.Text 
    End If 
End Sub 

:唯一のことは、私のような別の何かをしたいです。明確にするために、ただ1つの出口点を持つ関数をプログラムする方が望ましい。この場合には理由がありません。

関連する問題