Arduino шестнадцатеричный декодер строки не работает - PullRequest
Новая
       32

Arduino шестнадцатеричный декодер строки не работает

0 голосов
/ 03 мая 2019

Я создаю HEX-> STR декодер, используя ЖК-экран (отображает HEX, а также декодированные значения), клавиатуру (для ввода кода HEX) и кнопку (нажимая, чтобы декодировать HEX значения). Иногда декодер работает, но иногда он дает сбой и дает неожиданные значения. 

#include <Keypad.h>
#include<LiquidCrystal.h>

String hex; // store the hex values
String text; // store the decoded text
int calcButton = 0;

const byte ROWS = 4; 
const byte COLS = 4; 

char hexaKeys[ROWS][COLS] = {
  {'1', '2', '3', 'F'},
  {'4', '5', '6', 'E'},
  {'7', '8', '9', 'D'},
  {'A', '0', 'B', 'C'}
};

byte rowPins[ROWS] = {9, 8, 7, 6}; 
byte colPins[COLS] = {13, 12, 11, 10}; 

Keypad customKeypad = Keypad(makeKeymap(hexaKeys), rowPins, colPins, ROWS, COLS); 
LiquidCrystal lcd(5, 4, 3, 2, A0, A1);

char nibble2c(char c) {
   if ((c>='0') && (c<='9'))
      return c-'0' ;
   if ((c>='A') && (c<='F'))
      return c+10-'A' ;
   if ((c>='a') && (c<='a'))
      return c+10-'a' ;
   return -1 ;
}

char hex2c(char c1, char c2) {
   if(nibble2c(c2) >= 0)
     return nibble2c(c1)*16+nibble2c(c2) ;
   return nibble2c(c1) ;
}

// resets string values and clears screen
void erase() {
  hex = "";
  Serial.println(hex);
  text = "";
  lcd.clear();
  lcd.setCursor(0,0);
}

// decode the hex values
String calculate(String hex) {
  if (hex.length()%2 != 0) {
    lcd.setCursor(0,0);
    lcd.print("No of characters");
    lcd.setCursor(0,1);
    lcd.print("needs to be even");
    delay(2000);
    erase();
  }
  else {
    for (int i = 0; i < hex.length() - 1; i+=2){
      for (int j = 1; j < hex.length(); j+=2){
         text += hex2c(hex[i], hex[j]);
      }
    }
  }
}



void setup() {
  pinMode(A2, INPUT);
  lcd.begin(16, 2);
  lcd.setCursor(0,0);
  Serial.begin(9600);
}

void loop() {
  calcButton = digitalRead(A2); // decode button
  char customKey = customKeypad.getKey();

  // if keypad is pressed, add hex values to str
  if (customKey){
    lcd.print(customKey);
    hex += customKey;
    Serial.println(hex);
  }

  // if button is pressed, decode
  if (calcButton == 1) {
    lcd.clear();
    calculate(hex);
    hex = "";
    lcd.print(text);
    text = "";
    delay(1500);
    lcd.clear();
  }
}

Я ввожу 49 и получаю I (что правильно), но когда я ввожу 4949, я ожидаю, что на выходе будет II, но он выводит IIII, а когда я ввожу 6F, ожидая o весь экран размывается и глюки.

1 Ответ

0 голосов
/ 03 мая 2019

Проблема здесь: 

for (int i = 0; i < hex.length() - 1; i+=2){
  for (int j = 1; j < hex.length(); j+=2){
     text += hex2c(hex[i], hex[j]);
  }
}

Обратите внимание, что вы перебираете шестнадцатеричную строку length()*length()/4 раз, комбинируя каждый четный шестнадцатеричный символ из строки с каждым нечетным символом вСтрока, а не только та, что сразу за ней.Для двузначных шестнадцатеричных строк это работает, потому что есть только один нечетный и один четный индексированный символ;для более длинных строк вы получите неправильные результаты.

4949 объединит 4 (# 0) и 9 (# 1), затем 4 (# 0) и 9 (# 3) (неверно!), Затем 4 (# 2) и 9 (# 1) (неверно!), Затем 4 (# 2) и 9 (# 3), что дает вам результат, который 49494949 должен дать вместо 4949.

То, что вы хотите, это просто: 

for (int i = 0; i < hex.length() - 1; i+=2){
  text += hex2c(hex[i], hex[i+1]);
}
...