Levée d'une exception lors de la tentative de voir si un élément est sélectionné dans une zone de liste

Question

Je tente d'élever une exception si un utilisateur clique sur le bouton Supprimer mais ne sélectionne pas un élément dans la zone de liste.

C'est ce que j'ai:

try 
{ 
    if (dataListBox.SelectedIndex == null) 
     throw new Exception; 

    //deletes selected items 
    dataListBox.Items.RemoveAt(dataListBox.SelectedIndex); 
    isDirty = true; 
} 
catch (Exception err) 
{ 
    //spits out the errors if there are any 
    MessageBox.Show(err.Message, "Enter something into the txtbox", MessageBoxButtons.OK, MessageBoxIcon.Error); 
} 

Answer 1

Utilisez

SelectedIndex == -1 

savoir que rien n'est sélectionné.

Answer 2

Vous avez vraiment pas besoin de lancer une exception ici, vous pourriez faire ce qui suit:

if (dataListBox.SelectedIndex == -1) 
{ 
    MessageBox.Show(err.Message, "Enter something into the txtbox", MessageBoxButtons.OK, MessageBoxIcon.Error) 
} 
else 
{ 
    //deletes selected items 
    dataListBox.Items.RemoveAt(dataListBox.SelectedIndex); 
    isDirty = true; 
} 

Answer 3

Vous ne devez pas utiliser d'exceptions pour vérifier les valeurs.

faire quelque chose comme:

if (dataListBox.SelectedIndex >= 0) 
{ 
    //standard flow code 
} 
else 
{ 
    MessageBox.Show("Enter something into the txtbox",_ 
MessageBoxButtons.OK, MessageBoxIcon.Error); 
} 

à la place.

Answer 4

Le test est incorrect, vérifiez pour -1. Et il manque les parenthèses, lancez une nouvelle exception().

Il s'agit cependant d'un code très inapproprié de plusieurs façons différentes. Cela commence par lancer Exception au lieu d'un objet de classe dérivée d'exception spécifique. Votre clause de capture va attraper chaque exception, y compris ceux lancés par d'autres mésaventures comme un bug dans votre code. Vous dites à l'utilisateur d'entrer quelque chose quand elle l'a fait.

De plus, les exceptions seulement doivent être utilisées dans des circonstances exceptionnelles. Il n'y a rien d'exceptionnel à ce que l'utilisateur oublie de faire quelque chose. Affichez simplement le message dans l'instruction if(), faites le reste dans la clause else.

En outre, vous ne devriez même pas permettre à l'utilisateur de tomber dans ce piège. Définissez uniquement la propriété Enabled du bouton Supprimer sur true lorsque vous voyez qu'elle a sélectionné quelque chose.