Как проверить данные из ListBox в C#?

Question

[Леонид]

Добрый день,
Прошу подсказать, как решить проблему.
Имею утилитку, которая должна из списка поочередно проверить список IP адресов. IP адреса добавляются через открытие текстового файла. Как после загрузки в ListBox данных начать их обрабатывать?

namespace AOConfig
{
    public partial class MassScript : Form
    {
        
        public MassScript()
        {
            InitializeComponent();
        }

        private void OpenFileBtn_Click(object sender, EventArgs e)
        {
            SourceList.Items.Clear();
            using (OpenFileDialog ofd = new OpenFileDialog() { Filter = "Text Documents(*.txt)|*.txt", ValidateNames = true, Multiselect = false })
            {
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    string[] lines = System.IO.File.ReadAllLines(ofd.FileName);
                    List<string> list = new List<string>();
                    foreach(string IPAddr in lines)
                    {
                        list.Add(IPAddr);
                        SourceList.Items.Add(IPAddr);
                        Count_label.Text = SourceList.Items.Count.ToString();
                    }
                }
            }
        }

        private void CheckBtn_Click(object sender, EventArgs e)
        {
            Ping ping = new Ping();
        }
    }
}

Answer 1

[Леонид]

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.IO;
using System.Net.NetworkInformation;

namespace AOConfig
{
    public partial class MassScript : Form
    {
        
        public MassScript()
        {
            InitializeComponent();
        }

        private void OpenFileBtn_Click(object sender, EventArgs e)
        {
            SourceList.Items.Clear();
            using (OpenFileDialog ofd = new OpenFileDialog() { Filter = "Text Documents(*.txt)|*.txt", ValidateNames = true, Multiselect = false })
            {
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    string[] switchList = System.IO.File.ReadAllLines(ofd.FileName,Encoding.UTF8);
                    SourceList.Items.AddRange(switchList);
                    TotalCount_label.Text = SourceList.Items.Count.ToString();
                }
            }
        }

        private void StartPingBtn_Click(object sender, EventArgs e)
        {
            Ping switchPing = new Ping();
            PingReply pingReply = null;
            PingOptions options = new PingOptions();
            options.DontFragment = true;
            // options.Ttl = 64;
            int timeout = 1000;
            foreach (var Item in SourceList.Items)
            {
                string Host = SourceList.GetItemText(Item);
                byte[] buffer = Encoding.ASCII.GetBytes(Host);
                pingReply = switchPing.Send(Host, timeout, buffer, options);

                if (pingReply.Status == IPStatus.Success)
                {
                    LiveList.Items.Add(Host);
                    LiveCount_label.Text = LiveList.Items.Count.ToString();
                }
                else
                {
                    continue;
                }
            }
        }
    }
}

Answer 2

[yuopi]

Вынесите из метода ваш лист:
List<string> list = new List<string>();
можно в этот же класс формы. Его и перебирайте из другого метода

Comments

[Леонид]

можете показать кусок кода как это должно выглядеть?
ничего же непонятно

Answer 3

[SlNik]

Вот держи подробный пример:

namespace TestWindowsForms
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            SourceList.Items.Clear();
            using (OpenFileDialog ofd = new OpenFileDialog() { Filter = "Text Documents(*.txt)|*.txt", ValidateNames = true, Multiselect = false })
            {
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    string[] lines = { "sdfs", "sdfesd", "dfefgse" };

                    // Если List<string> не используешь убери
                    List<string> list = new List<string>();
                    list.AddRange(lines);

                    SourceList.Items.AddRange(lines);
                    Count_label.Text = SourceList.Items.Count.ToString();

                    // Избавляемся от не нужноно нам цикла
                    //foreach (string IPAddr in lines)
                    //{
                    //    list.Add(IPAddr);
                    //    SourceList.Items.Add(IPAddr);
                    //    Count_label.Text = SourceList.Items.Count.ToString();
                    //}
                }
            }
        }

        public void TestingIp()
        {
            object delItem = null;
            foreach (var item in SourceList.Items)
            {
                // Тут делаешь всё что тебе вздумается

                // Получаем текст элемента
                string textItem = SourceList.GetItemText(item);

                // Хотим удалить определённый элемент
                if (textItem == "sdfs")
                    delItem = item;
            }

            // Удаляем 
            SourceList.Items.Remove(delItem);

        }

        private void button_Click(object sender, EventArgs e)
        {
            TestingIp();
        }
    }
}

Comments

[Леонид]

Спасибо большое, ваш ответ очень помог!
В итоге код получился таким.

Не подскажете, почему результат проверки записывается рандомно (каждое нажатие на "Ping" показывает разное количество адресов и ни разу все не показало), хотя IP в списке все доступны

namespace AOConfig
{
    public partial class MassScript : Form
    {
        
        public MassScript()
        {
            InitializeComponent();
        }

        private void OpenFileBtn_Click(object sender, EventArgs e)
        {
            SourceList.Items.Clear();
            using (OpenFileDialog ofd = new OpenFileDialog() { Filter = "Text Documents(*.txt)|*.txt", ValidateNames = true, Multiselect = false })
            {
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    string[] switchList = System.IO.File.ReadAllLines(ofd.FileName,Encoding.UTF8);
                    SourceList.Items.AddRange(switchList);
                    TotalCount_label.Text = SourceList.Items.Count.ToString();
                }
            }
        }

        private void StartPingBtn_Click(object sender, EventArgs e)
        {
            Ping switchPing = new Ping();
            PingReply pingReply = null;
            PingOptions options = new PingOptions();
            options.DontFragment = true;
            // options.Ttl = 64;
            int timeout = 100;
            foreach (var Item in SourceList.Items)
            {
                string Host = SourceList.GetItemText(Item);
                byte[] buffer = Encoding.ASCII.GetBytes(Host);
                pingReply = switchPing.Send(Host, timeout, buffer, options);

                if (pingReply.Status == IPStatus.Success)
                {
                    LiveList.Items.Add(Host);
                    LiveCount_label.Text = LiveList.Items.Count.ToString();
                }
                else
                {
                    break;
                }
            }
        }
    }
}

[SlNik]

Леонид, К сожалению сейчас нет компьютера под рукой, но возможно проблема в том, что у тебя стоит условие else в цикле foreach. То есть если хотя бы один из ip адресов имеет статус не IPStatus.Success то все остальные ip адреса не будут обработаны в следствии выполнения оператора break который в свою очередь завершает выполнения цикла foreach.
А разный список каждый раз возможно из-за того, что время выделенное тобой для получения ответа (timeout = 100) используется как условие, если пакет не успеет за 100 мс дойти до указанного IP адреса и вернуться обратно то он уже будет считаться не Success. А условие может не выполняться из-за того, что нагрузка на маршрутизатор, коммутатор бывает разная, иногда пакет успевает проскочить за указанное время, а иногда не успевает. Увеличь время timeout и убери условие else.

Вот пример когда на один и тот же ip адрес, но с разным временем ping проходит и не проходит:

[Леонид]

SlNik, Да, это помогло, спасибо