Równoległe ładowanie picturebox

0

Cześć, mam mały problem, otóż na początku programu (Load mojej formy) muszę załadować dosyć sporo obrazków (koło 180) i trwa to parę sekund, chciałbym aby wywoływało się to w równoległym wątku, aby nie było "laga" aplikacji (tym bardziej, że obrazki są w drugiej zakładce i po starcie programu najpierw trzeba zrobić jakąś operację, aby przejść do nich). Nie wiem jak to ugryźć, nigdy takiego czegoś nie pisałem, oto kod:

private void LoadPicturesColors()
        {
            string exePath = System.Reflection.Assembly.GetExecutingAssembly().GetName().CodeBase;
            string dir = System.IO.Path.GetDirectoryName(exePath);
            string colorsDir = System.IO.Path.Combine(dir, "Colors");
            string[] pngFiles = System.IO.Directory.GetFiles(new Uri(colorsDir).LocalPath, "*.png");
            FileInfo imageInfo;
            for (int i = 0; i < pngFiles.Length; i++)
            {
                this.pictureBoxColors.Add(new PictureBox());
                this.pictureBoxColors[i].SizeMode = PictureBoxSizeMode.Zoom;
                this.pictureBoxColors[i].Size = new Size(150, 150);
                this.pictureBoxColors[i].Image = Image.FromFile(pngFiles[i]);
                this.pictureBoxColors[i].Location = new Point(CalculateImageColorWidth(), CalculateImageColorHeight());
                imageInfo = new FileInfo(pngFiles[i]);
                this.pictureBoxColors[i].Name = imageInfo.Name.Remove(imageInfo.Name.Length - 4);
                this.pictureBoxColors[i].Click += new EventHandler(this.pictureBoxColors_Click);
                this.tabPageColor.Controls.Add(this.pictureBoxColors[i]);

                this.radioButtonColors.Add(new RadioButton());
                this.radioButtonColors[i].AutoSize = true;
                this.radioButtonColors[i].TabIndex = i;
                this.radioButtonColors[i].Name = this.pictureBoxColors[i].Name;
                this.radioButtonColors[i].Location = new Point(CalculateImageColorWidth() + 75, CalculateImageColorHeight() + 150);
                this.tabPageColor.Controls.Add(this.radioButtonColors[i]);
            }
        } 
0

Próbuję użyć BackgroundWorkera, ale otrzymuję System.InvalidOperationException, oto kod:

private void tabControlCalculator_Enter(object sender, EventArgs e)
        {
            //initializing backgroundWorker
            backgroundWorker.DoWork += backgroundWorker_DoWork;
            backgroundWorker.RunWorkerAsync();
        }
        private void backgroundWorker_DoWork(object sender, DoWorkEventArgs e)
        {
            LoadPictures();
            LoadTextBoxes();
        }
        private void LoadPictures()
        {
            LoadPicturesTypes();
            LoadPicturesColors();
        }
        private void LoadTextBoxes()
        {
            LoadTextBoxesTypes();
        }
        private void LoadPicturesTypes()
        {
            string[] namesOfTypes =
            {
                "Prosty",
                "L",
                "U",
                "L z wyspą"
            };
            string exePath = System.Reflection.Assembly.GetExecutingAssembly().GetName().CodeBase;
            string dir = System.IO.Path.GetDirectoryName(exePath);
            string colorsDir = System.IO.Path.Combine(dir, "Types");
            string[] pngFiles = System.IO.Directory.GetFiles(new Uri(colorsDir).LocalPath, "*.png");
            FileInfo imageInfo;
            for (int i = 0; i < this.pictureBoxTypes.Length; i++)
            {
                this.pictureBoxTypes[i] = new PictureBox();
                this.pictureBoxTypes[i].SizeMode = PictureBoxSizeMode.AutoSize;
                this.pictureBoxTypes[i].Image = Image.FromFile(pngFiles[i]);
                this.pictureBoxTypes[i].BackColor = Color.Gray;
                this.pictureBoxTypes[i].Location = new Point(6, CalculateImageHeight(i));
                this.pictureBoxTypes[i].Click += new EventHandler(this.pictureBoxTypes_Click);
                imageInfo = new FileInfo(pngFiles[i]);
                this.pictureBoxTypes[i].Name = imageInfo.Name.Remove(imageInfo.Name.Length - 4);
                this.tabPageType.Controls.Add(pictureBoxTypes[i]);

                this.radioButtonTypes[i] = new RadioButton();
                this.radioButtonTypes[i].AutoSize = true;
                this.radioButtonTypes[i].TabIndex = i;
                this.radioButtonTypes[i].Text = namesOfTypes[i];
                this.radioButtonTypes[i].Click += new EventHandler(this.radioButton_Click);
                this.radioButtonTypes[i].Location = new Point(this.pictureBoxTypes[i].Width + 10, CalculateImageHeight(i) + this.pictureBoxTypes[i].Height / 2);
                this.radioButtonTypes[i].Name = this.pictureBoxTypes[i].Name;
                if (i == 0)
                    this.radioButtonTypes[i].Checked = true;
                this.tabPageType.Controls.Add(radioButtonTypes[i]);
            }
        } 
