next topic »

[on:]

I'm reading a string of characters from serial but I'm only interested in one byte of data.  I thought the most efficient way to do this was to use an array of 3 chars (char c[2];) and look for my byte based on the bytes either side...

Code:

while (Serial.available() > 0) {
  c[0] = Serial.read();
  if (c[2] == 'P' && c[0] == 'V') {
    if (bitRead(c[1],0) == 0) {
      // do some stuff here
      }
    }
  c[2] = c[1];
  c[1] = c[0];
}

My code doesn't work.  The last two lines are breaking it but I can't see what I'm doing wrong.  I'm sure it's something basic.

Please help.

Thank you.

[Reply #1 on:]

you're missing a } at the very end

[Reply #2 on:]

Sorry, no, that was just be copy/pasting the bit of code out the bigger program.  I've added it back into the top post, thanks.

There is a whole load of debug lines in my code now as I've been trying to work out what's happening 

[Reply #3 on:]

I thought the most efficient way to do this was to use an array of 3 chars (char c[2] and look for my byte based on the bytes either side...

An array of 2 characters, one of which will be the NULL, you mean. The value in the brackets is the count, not the upper index. You need to allow room for the trailing NULL, too.

[Reply #4 on:]

You absolute star!  Thank you :-)

[Reply #5 on:]

Why should there have to be a NULL, Paul? I see no string or print commands.

[Reply #6 on:]

... use an array of 3 chars (char c[2] and look for my byte based on the bytes either side...

Code:

  if (c[2] == 'P' && c[0] == 'V') {
}

Regardless of whether there is a null or not, an array of 2 characters has indexes 0 and 1. So you can't check index 2 for a P.

[Reply #7 on:]

.. to use an array of 3 chars (char c[2]

In fact, even that should give you pause. An array of 3 chars you say?

Code:

char c [3];

[Reply #8 on:]

Isn't an initialization step missing ? Something like "if numCharsReceived >=3 <code here> else <do nothing>".

Or maybe the fact that c[] has random contents is enough to cover this case ?

[Reply #9 on:]

I've changed this to the following now...

Code:

char c[3] = {0,0,0};

... is that what you mean?

[Reply #10 on:]

If your array "c" is a global or a static, the compiler will helpfully set all elements to zero, unless you tell it to set them to some other value.
Difficult to tell from snippets though (hint)

[Reply #11 on:]

Well, I guess explicitly initializing the array is a simpler way to solve the issue.