#2 Refactor Parsing: Perhaps using a String array instead of three seperate String variables?

array docs

Perhaps it should be: where_at?

String raw_parsings[3];

Then the parse_by_comma and parse_order functions would need to update their access to that. (Perhaps we make parse_by_comma so it does return... String[]?)

Todo:

• Moved from 3 seperate variables into a single variable array.
• Changed parse_by_comma to return value, thus removing the need of the global array.

Unfortunately if I want to use an array to store the parsed code, bit_size, and repeat boolean...

It appears I can't get the result to persist... it's a pointer and thus when we finally get to use the results it gets wiped. (So we are using a global array)

Todo:

• Get away from String and use const char* instead.

Here's a sample decoder that decodes the hex values and other things. (I'm not 100% sure what the 3rd value is supposed to be.)

To compile in linux: gcc -o hexdecode hexdecode.c

You might also need:

    if (in == '\r') {
      continue;
    }

#include <ctype.h>
#include <stdio.h>
#include <string.h>

// characters to send for testing
const char *TEST = "AA550101,32,A\n";
int pos = 0;

const int DEBUG_OUTPUT = 1;

// replacement for Serial.available()
int available(void) { return (pos != strlen(TEST)); }

// replacement for Serial.read()
char read(void) {
  char c = TEST[pos];
  ++pos;
  return c;
}

const char hexadecimal[] = "0123456789ABCDEF";

int hextoi(char hexch) {
  char *pos = strchr(hexadecimal, toupper(hexch));
  if (pos == NULL) {
    printf("WARN: Invalid hex digit [%c/%d]\n", hexch, hexch);
    return 0;
  }
  return (pos - hexadecimal);
}

// setup
int main(void) {
  long code = 0;
  int count = 0;
  int repeat = 0;
  int parse_item = 0;

  printf("Input: [%s]\n\n", TEST);

  while (available()) {
    char in = read();
    if (in == ',') {
      // Ok, next item
      parse_item++;
      if (parse_item > 2) {
        printf("ERROR: TOO MANY ITEMS/','.\n");
        return 2;
      }
      continue;
    }
    if (in == '\n') {
      if (parse_item == 2) {
        // OK!
        printf("DECODED OUTPUT: %lX, %d, %d\n", code, count, repeat);

      } else {
        printf("ERROR: Missing items, I saw %d\n", parse_item + 1);
        return 2;
      }
      code = 0;
      count = 0;
      repeat = 0;
      parse_item = 0;
      continue;
    }
    switch (parse_item) {
    case 0:
      if (DEBUG_OUTPUT)
        printf("Before: %lX %c %d\n", code, in, hextoi(in));
      code = code << 4;
      if (DEBUG_OUTPUT)
        printf("After:  %lX\n", code);
      code += hextoi(in);
      if (DEBUG_OUTPUT)
        printf("After:  %lX\n", code);
      break;
    case 1:
      count *= 10;
      count += hextoi(in);
      break;
    case 2:
      repeat *= 10;
      repeat += hextoi(in);
      break;
    }
  }

  printf("DONE!\n");
  return 0;
}

\o/ Ok well we've not got these things done:

• Refactored parsing
• Got away from String

The Arduino code looks like it's complete. (Well except for #3 where we send a code and get counted twice)