Проблема с преобразованием String в unsigned long

Question

У меня есть небольшая проблема с преобразованием массива строки символов в unsigned long. Это мой ввод для executeCommand ().

0001000118218; 326

И это то, что я получаю.

Полученная команда: 0001000118218; 326

код передачи: 1821

период передачи: 32

Я не понимаю, почему выбрасывается последняя цифра ...

#define id_length 4
#define command_length 4
const String my_id = "0001";
//Command execution methods
void executeCommand(String inputData) {
  if(!my_id.equals(inputData.substring(0, id_length))) {
    return;
  }

  Serial.print("Received command: ");
  Serial.println(inputData);
  String command = inputData.substring(id_length, id_length + command_length);
  String parameters = inputData.substring(id_length + command_length);


  for (int i = 0; i < 3; i++) {
  if(command == "0001") {    //RF power command
    unsigned long rfid_long;
    unsigned long periodid_long;
    char rfid[parameters.indexOf(';')];
    char periodid[3];
    parameters.substring(0, parameters.indexOf(';')).toCharArray(rfid, parameters.indexOf(';'));
    parameters.substring(parameters.indexOf(';') + 1).toCharArray(periodid, 3);
    rfid_long = strtoul(rfid, NULL, 10);
    periodid_long = strtoul(periodid, NULL, 10);
    sendRF(rfid_long, periodid_long);
  }
  else {
    Serial.println("Unknown command received");
  }
  delay(10);
  }
}

void sendRF(unsigned long transmitCode, unsigned int transmitPeriod) {
 //digitalWrite(transmitLedPin, HIGH);
 Serial.print("transmit code: ");
 Serial.println(transmitCode);
 Serial.print("transmit period: ");
 Serial.println(transmitPeriod);
 RemoteSwitch::sendTelegram(generateTelegram(transmitCode, transmitPeriod), rfSubmitPin);
 //digitalWrite(transmitLedPin, LOW);
}

Answer 1

char periodid[3];

Должно быть char periodid[4], если вы хотите, чтобы в нем была строка длиной 3;3 элемента для содержимого и 1 для нулевого терминатора.

Вам также придется изменить toCharArray(periodid, 3); на toCharArray(periodid, 4); или лучше, toCharArray(periodid, sizeof(periodid));

Answer 2

Что если вы преобразуете команду в массив символов и используете strchr?

char* separator = strchr(command, ';');
if (separator == NULL) return;

char* rfid = command + id_length;
char* periodid = separator + 1;

unsigned long rfid_long = strtoul(rfid, &separator, 10);
unsigned long periodid_long = strtoul(periodid, NULL, 10);