ProgressBar – .NET – Fórum – Programujte.com

Ahoj, 

může mi prosím někdo poradit, proc se mi v kodu nize nerozjede ProgressBar?

Diky za odpověď.

private void button1_Click(object sender, EventArgs e)

        {
            progressBar1.Style = ProgressBarStyle.Marquee;
            progressBar1.MarqueeAnimationSpeed = 100;
            if (ComboBox1.SelectedIndex != -1)
            {
                List<object> Arguments = new List<object>();
                Arguments.Add(ComboBox.SelectedItem);
                Arguments.Add(TextBox.Text);
                Arguments.Add(TextBox.Text);
                backgroundWorker1.RunWorkerAsync(Arguments);
            }
        }

        private void DoWork(object sender, DoWorkEventArgs e)
        {
            List<object> GenericList = (List<object>)e.Argument;
            Object object = (Glass)GenericList[0];
            int cislo = GenericList[2].ToString() == string.Empty ? 1 : Convert.ToInt32(GenericList[2]);
            double jinecislo = GenericList[1].ToString() == string.Empty ? 0 : Convert.ToDouble(GenericList[1]);
            e.Result = Class2.Calculate(object, cislo, druhecislo);
        }

       private void RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            DataGridView.DataSource = (DataTable)e.Result;
            progressBar1.MarqueeAnimationSpeed = 0;

        }

Není možné, že ti prostě ten worker skončí rychle?

Ja myslim, ze ne. Nez se nactou data do tabulky trva pomerne dlouho. 

#1 Frank
Backgroudworker je standardní vlákno jako jakékoliv jiné akorát že se menuje Backgroudworker. Nemuže se ti prg.bar. rozject protože prostě neni čas na obnovování při přepínání vláken - pokud je poloha naplnění správně že - to tu vidět neni. Co třeba ProgressBar.Step ?

https://docs.microsoft.com/cs-cz/dotnet/api/system.windows.forms.progressbar.step?view=netframework-4.8

Taky musíš někam vložit Application.DoEvents() asi do DoWork. a pak taky je potřeba dát ProgressBar.Refresh() a možná i vyvolat Refresh nadřazené komponenty. WinForms jsou komplikovaný. Já bejt tebou bych přešel z WinForms na WPF.

jinak ten tvuj zápis kodu:

int cislo = GenericList[2].ToString() == string.Empty ? 1 : Convert.ToInt32(GenericList[2]);
double jinecislo = GenericList[1].ToString() == string.Empty ? 0 : Convert.ToDouble(GenericList[1]);

to je fakt tragédie ... ty asi budeš mladší ročník co ? ty ses učil programovat někde z nějaký "moderní" učebnice co ? ...

1/ deklarace+definice proměnných je vždy zvlášť na začátku metody pokud k tomu nemáš zvláštní důvody.

2/ použití ? místo if je spíše otázka C/C++ a moc se nepoužívá protože se fakt blbě čte.

3/ GenericList[2].ToString() == string.Empty  ?? to je co ??? přeci si deklaroval že

List<object> Arguments = new List<object>(); ?? ne ?? a přitom tam dáváš různý objekty

ty děláš konverzi na string ??? to nechápu ... a co použít GetType() ??? tohle skončí katastrofou ...

4/ progress bar inicializace:

            this.progressBar2.Visible = true;
            this.progressBar2.Minimum = minLimit;
            this.progressBar2.Maximum = (Int32)maxLimit;
            this.progressBar2.Value = 1;
            this.progressBar2.Step = 1;
            this.progressBar2.Refresh();

progresbar ta část co určuje pozici posuvníku v cyklu:

                this.progressBar2.Value = (Int32)limit; // aktualni pozice
                this.progressBar2.PerformStep();
                this.progressBar2.Refresh();

a ještě drobnost .... používej MS VS 2019.

#4 jerry
BackgroundWorker běží na pozadí v samostatném vlákně a UI neblokuje.

Marquee ProgressBar se spouští a zastavuje, ale nekrokuje, protože neindikuje konkrétní pozici.

Application.DoEvents je vždycky špatně. Dá se doporučit jen jako prasácká nouzovka ve stylu "to mu tady zatím stačí". Tady je to nesmysl.

#5 gna, Frank
týjo .. taková kritika :)))))  DoEvents() není špatný .. ale musíš vědět JAK a hlavně KDE to použít. Zjednodušuje to kod, pokud ste odrostli na assembleru a znáte jak funguje procesor a dáte si disassebmlovat C# .NET kod uvidíte že doevents nedělá nic špatnýho ...zkuste si někdy spustit profiler, je mnohem horší mít "seklej" thread na pozadí než sem tam vložit intelignetně doevents. WinForms jako visuální prvky sou jednovlánkový ... i když rozdělíš chod na jiné vlákno stejně se čeká na zprávu z toho vlákna formou callbacku jinak se ti to sekne což je to co se ti stalo.

https://blog.codinghorror.com/is-doevents-evil/

https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.application.doevents?redirectedfrom=MSDN&view=netframework-4.8#System_Windows_Forms_Application_DoEvents

#6 jerry
To byly jen faktické poznámky.

A zase jsi napsal krávovinu. Ty vždycky píšeš jen krávoviny.

#7 gna
njn je to se mnou špatný, jako programátor sem úplně nahovno :))) tak to je ...  vrátim se zpátky do školy začnu u basicu  :)))

Zkus to podle ukázkového příkladu:  https://docs.microsoft.com/en-us/dotnet/api/system.componentmodel.backgroundworker?view=netframework-4.8  Jestli jsem to dobře pochopil, na aktualizaci progressbaru je potřeba vytvořit ProgressChangedEventHandler.

hu

#9 hlucheucho 

Ahoj

kvuli tomu jsem se snazil pouzit Marquee ProgressBar, abych nemusel vytvaret ProgressChangedEventHandler.

Kazdopadne diky za odpoved, zkusim si to precist. 
 

Dekuju vsem za odpovedi, 

prisel jsem na to, ze pravdu mel gna. Proste to skoncilo prilis rychle. Nejdelsi operace je  DataGridView.DataSource = (DataTable)e.Result, ktera uz bezi v RunWorkerCompleted. Myslel jsem, ze dlouhy je vypocet radku tabulky, ne zapsani DataTable do DataGridView. Zbyva vyresit, jak pristoupit k datagridview z DoWork. Nebo tam proste dat presypaci hodiny a nasrat. 

Jeste se teda podivam na int cislo = GenericList[2].ToString() == string.Empty ? 1 : Convert.ToInt32(GenericList[2]). Opravdu se ucim programovat z ucebnice. Do List<object> Arguments predavam jeden objekt a dva stringy z textboxu. Myslel jsem, ze to pak musim zpatky prevest. 

You must be careful not to manipulate any user-interface objects in your DoWork event handler. Instead, communicate to the user interface through the BackgroundWorker events.

Když se podíváš do dokumentace, události jsou 3:
- DoWork je samotná práce BackGroundWorkeru
- ProgressChanged slouží ke komunikaci s GUI za běhu, ukázky většinou posouvají ProgressBar, ale může se tam provádět i jiná akce
- RunWorkerCompleted je volána po skončení práce BackGroundWorkeru a umožňuje předat GUI výsledek

 hu