Re: Open Child Form and threading - C# Discussion Boards

0x3c012-Aug-09 4:16

0x3c0

12-Aug-09 4:16

There are three things which could cause the delay:

this.Cursor = AdvancedCursors.Create(Path.Combine(Application.StartupPath, "wagtail.ani"));
frm2 = new frmGrid2(m_ADBCnn, TechnicianstoolStripMenuItem1.Text, frm1);
this.VerticalTiletoolStripMenuItem1.PerformClick();

Which of those lines causes the delay? You can find this out by stepping through the code and seeing which line seems to take longer to execute.

When you know which line takes the longest time to execute, put it into a separate thread.

Between the idea
And the reality
Between the motion
And the act
Falls the Shadow

Saamir12-Aug-09 4:33

Saamir

12-Aug-09 4:33

It's definately frm2 = new frmGrid2(m_ADBCnn, TechnicianstoolStripMenuItem1.Text, frm1); and frm2.show(). The part that takes long is the loading of the child form because it pulls quite a bit of data so it opens it but then the grid in child form is blank until all data is pulled.

Sameer

Sameer

0x3c012-Aug-09 4:49

0x3c0

12-Aug-09 4:49

What you need to do is get the data in a separate thread. In the GUI thread, set the grid's data source to your table. Then start the separate thread which adds the data to the data source. The grid should get updated automatically.

Between the idea
And the reality
Between the motion
And the act
Falls the Shadow

Saamir12-Aug-09 5:27

Saamir

12-Aug-09 5:27

Pardon my ignorance as I am not the best in using threads. Currently on load of child form I am calling the runworkerasync method of the background worker with retrieves data and upon completion assigns dataset to grid and formats the grid.

Here is the change I have made and I am getting an error of cross thread operation.

worker1
Async in form load
do work retrieves data from database
Completed calls Worker2Async

Worker2
Async in Completed of Worker1
Do work assigns datagridview datasource to dataset containing data retrieve by worker1
Completed formats the grid

I am getting an error in Do work for Worker 2, when I call datagridview.datasource=dataset, it says cross thread operation... datagridview accessed from a different thread then the one it was created in... Any suggestions, am I doing what you suggested?

Please help

Sameer

Dave Kreskowiak12-Aug-09 6:23

Dave Kreskowiak

12-Aug-09 6:23

You only need one worker that gets the data from the database.

The form should kick off the background worker, then format it's datagrid as appropriate on the UI thread. ANY interaction with a control MUST be done on the UI thread, not from a background worker.

When the data fetching background work finishes it's job, just the the grid's DataSource property to the data returned.

A guide to posting questions on CodeProject[^]

Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007, 2008
But no longer in 2009...

Saamir12-Aug-09 6:51

Saamir

12-Aug-09 6:51

Thank you Dave for the guidance. I have switched my code back to your suggestion for the most part. I now have a background worker that does the following:

On form load I am calling the runworkerasync

dowork: retrieves all the data to database and assigns it to a datatable
Completed: Assigns datatable to datagridview datasource

Now I need to run a format grid function that goes through the grid and formats some cells. Where do I call that, should I call that in the completed of the worker?

Please advice

Sameer

Cut,Copy ,Paste Problem...

Luc Pattyn12-Aug-09 7:59

Luc Pattyn

12-Aug-09 7:59

Hi,

the basic idea is:
- long computations (including waiting on a database or a network...) belong in a separate thread, which could be a BackgroundWorker or a ThreadPool thread or a real Thread instance.
- manipulations of Controls (including Forms and DataGrids) need to run on the main thread

You can organize that with BackgroundWorkers, as they have two events (Progress and Completed) that always will run on the main thread; or you can get similar results with Control.Invoke

I have a little article[^] that explains more.

Smile | :)

Luc Pattyn [Forum Guidelines] [My Articles]

The quality and detail of your question reflects on the effectiveness of the help you are likely to get.
Show formatted code inside PRE tags, and give clear symptoms when describing a problem.

Rahul DSG12-Aug-09 3:00

12-Aug-09 3:00

Private StringBuilder sb=null

private void CutCopyPaste(bool Cut, bool Copy, bool Paste)
{
foreach (Form f in this.MdiChildren)
{
if (f.ContainsFocus)
{
foreach (Control c in f.Controls)
{
if (c.Focused)
{
if (c.GetType() != typeof(Label))
{
if (Cut)
{
sb.Append(c.Text);
c.Text = "";
}
else if (Copy)
sb.Append(c.Text);
else if (Paste)
c.Text = sb.ToString();
break;
}
}
}
}
}
}

private void copyToolStripMenuItem_Click(object sender, EventArgs e)
{
try
{
sb = null;
sb = new StringBuilder();
CutCopyPaste(false, true, false);
}
catch (Exception ex)
{
MessageBox.Show(ex.Message);
}
}

private void pasteToolStripMenuItem_Click(object sender, EventArgs e)
{
try
{
if (sb != null)
CutCopyPaste(false, false, true);
}
catch (Exception ex)
{
MessageBox.Show(ex.Message);
}
}

private void cutToolStripMenuItem_Click(object sender, EventArgs e)
{
try
{
sb = null;
sb = new StringBuilder();
CutCopyPaste(true, false, false);
}
catch (Exception ex)
{
MessageBox.Show(ex.Message);
}
}

It is the complete code for cut copy and paste but if we take textbox inside the container like groupbox,panel etc. its not working ...and i want also include the feature Undo and SelectAll...how can i do this...

ricmil4212-Aug-09 3:46

ricmil42

12-Aug-09 3:46

Group and Panel controls have their own Collection of controls. So once you find one of these controls you have to do a foreach() to access the controls that it holds.

You can use a recursive method to access the control you are looking for, or write code to access these special cases.

My choice would be recursion since it will handle the nesting of these special cases.

Rahul DSG12-Aug-09 18:38

12-Aug-09 18:38

yes u are right...how can i acess those controls that it holds...
guide me...

ricmil4213-Aug-09 3:26

ricmil42

13-Aug-09 3:26

Try something like this (not tested).

public void ProcessLabel ( Control.ControlCollection c )
{
    foreach ( Control control in c )
    {
        if ( control.Controls.Count != 0 )
            ProcessLabel ( control.Controls ) ;

        if ( control is Label )
            {
                // Do something with the label.
            }
    }
}

Call it like

ProcessLabel ( Controls )    // This is the Control array from your main form or Panel or Group control.

If you need to send more parameters, modify the method but be sure you also add them to the recursive call as well.

public void ProcessLabel ( Control.ControlCollection c, bool cut, bool copy, bool paste )
{
    foreach ( Control control in c )
    {
        if ( control.Controls.Count != 0 )
            ProcessLabel ( control.Controls, cut, copy, paste ) ;

        if ( control is Label )
            {
                // Do something with the label.
            }
    }
}

Rahul DSG22-Aug-09 3:11

22-Aug-09 3:11

how to call this method...
processlabel(whichcontrol);

ricmil4224-Aug-09 3:19

ricmil42

24-Aug-09 3:19

You can change the signature like this to add the specific label control you are interested in processing.

public void ProcessLabel ( Control.ControlCollection c, Label label )
{    foreach ( Control control in c )
    {
        if ( control.Controls.Count != 0 )
            ProcessLabel ( control.Controls ) ;

        if ( control is Label && control.Name = label.Name )
        {
            // Do something with the label.
            label.ForeColor = Color.Red ; // Or whatever.
        }
    }
}

How to call this method

// "Controls" is the Control array from your main form or Panel or Group control.
// The container that holds the label control.
ProcessLabel ( Controls, myLabel )

If you need to send more parameters, modify the method but be sure you also add them to the recursive call as well.

public void ProcessLabel ( Control.ControlCollection c,
                           Label label,
                           bool cut,
                           bool copy,
                           bool paste )
{
    foreach ( Control control in c )
    {
        if ( control.Controls.Count != 0 )
            ProcessLabel ( control.Controls, label, cut, copy, paste ) ;

        if ( control is Label && control.Name = label.Name )
        {
            // Do something with the label.
            label.ForeColor = Color.Red ; // Or whatever.
        }
    }
}

