C# Receive Data from Socket and put it in a String?

Question

I have a socket server and am trying to receive a string from the client.

The client is perfect and when I use this

Socket s = myList.AcceptSocket(); Console.WriteLine("Connection accepted from " + s.RemoteEndPoint); byte[] b = new byte[100]; int k = s.Receive(b); Console.WriteLine(k); Console.WriteLine("Recieved..."); for (int i = 0; i < k; i++) { Console.Write(Convert.ToChar(b[i])); ASCIIEncoding asen = new ASCIIEncoding(); s.Send(asen.GetBytes("The string was recieved by the server.")); }

All is okay and I get my string in the console.

But how can I get now my receive into a string so I can use it in a switch case?

Like this:

string action = Convert.ToChar(b[i]);

Error:

The Name i isn't in the current context. its the only Error Message i get.

Please enhance your code example and show the definition of b and possibly s. Next, you certainly did not get the error you are describing because it clearly has typos in it. Please post a copy-pasted version of the exception plus the callstack, not a paraphrased write-up. I suspect you got the error described in this post: stackoverflow.com/questions/706603/… — Christoph
– Christoph, Commented Sep 5, 2015 at 12:03
@Christoph I checked your link but not much helpfull or informing for me. I must say iam not often program with C# — Hanselman
– Hanselman, Commented Sep 5, 2015 at 12:08
You can't just say Convert.ToChar(b[i]) and expect it to return a string in which the variable i looped through all the bytes of the b-array. Explicity create the loop variable i in a for statement, otherwise the variable won't exist. string action = ""; for(int i=0; i < b.Length; i++) action += Convert.ToChar(b[i]); would fill the action variable appropiatly. — Maximilian Gerhardt
– Maximilian Gerhardt, Commented Sep 5, 2015 at 12:13
"The Name i isn't in the current context" - then you're not showing your actual code. Do you have a semicolon after the for();? — CodeCaster
– CodeCaster, Commented Sep 5, 2015 at 12:24

Rodrigo Reis · Accepted Answer · 2017-04-27 18:50:15Z

This way no need set buffer size, it fits to response:

public static byte[] ReceiveAll(this Socket socket) { var buffer = new List<byte>(); while (socket.Available > 0) { var currByte = new Byte[1]; var byteCounter = socket.Receive(currByte, currByte.Length, SocketFlags.None); if (byteCounter.Equals(1)) { buffer.Add(currByte[0]); } } return buffer.ToArray(); }

I think a minor optimization would be to put the currByte variable outside the while scope. Also, instead just taking a single byte at a time, you could take all the available. Nonetheless, a very robust solution!

Maximilian Gerhardt · Accepted Answer · 2015-09-05 12:35:44Z

4

Assuming s is a Socket object on which you call receive, you get an byte[] back. To convert this back to a string, use the appropiate encoding, e.g.

string szReceived = Encoding.ASCII.GetString(b);

Edit: Since the buffer b is always 100 bytes, but the actual number of bytes received varies with each connection, one should use the return value of the Socket.Receive() call to only convert the actual number of received bytes.

byte[] b = new byte[100]; int k = s.Receive(b); string szReceived = Encoding.ASCII.GetString(b,0,k);

edited Sep 5, 2015 at 12:35

answered Sep 5, 2015 at 12:07

Maximilian Gerhardt

5,3633 gold badges34 silver badges72 bronze badges

3 Comments

Hanselman Over a year ago

It works half Thanks ! :) But now it is showing correct when i write it in Console.... but My Switch Case will not work.. happend nothing :o Console.WriteLine(szReceived); Console.WriteLine("Recieved..."); switch (szReceived) { case "ping": Console.WriteLine("Ping wird ausgeführt!"); break; case "somethingelse": break; }

Maximilian Gerhardt Over a year ago

My first intuition says that you should cut off the array at the number of bytes you received. Socket.Receive() has a return value, and you need that to chop off your array at the actual bytes received. Use this: int k = s.Receive(b); string szReceived = Encoding.ASCII.GetString(b.Take(k).ToArray()); Then the string should be constructed perfectly. (You need a using System.Linq; for that, or use the second overload string szReceived = Encoding.ASCII.GetString(b,0, k);)

Hanselman Over a year ago

You did it ! :D Thanks for all.

Masoud Siahkali · Accepted Answer · 2016-10-31 12:16:26Z

Init socket

Socket socket = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp); IPAddress ipAdd = System.Net.IPAddress.Parse(m_ip); IPEndPoint remoteEP = new IPEndPoint(ipAdd, m_port);

Connect socket

socket.Connect(remoteEP);

Receive from socket

byte[] buffer = new byte[1024]; int iRx = socket.Receive(buffer); char[] chars = new char[iRx]; System.Text.Decoder d = System.Text.Encoding.UTF8.GetDecoder(); int charLen = d.GetChars(buffer, 0, iRx, chars, 0); System.String recv = new System.String(chars);

Send message

byte[] byData = System.Text.Encoding.ASCII.GetBytes("Message"); socket.Send(byData);

Close socket

socket.Disconnect(false); socket.Close();

Collectives™ on Stack Overflow

C# Receive Data from Socket and put it in a String?

3 Answers 3

1 Comment

3 Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

3 Comments

Comments

Linked

Related