|
the problem is not the freeze of the GUI. that i can simply avoid by multithreading as you pointed out. the issue is stopping any further calls of the function during the execution of the task.
i have run into this problem when i had a task that needs like about 500 miliseconds to run (more exactely it's a MoveNext). i decided not to multythread it since it's quite short. the problem is when a users starts pounding the NextButton with 20 clicks a second (it happend to my by accident in a test) the application starts to slow down since it's ordered to do like 20 tasks/second and it can only do 2/second. not to mention sometimes it has to acces sensible data that has to be locked down for thread safety purposes and it sometimes takes like 2-5 minutes for it to actualy recover. there are more issues here like race conditions and so forth.
now i thought that blocking any calls to the function would be the easyest way to do it. however it turned out not to be.
|
|
|
|
|
Why don't you simply disable the button and enable it after the operation completes? Something like
private void buttonClick(object sender, EventArgs e)
{
thisButton.Enabled = false;
DoLongOperation();
thisButton.Enabled = true;
}
If you can't enable/disable, you can have a flag to simulate it, set the flag to false before the operation and reset it after the operation. The function would then first check if the flag is true before proceeding.
Regards
Senthil
_____________________________
My Blog | My Articles | WinMacro
|
|
|
|
|
obviously you haven't payed attention to my first post. the first example does just that. disabling the button and enabling it after the task is done. however it doesn't behave as expected. Visualy the button is disabled, but the function executes as many times as you push the button, regardless of it's state (disabled/enabled). if you put a counter in it and look at it after the test you'll see it's nice and neat the number of clicks you made.
i think that this is because somehow it stacks the events and fires them after the function executes. try and run this code:
<br />
public void task()<br />
{<br />
Thread.Sleep(1000);<br />
this.counter++;<br />
}<br />
private void button1_Click(object sender, System.EventArgs e)<br />
{<br />
this.button1.Enabled=false;<br />
Thread t=new Thread(new ThreadStart(task));<br />
t.Start();<br />
while((t.ThreadState & (System.Threading.ThreadState.Stopped | System.Threading.ThreadState.Unstarted)) == 0)<br />
{<br />
Thread.Sleep(100);<br />
}<br />
t.Join();<br />
this.button1.Enabled=true;<br />
Debug.WriteLine(this.counter);<br />
}<br />
this has multi threading too. i can't exclude the possibility i'm doing something wrong but i can't seem to see where. pointers anyone?
|
|
|
|
|
Yeah, you're right. What's happening is after the button gets disabled, all the clicks keep getting added to the message queue and once your button gets enabled, they get dispatched to the button and so you keep getting the event as many times as the user clicked.
To prevent that from happening, just call Application.DoEvents() before enabling the button. That'll clear off all those click messages in the queue and therefore you wont get the Click event. Just do
public void task()
{
Thread.Sleep(1000);
this.counter++;
}
private void button1_Click(object sender, System.EventArgs e)
{
this.button1.Enabled=false;
Thread t=new Thread(new ThreadStart(task));
t.Start();
while((t.ThreadState & (System.Threading.ThreadState.Stopped | System.Threading.ThreadState.Unstarted)) == 0)
{
Thread.Sleep(100);
}
t.Join();
Application.DoEvents();
this.button1.Enabled=true;
Debug.WriteLine(this.counter);
}
Regards
Senthil
_____________________________
My Blog | My Articles | WinMacro
|
|
|
|
|
neat! works. thanks for the help
|
|
|
|
|
Alternatively, you could just decouple the event handler and then recouple it after the call.
|
|
|
|
|
yeah that crossed my mind too. but it seems a bit sadistic to me anyways.
funny thing is i knew about Application.DoEvents() and actualy used it .. but to no result. in my test i placed it in the while loop because normaly you want from time to time the app to respond to other calls but oddly it didn't work(no idea why thow).
the other thing that crossed my mind was to add a messageFilter before thread.Start and remove it after my task finished. but that's a bit too costly in my opinion.
|
|
|
|
|
put this as a class variable
<br />
private ThreadStart ts= new ThreadStart(urfunctionhere);<br />
private Thread t=new Thread(ts);<br />
then in ur button u could do
{<br />
<br />
button.enabled=false;<br />
<br />
if(t!=null&&!(t.isAlive))
{<br />
t.start(); <br />
}<br />
<br />
button.enabled=true;<br />
<br />
}
hope this help u
|
|
|
|
|
well for one your example doesn't work. mainly because that's not the propper way to test if a thread is running. however even if i correct that problem it still doesn't solve my problem because now you have to sincronize the started thread. if you will read my third post you'll see a sincronised example that well you guessed it .. doesn't stop the calls.
|
|
|
|
|
You may try the following method if it works (it apparently works on my machine; however I cannot test in your conditions).
As an event can be handled by more than one event handler, you may declare two event handlers for the same button click by using two different methods, as under:
this.button1.Click += new System.EventHandler(this.button1_Click);
this.button1.Click += new System.EventHandler(this.button1_Click2);
It is pertinent to mention here that when an event is handled by more than one handler, all the methods handling that event are executed. The order in which the methods execute is the same as the order in which the association was created. Thus, in this case, "button1_Click" method will be executed first and thereafter the second method "button1_Click2" will be executed. What I suggest it that you disable the button1 in the first such method and do your long task in the second method. As by the time the second method starts, the first method must already have been executed (meaning that the button1 is now already disabled), you can safely do your long task in the second method. The code can be something like this:
private void button1_Click(object sender, System.EventArgs e)
{
this.button1.Enabled = false;
}
private void button1_Click2(object sender, System.EventArgs e)
{
this.button1.Enabled = true;
}
I am not sure, but I hope that your problem can be solved in this manner.
|
|
|
|
|
Hi all,
one big disadvantage (that's my feeling) of .Net-App is, the application startup delay on the first start (loading libraries etc.). I am going to write a "Do nothing Apps" which will be loaded on autostart, so that the libraries are in memory an the other applications will start faster.
.... or is there a much better way to improve the startingtime ?
Frank
|
|
|
|
|
That might work, until you load something that the OS decides is more important and frees up the unused, but still cached, libraries.
I guess I wouldn't bother. All you're doing is moving the problem around. Now you're computer will boot that much slower! And you still won't have solved your "first time load" time delay.
Marc
MyXaml
Advanced Unit Testing
YAPO
|
|
|
|
|
I have a new lunatic idea,
I write a VB6 app's, showing a pretty logo and the hourglass cursor, do a process-start of my .net app's and make a challange to the user to cook e new coffee, make two or three phone calls and than...... my app's still ready to work.
.... o.k we are in the Einstein Year - time is relative - if we move slower ... time goes faster or something like that.
That's another day of microsoft frustation.
Frank
|
|
|
|
|
Don't do it.
If you send the loading time to system startup, you will make it slower every time; and it's not every time that your users turn on their computer to use your app, is it?
-- LuisR
Luis Alonso Ramos
Intelectix - Chihuahua, Mexico
Not much here: My CP Blog!
|
|
|
|
|
|
hi,
i use asqldatareader to read my data which the commandtext
has two transaction inside,when i attempt to get the data by Read() method it gets the first result set and the other is lost,when i use NextResult() method i get the other result set but first result of the set is skipped,
what should i do?
here is my commandtext:
mycommand.CommandText="SELECT COUNT(TypeId) FROM mytable1; SELECT Name, TypeId FROM mytable2 ";
|
|
|
|
|
|
hi,
here is my code:
com.CommandText="SELECT COUNT(TypeId) AS idcount "<br />
+"FROM NewsTypes ; SELECT Name, TypeId FROM NewsTypes "<br />
+"ORDER BY TypeId";<br />
try<br />
{<br />
SqlDataReader rd=com.ExecuteReader();<br />
rd.Read();<br />
temp=new NewsTypes[(int)rd["idcount"]];<br />
int index=0;<br />
while(rd.NextResult())<br />
{<br />
rd.Read();<br />
temp[index]=new NewsTypes((string)rd["name"],(short)rd["typeid"]);<br />
index++;<br />
}
thanks
|
|
|
|
|
Hi UAU,
The problem is with your while statement. Instead of that use a do while loop while using the NextResult on datareader. Guess the following code will help you.
SqlConnection conn = new SqlConnection();
conn.ConnectionString = "";
conn.Open();
SqlCommand percommand = new SqlCommand("select * from person;select count(*)as name from person;",conn);
SqlDataReader reader = percommand.ExecuteReader();
do
{
while(reader.Read())
{
MessageBox.Show(reader["name"].ToString());
}
}while(reader.NextResult());
Thanks and Regards
SGS
|
|
|
|
|
|
unfortunatley this doesnt work yet,i lost first row of
the next resultset,i examined several ways but i loose my rows
|
|
|
|
|
|
I did so but i lost the first row of resultset.
I think i'd better use dataset instead.
|
|
|
|
|
My windows application creates word documents from template document with bookmarks and data from database. I create one document for each user, e.g. I select five users and application creates five MS word documents filled by data from database. Documents are saved in memory. I use reference to "Microsoft Word 9.0 Object Library" (MS Word 2000). My target is application creates word documents and merges into one document and saves it. Can you help me?
|
|
|
|
|
Have a look here[^].
It's a Web-based solution but it's quite similar to what you are after, and can easily be ported to a Windows application.
Hope this helps
Regards,
Polis
Can you practice what you teach?
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
