Question

我的项目涉及通过XBEE Wifi将命令从PuTTY发送到Dragon12-Plus2开发板（CodeWarrior 5.1）。我正在使用4个命令来控制来自LBEbooks.com的LBE_DRAGON12_Plus项目中Dragon板上的LED {led_enable（），leds_on（十六进制或十进制的整数），leds_off（）和led_disable（）}。这是我的代码：

// Final Project: Control leds via XBEE Wifi
#include <hidef.h>      /* common defines and macros */
#include <mc9s12dg256.h>     /* derivative information */
#include "queue.h"
#pragma LINK_INFO DERIVATIVE "mc9s12dg256b"

#include "main_asm.h" /* interface to the assembly module */

void main(void){
  int x,y,z,parser;
  int cmdLen = 0;                    
  char c,d,e;
  char cbuff[20];
  PLL_init();                             // set system clock frequency to 24 MHz 
  lcd_init();                             // enable lcd
  SCI1_init(9600);                        // initialize SCI1 at 9600 baud
  SCI0_init(9600);                        // initialize SCI0 at 9600 baud
  seg7_disable();                         // disable 7 segment display
  led_enable();                           // enable 8 leds
  while(1){
    if(SCI1SR1_RDRF == 1){
      c = inchar1();
      cbuff[cmdLen] = c;
      outchar0(cbuff[cmdLen]);
      if(c == ';'){
        if((cbuff[0] == 'l') && (cbuff[10] == ';')){
          leds_off();
          cmdLen = 0;
        } 
        else if((cbuff[0] == 'l') && (cbuff[12] == ';')){
          led_enable();
          cmdLen = 0;
        } 
        else if((cbuff[0] == 'l') && (cbuff[4] == 'd')){
          led_disable();
          cmdLen = 0;
        }
        else if((cbuff[0] == 'l') && (cbuff[13] == ';')){
          d = cbuff[10];
          e = cbuff[11];
          switch(d){                        // change first number to integer
            case('0'):
              x = 0x00;
              break;
            case('1'):
              x = 0x10;
              break;
            case('2'):
              x = 0x20;
              break;
            case('3'):
              x = 0x30;
              break;
            case('4'):
              x = 0x40;
              break;
            case('5'):
              x = 0x50;
              break;
            case('6'):
              x = 0x60;
              break;
            case('7'):
              x = 0x70;
              break;
            case('8'):
              x = 0x80;
              break;
            case('9'):
              x = 0x90;
              break;
            case('a'):
              x = 0xa0;
              break;
            case('b'):
              x = 0xb0;
              break;
            case('c'):
              x = 0xc0;
              break;
            case('d'):
              x = 0xd0;
              break;
            case('e'):
              x = 0xe0;
              break;
            case('f'):
              x = 0xf0;   
              break;
            default:
              break;
          }
          switch(e){                        // change second number to integer
            case('0'):
              y = 0x00;
              break;
            case('1'):
              y = 0x01;
              break;
            case('2'):
              y = 0x02;
              break;
            case('3'):
              y = 0x03;
              break;
            case('4'):
              y = 0x04;
              break;
            case('5'):
              y = 0x05;
              break;
            case('6'):
              y = 0x06;
              break;
            case('7'):
              y = 0x07;
              break;
            case('8'):
              y = 0x08;
              break;
            case('9'):
              y = 0x09;
              break;
            case('a'):
              y = 0x0a;
              break;
            case('b'):
              y = 0x0b;
              break;
            case('c'):
              y = 0x0c;
              break;
            case('d'):
              y = 0x0d;
              break;
            case('e'):
              y = 0x0e;
              break;
            case('f'):
              y = 0x0f;
              break;
            default:
              break;
          }
          z = (x + y);                      // add integers together
          leds_on(z);                       // display hex number on leds
          cmdLen = 0;                       // reset parser

        }
        for(parser = 0; parser < 20; parser++){
          cbuff[parser] = '\0';
        }
      }
      cmdLen++;
    }  
  }
}

我发送的第一个命令按预期工作（例如，如果我键入leds_on（0xaa）;在PuTTY中，4个相应的指示灯亮起），但是我发送的任何命令都不执行任何操作。我不确定如何正确地重置cbuff，以便每次都以相同的方式运行，而不仅仅是第一次。

编辑显示建议的更改！

Answer 1

如果要清除数组中的所有元素，那么memset将是明智的选择：

memset(cbuff, 0, 20);

话虽如此，我非常赞同@John Bollinger，将输入和解释功能分开是明智的。

但是，根据应用程序memset可能有点矫枉过正。通过简单地将数组的元素设置为空字符（'\0'），也可以实现简单的清除数组。只需为数组中的每个元素调用cbuff[0] = '\0'。

Answer 2

多么奇怪的解析器。我建议你将输入阶段与命令解释阶段分开。也就是说，读取和计算字符，直到您收到分号（;）命令终止符，然后才尝试解释该命令。

仅在每次尝试解释命令后，重置parser变量（更准确地命名为command_length），无论是否成功，或者缓冲区是否即将溢出。 / p>

如何正确重置缓冲区，以便下次读取正确？

2 个答案: