[C#] Accesso a Strumenti fuori dal Thread

[race2]

Salve,
devo creare un nuovo Thread da "Form1" e gestire l'avanzamento della ProgressBar che si trova in "Form1" dal nuovo Thread, come posso fare ???

namespace Test_Thread
{
    public partial class Form1 : Form
    {
        Thread t;


        public Form1()
        {
            InitializeComponent();
        }

        private void btnStart_Click(object sender, EventArgs e)
        {
            DataThread data = new DataThread(progressBar1);

            t = new Thread(new ParameterizedThreadStart(TestThread));
            t.Start(data);
        }

        public void TestThread(object data)
        {
            if (data != null)
            {
                ((DataThread)data).PBar.Maximum = 100;

                int n = 1000;

                for (int i = 0; i <= n; i++)
                {
                    ((DataThread)data).PBar.PerformStep();
                }
            }
        }
    }



    class DataThread
    {
        private ProgressBar m_oPb;

        public DataThread(ProgressBar _oPb)
        {
            m_oPb = _oPb;
        }

        public ProgressBar PBar
        {
            get { return m_oPb; }
        }
    }
}

mi restituisce questo errore:

Cross-thread operation not valid: Control 'progressBar1' accessed from a thread other than the thread it was created on.

[giangian2]

Ciao,
non è possibile modificare l'interfaccia grafica da un'altro thread.
Dai un'occhiata alla classe BackgroundWorker.

ciao ciao

[race2]

Ho risolto con questo metodo, cosa ne dici ??
la differenza tra quello ceh mi hai detto tu e questo ???

namespace Test_Thread
{
    public partial class Form1 : Form
    {
        Thread t;


        public Form1()
        {
            InitializeComponent();
        }

        private void btnStart_Click(object sender, EventArgs e)
        {
            progressBar1.Minimum = 0; 
            progressBar1.Maximum = 100;

            //DataThread data = new DataThread(progressBar1);

            t = new Thread(new ParameterizedThreadStart(TestThread));
            t.Start();
        }

        public void TestThread(object data)
        {
            if (data == null)
            {
                for (int i = 0; i <= progressBar1.Maximum; i++)
                {
                    ActuallyPerformStep(progressBar1);
                }
            }
        }

        delegate void CallPerformStep(ProgressBar myProgressBar);

        private void ActuallyPerformStep(ProgressBar myProgressBar)
        {
            if (myProgressBar.InvokeRequired)
            { 
                CallPerformStep del = ActuallyPerformStep; 
                myProgressBar.Invoke(del, new object[] { myProgressBar }); 
                return; 
            }

            myProgressBar.PerformStep();
        }
    }



    class DataThread
    {
        private ProgressBar m_oPb;

        public DataThread(ProgressBar _oPb)
        {
            m_oPb = _oPb;
        }

        public ProgressBar PBar
        {
            get { return m_oPb; }
        }
    }
}

[giangian2]

Ciao,
va benissimo anche così, è il metodo corretto per aggiornare la grafica da un'altro thread.

;-)

[race2]

ora ho il problema di intercettare la fine di tutti i Threads per potere eseguire una funzione o inviare un messaggio di fine:

un primo step per risolvere è il seguente, ma non so ancora come passare i Parametri con il metodo che ho adottato nell'esempio di prima.

Qualche suggerimento ??

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

        private void btnStart_Click(object sender, EventArgs e)
        {
            const int numThreads = 5;

            ManualResetEvent[] manualEvents = new ManualResetEvent[numThreads];
            State stateInfo;

            for (int i = 0; i < numThreads; i++)
            {
                manualEvents[i] = new ManualResetEvent(false);

                stateInfo = new State(manualEvents[i]);

                ThreadPool.QueueUserWorkItem(new WaitCallback(Writer.WriteToFile), stateInfo);
            }

            //WaitHandle.WaitAll(manualEvents);

            foreach(WaitHandle handle in manualEvents)
            {
                handle.WaitOne();
            }
            
            MessageBox.Show("Ho finito con tutti i Thread !!");
        }
    }


    class State
    {
        public ManualResetEvent manualEvent;

        public State(ManualResetEvent manualEvent)
        {
            this.manualEvent = manualEvent;
        }
    }


    class Writer
    {
        public static void WriteToFile(object state)
        {
            State stateInfo = (State)state;

            try
            {
                //Code
                //
                //
            }
            finally
            {
                stateInfo.manualEvent.Set();
            }
        }
    }

[race2]

