Hi.Though I feel like I'm masking symptoms, this seems to correct that particular problem:
private void _SerialPort_LineReceived(string Text) { _DebugPrint('I', Text + "\r\n"); // Did we enter command mode? if (Text == "CMD" && this._Mode == Modes.Idle) { this._Mode = Modes.CommandMode; _DebugPrint('D', "Successfully enterred the command mode"); return; } else { if (Text.IndexOf("CMD") >= 0) { Debug.Print("FOUND CMD IN STRING: " + Text); this._Mode = Modes.CommandMode; _DebugPrint('D', "Pretending to Successfully enterred the command mode"); return; } }
There are couple other issues... I'll post when i have more to say about..
thx
2 issues, first it appeared there was a race condition that sometimes truncated the last string, if you set Mode to idle before setting the _SerialPort_StreamBuffer. Here the code:
private void _SerialPort_DataReceived(object sender, SerialDataReceivedEventArgs e)
{
if (this._SerialPort.BytesToRead == 0) return;
// Reads out the data and converts it to a string
byte[] ReadBytes = new byte[this._SerialPort.BytesToRead];
this._SerialPort.Read(ReadBytes, 0, ReadBytes.Length);
string ReadBuffer = new string(Tools.Bytes2Chars(ReadBytes));
// While in Streaming mode, we need to handle the data differently
if (this._Mode == Modes.StreamingMode)
{
bool newModeIdle = false;
_DebugPrint('I', ReadBuffer);
string NewBuffer = this._SerialPort_StreamBuffer + ReadBuffer;
// Fixes the "*CLOS*" issue as described below
this._SerialPort_EndStreamCheck += ReadBuffer;
if (this._SerialPort_EndStreamCheck.Length > this._SocketCloseString.Length)
this._SerialPort_EndStreamCheck = this._SerialPort_EndStreamCheck.Substring(this._SerialPort_EndStreamCheck.Length - this._SocketCloseString.Length);
// Do we need to leave Streaming Mode?
int CheckPos = NewBuffer.IndexOf(this._SocketCloseString);
if (CheckPos >= 0)
{
this._SerialPort_TextBuffer = NewBuffer.Substring(CheckPos + this._SocketCloseString.Length);
NewBuffer = NewBuffer.Substring(0, CheckPos);
//this._Mode = Modes.Idle;
newModeIdle = true;
_DebugPrint('D', "1 Left streaming mode");
_DebugPrint('D', NewBuffer);
}
// The closing string (default: "*CLOS*") is many times sent in multiple packets.
// This causes annoying issues of connections not shutting down well.
// This fixes that issue, but it's possible the last few bytes of the stream contain something like "*CL" or something.
else if (_SerialPort_EndStreamCheck == this._SocketCloseString)
{
//this._Mode = Modes.Idle;
newModeIdle = true;
_DebugPrint('D', "2 Left streaming mode");
NewBuffer = "";
}
this._SerialPort_StreamBuffer = NewBuffer;
if (newModeIdle)
this._Mode = Modes.Idle;
return;
}
The other thing is just a nicety to remove extraneous *CLOS from the last string (assuming that's not really part of the message).In HTTP_Client
private HTTP_Response _DoRequest(string RequestData)
{
// Opens the connection
this._Socket.Connect();
// Sends out the request
this._Socket.Send(RequestData);
// Fetches the returned data
string ResponseData = "";
string tmp = "";
while (this._Socket.IsConnected || this._Socket.BytesAvailable > 0)
{
tmp = this._Socket.Receive();
if (tmp.IndexOf("HTTP/1.1 200 OK") >= 0)
{
Debug.Print("found OK HEADER at " + tmp.IndexOf("HTTP/1.1 200 OK") + ", resetting input string");
ResponseData = tmp;
}
else
ResponseData += tmp;
}
// Closes the connection
this._Socket.Close();
Debug.Print("RESPONSE: " + ResponseData[ResponseData.Length-1]);
string ending = ResponseData.Substring(ResponseData.Length-5);
string foo = ending.Substring(1, 4);
string foo2 = ending.Substring(2, 3);
string foo3 = ending.Substring(3, 2);
if (ending == "*CLOS") ResponseData = ResponseData.Substring(0, ResponseData.Length - 5);
else if (ending.Substring(1, 4) == "*CLO") ResponseData = ResponseData.Substring(0, ResponseData.Length - 4);
else if (ending.Substring(2, 3) == "*CL") ResponseData = ResponseData.Substring(0, ResponseData.Length - 3);
else if (ending.Substring(3, 2) == "*C") ResponseData = ResponseData.Substring(0, ResponseData.Length - 2);
else if (ending.Substring(4, 1) == "*") ResponseData = ResponseData.Substring(0, ResponseData.Length - 1);
// Parses the response data
HTTP_Response RetVal = new HTTP_Response(ResponseData);
// Parses cookies and such
this._ParseHeaders(RetVal.GetAllHeaders());
return RetVal;
}
looks OK now, but cursory checks.. will do more checkingFritz
Edited by Stefan, 23 June 2012 - 08:09 AM.
Added [code] tags
