C# ListBox Élément Sélectionné À L'Exception De Null

L'utilisateur peut cliquer sur un élément dans une zone de liste comme suit:

private void listBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    textBox2.Clear();
    listBox2.Items.Clear();

    string[] p = 
         Directory.GetFiles(
             textBoxDir.Text, 
             listBox1.SelectedItem.ToString(),
             SearchOption.AllDirectories);

    foreach (string open in p) 
        ......
}

Tout est très bien. Toutefois, si l'utilisateur clique sur un espace vide dans la zone de liste, il affiche l'erreur suivante:

System.NullReferenceException

C'est à cause de cette ligne de code:

string[] p = 
    Directory.GetFiles(
        textBoxDir.Text, 
        listBox1.SelectedItem.ToString(),   
        SearchOption.AllDirectories);

Quelqu'un aurait-il un travail intelligent? Ou suggérer une aalternative à mon code?

Avez-vous déboguer votre code?

OriginalL'auteur Nigel.Harris | 2013-04-27

  1. 8

    La solution de contournement consiste à vérifier une valeur null, et la sortie précoce.

    if (listBox1.SelectedItem == null)
{
    return;
}

    Cela permet d'éviter la nidification introduit par les autres réponses, ce qui rend le code moins lisible.

    OriginalL'auteur devdigital

  2. 4

    Vous pouvez vérifier SelectedIndex avant que de ligne:

    if(listBox2.SelectedIndex < 0)
    return;

    OriginalL'auteur Hossein Narimani Rad

  3. 1

    Comment faire un

    if(listBox1.SelectedItem != null){
    //... do your work with listBox1.SelectedItem here
}

    ce qui devrait éviter que l'erreur se produise.

    OriginalL'auteur mauris

  4. 1

    Comment sur

    if (listBox1.SelectedItem != null)
{
    string[] p = Directory.GetFiles(textBoxDir.Text, listBox1.SelectedItem.ToString(),   SearchOption.AllDirectories);
}

    Code Complet

    private void listBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    if (listBox1.SelectedItem != null)
    {    
        textBox2.Clear();
        listBox2.Items.Clear();

        string[] p = Directory.GetFiles(textBoxDir.Text, listBox1.SelectedItem.ToString(), SearchOption.AllDirectories);

        foreach (string open in p) 
         ...... }
    }
}

    OriginalL'auteur Nikhil Agrawal

  5. 1

    viens de vérifier d'abord si listbox1.SelectedItem est vide ou n'est pas avant l'appel à cette ligne:

    string[] p = Directory.GetFiles(textBoxDir.Text, listBox1.SelectedItem.ToString(),   SearchOption.AllDirectories);

    OriginalL'auteur lem.mallari

  6. -1

    Toutefois, aucun de ceux-ci vous permettent de revenir en arrière et re-sélectionner dans la zone de liste. Travaille encore sur cette question - sera mise à jour.

    while (user == null) {
    try {
        user = this.lstAdministratorName.SelectedItem.ToString();
    } catch { 
        lstAdministratorName.ClearSelected(); 
        return; 
    }
}
    while (utilisateur == null) { try { utilisateur = ce.lstAdministratorName.SelectedItem.ToString(); } catch { lstAdministratorName.ClearSelected(); return; } }
    Désolé, nouveau sur le forum et n'arrive pas à comprendre comment ajouter un code. Il fonctionne bien.
    J'ai édité ton post avec votre commentaire. Pensez à lire ce Markdown aide pour formater correctement post sur un Débordement de Pile.

    OriginalL'auteur fred conklin