Running a thread for Text box event

Question

1.00/5 (2 votes)

See more:

I have a textbox , through textbox i am doing chat and this textbox is continuously being used hence it has blocked my program UI other controls . So i want to make a thread for my textbox event . So that i would not block my program UI . Consider below as a textbox event for which i want to make a thread.

private void textBox1_KeyDown(object sender,KeyEventArgs e)
{

}

So how its Possible ??? I want to do it in C# .Net ..... Thanks ...

Posted 2-Aug-15 19:13pm

jahanzaib ali khan 2237-FBAS/BSCS/F11

Add a Solution

Comments

Sergey Alexandrovich Kryukov 3-Aug-15 1:31am

What does it mean, make a thread for an event? Do you mean starting a thread in a KeyDown event handler. Certainly, but it hardly can make any practical sense. So many threads? Why? Better tell us what do you want to achieve.
—SA

jahanzaib ali khan 2237-FBAS/BSCS/F11 3-Aug-15 2:05am

I want to start a thread for the text_Box_Event , because my this text box event is continuosly being used until program is closed , so this text_box_event blocks the UI . Thats why i want to make a thread and attach a event with a thread....

Ralf Meier 3-Aug-15 2:15am

In my opinion it is nearly impossible to make a Suggestion with this Information you gave.
But I agree with Sergey : in the Moment it makes no sense to have a Thread beside the KeyDown-Event-Handler. For what ?
My Suggestion is this : give us more info about your programm-functionality and what happens what you don't want to have.
What is meant with : "it blocks the UI" ? Normally a correct Event-handling could not block the UI ...

Sergey Alexandrovich Kryukov 3-Aug-15 2:51am

It does not mean to make any sense. If you really have such a problem, create a minimal but 100% comprehensive self-contained code and show it using "Improve question".
—SA

himanshu agarwal 3-Aug-15 5:00am

If using textbox is making your application freeze, then I would like to know what other event handlers have you hooked on to on that text box?

Adding, another event handler will only degrade the performance.

jahanzaib ali khan 2237-FBAS/BSCS/F11 3-Aug-15 10:05am

Below is the full chat code which i am using it is blocking . I think the textBox1_KeyDown event handler is blocking ..

private void textBox1_KeyDown(object sender, KeyEventArgs e)
{
try
{
if (e.KeyCode == Keys.Enter)
{
writer.Write(Form1.GetSysName() + " : " + input_text.Text); // Send to Sever

output_text.Text += "\r\n" + Form1.GetSysName() + " : " + input_text.Text;

input_text.Clear();

}
}
catch (SocketException se)
{
output_text.Text += "\n Error waiting Object";
}
}

public void Run_client()
{

try
{
//output_text.Text += "Attempting to Connecting to the server\n";

// Step 1: Create TCPClient and connect to the Server
client = new TcpClient();
string var4 = ipaddress2;
client.Connect(var4, 5000); // 5000 is the port number that the Sever is listening on it

// Step 2: Get NetworkStream Associated With TcpClient
output_stream = client.GetStream();

// Step 3: Create Object for Writing and Reading Across Stream( NetworkStream)
writer = new BinaryWriter(output_stream);

reader = new BinaryReader(output_stream);
// output_text.Text += "\r\nGot IO Stream \r\n";
input_text.ReadOnly = false;

do
{
//Step 3:
try
{
//Reading the message form Server

message = reader.ReadString();

output_text.Text += "\r\n : " + message;

}
catch (Exception e)
{
//System.Environment.Exit(System.Environment.ExitCode);
}
}while (message != "Server>>terminate");

// Step 4: Closing Connection
writer.Close();
reader.Close();
output_stream.Close();
client.Close();
}
catch (SocketException se)
{
MessageBox.Show(se.Message);
}

}

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Answer 1 · 2015-08-02T22:56:00

Solution 2

You shouldn't do that. You should instead create a worker thread for the chat communication and pass data to such worker thread from your text box (see "How to: Make Thread-Safe Calls to Windows Forms Controls"[^]).

Posted 2-Aug-15 22:56pm

CPallini

Comments

F-ES Sitecore 5-Aug-15 4:59am

+5d The trick the OP is missing is that he needs to call "Invoke" so the update is executed on the UI thread. I'd recommend not using a textbox for the whole chat though, maybe maintain a label or some other text area, and just have a textbox for writing out messages.

himanshu agarwal · Answer 2 · 2015-08-04T22:51:00

I would recommend to use a thread pool, to queue your logic of sending data to server.

C#

ThreadPool.QueueUserWorkItem(SendData, "data");

then, you may define the SendData function with the logic you wrote in the KeyDown handler.

C#

public void SendData(object inputParam)
{
  try
  {
    string input = inputParam as string;
    writer.Write(Form1.GetSysName() + " : " + input); // Send to Sever
    ...
  }
  catch (SocketException se)
  {
    output_text.Text += "\n Error waiting Object";
  }
}

However, if you want your messages to be sent in synchronized way then, threadpool will not be a good choice, there you may just spawn a thread.

C#

Thread th = new Thread(SendData);
th.Start();

But, you may also need to define a mechanism to keep queuing your requests.