string

macmac38

Hello, This arrives at my serial port:
924.09,-4.06,676.26,0.156488,-0.653856,-0.714853,-0.192273
CString nx; while(port.Read(cReceivedChar, 1 , overlapped, &dwBytesRead)) { nx += cReceivedChar[0]; SetDlgItemText(IDC_SHIFT_X,("%c", nx)); } but my dialog only shows 06,676 ?? Why? Thanks, Mark

jmkhael

What is in nx? Try a MessageBox call and why not a SetDlgItemText(IDC_SHIFT_X,(LPCSTR)nx); Papa while (TRUE) Papa.WillLove ( Bebe ) ;

macmac38

Hello, nx contains nothing when i trace it. cReceivedChar[0] contains confused values from the string.. I think the whole procedure must be rewied Thanks, Mark

David Crow

CString nx;
char cReceivedChar;

while(port.Read(&cReceivedChar, 1, overlapped, &dwBytesRead))
nx += cReceivedChar;

SetDlgItemText(IDC_SHIFT_X, nx);

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

char cReceivedChar; is not working (other type expected) BOOL CSerialPort::Read(void* lpBuf, DWORD dwCount, OVERLAPPED& overlapped, DWORD* pBytesRead) Thanks, Mark

David Crow

How are you calling the Read() method?

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

while(port.Read(cReceivedChar, 1 , overlapped, &dwBytesRead)) Thanks, Mark

David Crow

The first parameter is supposed to be a pointer.

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

char *cReceivedChar; while(port.Read(cReceivedChar, 1, overlapped, &dwBytesRead)) Thanks, Mark

David Crow

But then you'd have to allocate one byte of memory from the heap (i.e., cReceivedChar = new char). Why not just use this instead:

char cReceivedChar;
while(port.Read(&cReceivedChar, 1, overlapped, &dwBytesRead))
...

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

yep, it seems to be better to use a reference. with this procedure my string output is only "0.1922" remember incoming: 924.09,-4.06,676.26,0.156488,-0.653856,-0.714853,-0.192273 It looks like that the first values running through the digits in the dialog..mmh?? Is this so complicate to give out the full string? Thanks, Mark

David Crow

macmac38 wrote: yep, it seems to be better to use a reference. My example was not a reference. It was a pointer, exactly what the Read() method is expecting. Had it wanted a reference, the signature would look like:

BOOL Read(void& lpBuf, ...);

You mentioned that 924.09,-4.06,676.26,0.156488,-0.653856,-0.714853,-0.192273 is arriving at the serial port. How are you verifying this?

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

This is the position string from a robot arm and every time the same format. x,y,z shift and quaternion 1-4 I have to extract them later from the string in float variables Thanks, Mark

David Crow

That's all well and good, but in your original post, you indicated that "This arrives at my serial port," implying that the data is indeed there. That's why I asked how you were verifying it. In the while loop that you are using to read the incoming data, how many times does the loop execute? After each call to Read(), what is the value of dwBytesRead? If the incoming data is always in the same format and is always less than X bytes, you might could use this instead:

char received[X];
port.Read(received, sizeof(received), overlapped, &dwBytesRead);
received[dwBytesRead] = '\0';

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

received[dwBytesRead] = '\0'; what does this line mean? I have "access violation" with this line. Thanks, Mark

David Crow

macmac38 wrote: what does this line mean? It appends a \0 character to the end of the string. macmac38 wrote: I have "access violation" with this line. How big is the received array, and what is the value of dwBytesRead?

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

char received[40]; port.Read(received, sizeof(received), overlapped, &dwBytesRead); TRACE( "Bytes read: %d",dwBytesRead); received[dwBytesRead] = '\0'; SetDlgItemText(IDC_SHIFT_X, received); Bytes read: 1Bytes read: -858993460Bytes read: -858993460Bytes read: -858993460 Thanks, Mark

David Crow

macmac38 wrote: TRACE( "Bytes read: %d",dwBytesRead); Since dwBytesRead is a DWORD, you'll need to use the %lu format. Nonetheless, if received[] is only 40 bytes, trying to access the -858993460th position, or any value outside of 0-39, is obviously wrong. Check the return value of Read() beforehand. In your original post, you indicated that 58 characters were being sent to the serial port. You'll need room for that plus one more so received[] should be 59 not 40. This is not the root of the problem, however, but will eventually need to be addressed.

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)

macmac38

char received[59]; port.Read(received, sizeof(received), overlapped, &dwBytesRead); TRACE( "Bytes read: %lu",dwBytesRead); received[dwBytesRead] = '\0'; SetDlgItemText(IDC_SHIFT_X, received); Bytes read: 1 I donn't know where's the problem? Normaly it seems easy..? Like to solve it today ;-) Thanks, Mark

David Crow

macmac38 wrote: I donn't know where's the problem? Are you sure a problem exists? Maybe there was only one byte available to read.

"The pointy end goes in the other man." - Antonio Banderas (Zorro, 1998)