Issue using XMLSerializer to Serialize and Deserialize

I am working on C# sockets and using XMLSerializer to send and receive data. The XML data are sent from a server to a client over a network connection using TCP/IP protocol. The XML.Serializer.Serialize(stream) serializes the XML data and send them over the socket connection but when I want to use the XMLSerializer.Deserialize(stream) to read. The sent data returns a xml parse error.

Here is how I'm serializing:

Memory Stream ms = new MemoryStream();
FrameClass frame= new FrameClass ();
frame.string1 = "hello";
frame.string2 = "world";

XmlSerializ开发者_JS百科er xmlSerializer = new XmlSerializer(frame.GetType());
xmlSerializer.Serialize(ms, frame);

socket.Send(ms.GetBuffer(), (int)ms.Length, SocketFlags.None);

Deserializing:

FrameClass frame;
XmlSerializer xml = new XmlSerializer(typeof(FrameClass));
frame= (FrameClass)xml.Deserialize(new MemoryStream(sockCom.SocketBuffer));
listbox1.Items.Add(frame.string1);
listbox2.Items.Add(frame.string2);

I think it has something to do with sending the data one right after another. Can anyone teach me how to do this properly?

Have you received all of the data before attempting to deserialize (it's not clear from your code). I'd be inclined to receive all of the data into a local string and the deserialize from that rather than attempting to directly deserialize from the socket. It would also allow you to actually look at the data in the debugger before deserializing it.

Try this:

using (MemoryStream ms = new MemoryStream())
{
    FrameClass frame= new FrameClass ();
    frame.string1 = "hello";
    frame.string2 = "world";

    XmlSerializer xmlSerializer = new XmlSerializer(frame.GetType());
    xmlSerializer.Serialize(ms, frame);
    ms.Flush();
    socket.Send(ms.GetBuffer(), (int)ms.Length, SocketFlags.None);
}

If you're sending the Frame XML one right after the other, then you're not sending an XML document. The XML Serializer will attempt to deserialize your entire document!

I don't have time to research this now, but look into the XmlReaderSettings property for reading XML fragments. You would then create an XmlReader over the memorystream with those settings, and call it in a loop.

The important thing is to flush the stream. It's also useful to put the stream in a using block to ensure it's cleaned up quickly.

Besides what @John said about the Flush call, your code looks alright.

You say you're sending multiple FrameClass data pieces, then the code should work sending just a single piece of data.

If you need to send multiple data objects, then you cannot send them all in one go, otherwise the deserialization process will stumble over the data. You could setup some communication between the server & the client so the server knows what it's getting.

client: I have some data

Server: ok I'm ready, send it

client: sends

Server: done processing

repeat process...