Risolto:

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

        private void btnStart_Click(object sender, EventArgs e)
        {
            const int numThreads = 5;

            ManualResetEvent[] manualEvents = new ManualResetEvent[numThreads];
            State stateInfo;

            for (int i = 0; i < numThreads; i++)
            {
                manualEvents[i] = new ManualResetEvent(false);

                stateInfo = new State(manualEvents[i]);

                ThreadPool.QueueUserWorkItem(new WaitCallback(Writer.WriteToFile), stateInfo);
            }

            //WaitHandle.WaitAll(manualEvents);

            foreach(WaitHandle handle in manualEvents)
            {
                handle.WaitOne();
            }

            MessageBox.Show("Ho finito con tutti i Thread !!");
        }
    }


    class State
    {
        public ManualResetEvent manualEvent;

        public State(ManualResetEvent manualEvent)
        {
            this.manualEvent = manualEvent;
        }
    }


    class Writer
    {
        public static void WriteToFile(object state)
        {
            State stateInfo = (State)state;

            try
            {
                //Code
                //
                //
            }
            finally
            {
                stateInfo.manualEvent.Set();
            }
        }
    }

Solo che ho perso tuta la mattina per capire come fare e poi vado ad applicare il codice al mio progetto e si blocca tutto !!!

Nel nuovo progetto "Win Form" dove l'ho testato tutto OK !!!

Nel mio progetto delle Newsletter mi blocca tutto, mi frizza il Form1...

ho provato a fare il DEBUG e in questo punto:

            
foreach (WaitHandle handle in aManualResetEvent)
{
    handle.WaitOne();
}

mi fa 2 Loop giustamente perchè ho impostato 2 Thread, e poi al prossimo "F10" mi esce dal DEBUG e si FRIZZA tutto il Form1 !!!

Mica sarà che è un Form "MdiContainer" con altri Form Figli aperti ???

[race2]

Scusate un altro dettaglio:

io lancio tutto da un "Form Figlio" di "Form1" e non dal "Form1" stesso !!!

[race2]

Tutto il lavoro fatto su questo post non lo posso utilizzare,
facendo questa procedura invece che aprire semplicemente un nuovo Thread, mi tiene impegnato anceh il Thread principale del Form dove lo lancio, tipo come se avessi fatto un Join con il Thread del Form, in questo modo fino a che non finisce di spedire tutte le Mail in coda, non mi permette neppure smuovere o ridurre il Form.

[giangian2]

Originariamente inviato da race2

Tutto il lavoro fatto su questo post non lo posso utilizzare,
facendo questa procedura invece che aprire semplicemente un nuovo Thread, mi tiene impegnato anceh il Thread principale del Form dove lo lancio, tipo come se avessi fatto un Join con il Thread del Form, in questo modo fino a che non finisce di spedire tutte le Mail in coda, non mi permette neppure smuovere o ridurre il Form.

Ovviamente sì... il problema è che tieni bloccato il Thread principale (quello della form) perchè attendi la fine di tutti i thread in esecuzione:

foreach(WaitHandle handle in manualEvents)
{
    handle.WaitOne();
}

Torno a consigliarti il BackgroundWorker che espone un evento RunWorkerCompleted generato quando l'operazione in background è terminata.

[race2]

BackgroundWorker non usa i Thread ???

Ho letto che BackgroundWorker viene utilizzato per processi molto Impegnativi, quindi è migliore dei Thread ???

[giangian2]

Originariamente inviato da race2

BackgroundWorker non usa i Thread ???

da MSDN (il link che ho inserito)

La classe BackgroundWorker consente di eseguire un'operazione in un thread separato e dedicato.

La Classe BackgroundWorker non fa altro che eseguire le operazioni, contenute nell' evento "DoWork", su un Thread separato, alla pari di una Sub/Function eseguita su un Thread separato (con proprietà IsBackground impostata a True) da te creato.
Inoltre BackgroundWorker espone, anche, 2 metodi molto utili

ProgressChanged
RunWorkerCompleted

che vengono generati sul Thread principale.
Come già detto l'evento RunWorkerCompleted viene generato quando l'operazione in background è terminata.
Mentre l'evento ProgressChanged lo devi generare tu durante le operazioni eseguite nel DoWork tramite l'utilizzo del metodo: .ReportProgress, e devi impostare su True la proprietà WorkerReportsProgress dell'oggetto BackGroundWorker.

ciao ciao

[race2]

ok, perfetto, grazie mille, vado avanti in un altra cosa e poi ritorno sopra a questo.

Ciao!!