Подскажите пожалуйста в чем ошибка

[Antony22]

Доброго времени суток пытаюсь сделать прогресбар для функции подскажите пожалуйста где ошибся:

public delegate void ModifyProgressBarDelegate(int max, int val);
        public static void ModifyProgressBarMethod(int max, int val)
        {
            progressBar1.Maximum = max; //ругается на progressBar1
            progressBar1.Value = val;
        }
        protected static List<string> FileINFolder(string patch)
        {
            var dir = new DirectoryInfo(patch);
            var files = new List<string>();
            bool x = false;
            int max = files.Count;
            int val = 0;
            Thread b = new Thread(() => progressBar1.Invoke(new ModifyProgressBarDelegate(ModifyProgressBarMethod), max, val));
            b.Start();
            foreach (FileInfo file in dir.GetFiles())
            {
                if (file.Extension == ".xml")
                {
                    x = true; 
                    files.Add(file.FullName);
                }
                val++;

            }
            if (x == false)
            {
                System.Windows.MessageBox.Show("Не найдены файлы!");
            }

            return files;
        }

[Stilet]

Если я правильно понимаю у тебя в Thread идет анонимная функция, где нет цикла.
Она отрабатывает один раз progressBar1.Invoke и тред завершается.
Думаю тебе нужно в треад запулить сам цикл, и в нем вызывать прогрессбарное отрисовывание.

[Antony22]

Сообщение от Stilet

Если я правильно понимаю у тебя в Thread идет анонимная функция, где нет цикла.
Она отрабатывает один раз progressBar1.Invoke и тред завершается.
Думаю тебе нужно в треад запулить сам цикл, и в нем вызывать прогрессбарное отрисовывание.

Да было бы неплохо вот только если можно примерчик пожалуйста а я просто смутно представляю как это сделать.

[Stilet]

Увы я с Шарпом не дружу настолько.
Основные принципы знаю, а примеры из MSDN черпаю.

[Luuzuk]

Я верно понимаю, что у вас WPF или Silverlight приложение?

[Antony22]

Сообщение от Stilet

Увы я с Шарпом не дружу настолько.
Основные принципы знаю, а примеры из MSDN черпаю.

Жаль, все равно спасибо хоть буду знать в каком направлении капать
Я пытался и сейчас пытаюсь найти на MSDN подходящий пример но пока безуспешно. Возможно просто ищу плохо или не понимаю до конца.

Сообщение от Luuzuk

Я верно понимаю, что у вас WPF или Silverlight приложение?

Верно у меня WPF приложение.

[Luuzuk]

1) В отдельный поток следует вынести процесс поиска файлов и заполнения списка "files" (иначе UI не будет ни на что реагировать в процессе поиска, в т.ч. не будет отображать прогресс на progressbar'е). Кстати, проще использовать "Directory.EnumerateFiles(path, "*.xml")"

2) У прогрессбара свойства "Max" и "Value" забиндить на соответствующие свойства, которые вы создадите в вашем классе. Для обновления прогрессбара достаточно будет сгенерировать событие NotifyPropertyChanged с указанием имени свойства "Value"

3) Все это изначально не имеет ни малейшего смысла, т.к. вся долгая работа по поиску файлов выполняется при вызове "dir.GetFiles()", работу которого по прогрессбару не отследить. Все остальное (в т.ч. цикл, в к-ром вы перебираете найденные файлы) выполняется за миллисекунды => прогрессбар никто и увидеть не сумеет

P.S. да и прогрессбар у вас всегда будет показывать 100% из-за того, что вместо "int max = files.Count;" нужно было использовать "dir.GetFiles().Length". Хотя согласно п.3 ничего использовать и вовсе не надо было

[Antony22]

Luuzuk, большое вам спасибо за доходчивое объяснение попробую переписать функцию согласно вашим советам.