if(BytesRead[0] == '/') { // If we catch the null, read the next three things.

on the first line, I'll admit, I get so turned around by what is and is not correct on pointers (and is a prime reason pointers are a bad thing and left behind for the most part) - but that looks to me like it would interpret the VALUE of bytesread[0] as an address - that is probably not a good thing. I think you want either BytesRead or &bytesRead[0].

Secondly, character '/' is not a null, so as long as you are sending it as a terminator/delimiter, the code is fine, just that the comment does not match up. General convention is to send a 0x02 - start of transmission STX (actually a 0x10 and a 0x02 DLE-STX), but by no means a requirement (and end it with a 0x10 and 0x03 DLE-ETX, this help re-synchronize when bytes get dropped, a hint for future reference).

edit; BTW - as a novice programmer, I have seen far worse code from professionals - keep it up!

Who is online

Users browsing this forum: No registered users and 2 guests

You cannot post new topics in this forumYou cannot reply to topics in this forumYou cannot edit your posts in this forumYou cannot delete your posts in this forumYou cannot post attachments in this forum