C#ListBox Elemento seleccionado Excepción nula

nullreferenceexception

Question

El usuario puede hacer clic en un elemento en un ListBox de la siguiente manera:

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)
......
}

Todo está bien. Sin embargo, si el usuario hace clic en un espacio vacío en el ListBox, muestra el siguiente error:

System.NullReferenceException

Esto es debido a esta línea de código:

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

¿Alguien tiene un trabajo inteligente? ¿O sugerir una alternativa a mi código?

Answer 1

¿Qué hay de hacer una

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

eso debería evitar que ocurra ese error.

Answer 2

Puede verificar SelectedIndex antes de esa línea:

if(listBox2.SelectedIndex < 0)
return;

Answer 3

Qué tal si

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

Código completo

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)
...... }
}
}

Answer 4

simplemente verifique primero si listbox1.SelectedItem está vacío o no antes de llamar a esta línea:

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

Answer 5

Sin embargo, ninguno de estos le permite regresar y volver a seleccionar en el cuadro de lista. Todavía estoy trabajando en ese tema - se actualizará.

while (user == null) {
try {
user = this.lstAdministratorName.SelectedItem.ToString();
} catch {
lstAdministratorName.ClearSelected();
return;
}
}

Answer 6

La solución consiste en comprobar si hay un valor nulo y salir antes de tiempo.

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

Esto evita el anidamiento introducido por las otras respuestas, lo que hace que el código sea menos legible.