0

Prawdopodobnie problemem jest, że próbujesz dostać się do kontrolki z innego wątku (workera) niż została utworzona. Jeżeli masz komunikat w stylu Cross-thread operation not valid: Control ‘xxxxx’ accessed from a thread other than the thread it was created on to wystarczy zrobić Invoke na kontrolce z podaną akcją. Tu masz przykład: http://stackoverflow.com/questions/7963103/net-backgroundworker-invalidoperationexception-cross-thread-operation-not-v

Jeżeli inna treść błędu to nie znamy jej, więc na drugi raz od razu prócz typu wyjątku podaj treść.

0

Ok, zmieniłem, ale teraz otrzymuję błąd z wykroczeniem poza zakres tablicy (co jest niemożliwe) http://scr.hu/0u0e/o51hj

0

A jednak możliwe, zazwyczaj to człowiek się myli :) . Parametr i = 4. pictureBoxTypes ma 4 elementy, zapewne lica/tablica pictureBoxTypes jest indeksowana od 0.

0

Pytanie tylko dlaczego i wynosi 4, skoro to nie spełnia warunku w pętli?

0

Źle do tego podchodzisz w mojej ocenie.

Po 1, pisałeś, że obrazki są w drugiej zakładce, BackgroundWorker odpalasz na metodzie Enter, czyli i tak zanim się wyświetli to poczekasz, bo BackgroundWorker mieli, a zacznie mielić jak ktoś otworzy zakładkę (za późno),
Po 2, musisz używać Invoke żeby utworzyć PictureBox-y (w metodzie Enter), mając złączoną logikę ładowania obrazka i tworzenia PictureBox nic Ci ten BackgroundWorker nie da, nadal będzie mulić,
Po 3, chcesz żeby obrazki się załadowały równolegle wtedy kiedy odpala się aplikacja, czyli aplikacja się odpala, user zanim kliknie to w tle ładują się Twoje obrazki, klika na drugą zakładkę i bingo, wszystko jest.

Metodycznie:

  1. Rozdziel logikę ładowania obrazków od tworzenia tych PictureBox, pisałem o tym wcześniej, najpierw ładujesz listę obrazków, potem dla tej listy tworzysz PictureBox-y,
  2. Aplikacja startuje, w tym czasie odpala się MainForm, na zdarzeniu Load MainForm-a odpalasz Task, który ładuje obrazki do IList<Image>

Coś takiego:

Task.Factory.StartNew(() => {
 // 1. Ładuj obrazki.
 string exePath = System.Reflection.Assembly.GetExecutingAssembly().GetName().CodeBase;
 string dir = System.IO.Path.GetDirectoryName(exePath);
 string colorsDir = System.IO.Path.Combine(dir, "Types");
 string[] pngFiles = System.IO.Directory.GetFiles(new Uri(colorsDir).LocalPath, "*.png");
 IList<Image> imageList = new List<Image>();
 foreach(string pngFile in pngFiles)
 {
   imageList.Add(Image.FromFile(pngFile); // Możesz też użyć LINQ, zamiast pętli foreach, była sugestia w poprzednim wątku.
 }
 
 // 2. Tutaj uruchom Invoke, który zbuduje tą Twoję listę PictureBox-ów.
 this.Invoke((MethodInvoker)(() => {
   // Tutaj wykonaj to budowanie PictureBox-ów. this to MainForm.
 })));
});

Mogłem się gdzieś pomylić, pisałem z palca z pamięci.
Co uzyskasz ? Ano to, że aplikacja podczas Load MainForm-a odpali Taska, który najpierw załaduje obrazki nie blokując GUI, a dopiero potem już blokując GUI podepnie pod PictureBox-y, jeżeli user otworzy drugą zakładkę zanim Task zdąży załadować to będzie miał pusto, możesz pomyśleć nad jakimś kodem, który wypisze mu na ekranie "Please wait.... loading" - to jako zadanie domowe. Potem może mu chwilę przyciąć GUI, jeżeli mocno przytnie to pomyśl jak rozbić to rysowanie PictureBox-ów na kilka etapów np. rysuj 10 PictureBox-ów i zrób 50 ms przerwy (w tym Task-u), to już sam ocenisz, bylebyś nie robił Sleep-a w Invoke (!!!).

Jak coś nie jest jasne to pisz.

Pozdrawiam.

1 użytkowników online, w tym zalogowanych: 0, gości: 1