Rahul DSG30-Aug-09 18:55

30-Aug-09 18:55

I am trying But its not working...have u run this..

ConnectionString Problem with MSACCESS

Andrew Rissing12-Aug-09 3:47

Andrew Rissing

12-Aug-09 3:47

The reason it fails is because you're not doing a recursive search through the child controls of the Form. You're only looking at the top level of the controls - controls are hierarchal in nature. I would definitely recommend reworking your code to be more modular or atleast rely on a different method of finding the focused control.

One method might be hooking up a listener for the focused event on every control (assuming you don't have 1000+ controls) - see this[^].

You might also consider using the clipboard to maintain the text to be copied rather than the stringbuilder you've got - see this[^].

M Riaz Bashir12-Aug-09 2:19

M Riaz Bashir

12-Aug-09 2:19

hi,

I am using following connection string but one ISAM error comes whenever I run the application.
Kindly check the mistake that, What am I doing wrong?

Thanx in Advance
(Riaz)

"Provider=Microsoft.ACE.OLEDB.12.0;Data Source="+ strDBPath +"stockmain.accdb;Jet OLEDB:Database Password=abc;SET EXCLUSIVE=OFF;");

Re: ConnectionString Problem with MSACCESS

Mike Ellison12-Aug-09 2:43

Mike Ellison

12-Aug-09 2:43

Have you looked on http://www.connectionstrings.com[^]?

Do you need to add Persist Security Info=False; ? I'm just guessing.

www.MishaInTheCloud.com

Re: ConnectionString Problem with MSACCESS

Luc Pattyn12-Aug-09 2:54

Luc Pattyn

12-Aug-09 2:54

Have you bothered to actually look at the full connection string once it got assembled?

and does strDBPath end on a backslash?

Smile | :)

Luc Pattyn [Forum Guidelines] [My Articles]

The quality and detail of your question reflects on the effectiveness of the help you are likely to get.
Show formatted code inside PRE tags, and give clear symptoms when describing a problem.

Limit checkedListBox for maximum values [modified]

bonzaiholding12-Aug-09 2:08

bonzaiholding

12-Aug-09 2:08

Hi,
I have an checkedListBox with 7 items :
1. a
2. b
3. c
4. d
5. e
6. f
7. g

I want to limit the user to choose only two , after 2 checkbox are checked i want it to be impossible to let the user to check another checkbox until he unchecked at least one of the checkbox that been checked.

Example :

private void checkedListBox_YAxisOptions_ItemCheck(object sender, ItemCheckEventArgs e)
        {
            if (e.NewValue == CheckState.Checked && this.ThereIsToMuchCheckBox() )
            {
                //I Want To Uncheck the checkbox that been selected. (Cancel the Selection)
                checkedListBox_YAxisOptions.SetItemCheckState(e.Index, CheckState.Unchecked);
                MessageBox.Show("You can't choose more then 4 options!");
            }
        }

With this code i have a problem because every time i have a memory overflow problem.
How can i do it?

modified on Wednesday, August 12, 2009 8:24 AM

Ashfield12-Aug-09 2:09

Ashfield

12-Aug-09 2:09

Check the selecteditems.count

Bob
Ashfield Consultants Ltd
Proud to be a 2009 Code Project MVP

bonzaiholding12-Aug-09 2:11

bonzaiholding

12-Aug-09 2:11

In which event and if select one how can i stop him from select another one ?

Alan N12-Aug-09 2:34

Alan N

12-Aug-09 2:34

Hi,
Your program should handle the ItemCheck event. The ItemCheckEventArgs has three properties, CurrentValue, Index and NewValue.

NewValue is the proposed new check state of the item at Index. Typically to prevent a change of check state one would write

e.NewValue = e.CurrentValue;

[EDIT] Just seen your modified post. There is no need for SetItemCheckState which is causing the event to be fired again and again etc. Just changing the NewValue is sufficient.

Alan.