0

В общем, есть ListBox, мне нужно проверить, если в нем есть элементы, то должна выполниться функция, если он пустой, то ничего не должно происходить.

Вот код:

private void MusicPlayerButtonNext_Click(object sender, EventArgs e)
{
    if(MusicPlayerListBox.Items != 0)
    {
        if (Main.Files.Count > Main.CurrentTrackNumber + 1)
        {
            ClassMusicPlayer.Play(Main.Files[++Main.CurrentTrackNumber], ClassMusicPlayer.Volume);
            ClassMusicPlayer.EndPlayList = false;
        }
        else
            ClassMusicPlayer.EndPlayList = true;
        if (ClassMusicPlayer.EndPlayList)
        {
            MusicPlayerButtonStop_Click(this, new EventArgs());
            MusicPlayerListBox.SelectedIndex = Main.CurrentTrackNumber = 0;
            ClassMusicPlayer.EndPlayList = false;
        }
    }
}

Ошибка в строке: if(MusicPlayerListBox.Items != 0). Как мне сделать проверку?

7
  • WinForms или WPF?
    – VladD
    2 июн 2017 в 11:13
  • текст ошибки можно опубликовать?
    – Gardes
    2 июн 2017 в 11:13
  • @Gardes: MusicPlayerListBox.Items != 0 — несоответствие типов, думаю.
    – VladD
    2 июн 2017 в 11:14
  • @VladD WinForm. Да, не соответствие типов
    – Sanvirtus
    2 июн 2017 в 11:14
  • 1
    Ну так понятно. MusicPlayerListBox.Items — это ж не число.
    – VladD
    2 июн 2017 в 11:15

2 ответа 2

1
private void MusicPlayerButtonNext_Click(object sender, EventArgs e)
{
    if(MusicPlayerListBox.Items.Count != 0)
    {
        if (Main.Files.Count > Main.CurrentTrackNumber + 1)
        {
            ClassMusicPlayer.Play(Main.Files[++Main.CurrentTrackNumber], ClassMusicPlayer.Volume);
            ClassMusicPlayer.EndPlayList = false;
        }
        else
            ClassMusicPlayer.EndPlayList = true;
        if (ClassMusicPlayer.EndPlayList)
        {
            MusicPlayerButtonStop_Click(this, new EventArgs());
            MusicPlayerListBox.SelectedIndex = Main.CurrentTrackNumber = 0;
            ClassMusicPlayer.EndPlayList = false;
        }
    }
}
2
  • Для того чтобы не строить египетские пирамиды из вложенных блоков условия, можно изменить условие на противоположное и выйти из метода. ИМХО, код более читаемый, что проще и вам, и тем кто будет на этот код смотреть. Тоесть, первый блок условия заменить на if(MusicPlayerListBox.Items.Count == 0) { return; } а дальше по вашему тексту. И пропадает целый уровень вложенности.
    – test123
    2 июн 2017 в 13:24
  • @test123 спасибо, учту. действительно так выглядит намного лучше)
    – Sanvirtus
    3 июн 2017 в 6:05
0
if (MusicPlayerListBox.Items.Count > 0)
1
  • И чем этот ответ отличается от другого, уже существующего ответа?
    – VladD
    3 июн 2017 в 11:41

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.