c - 为什么我输入两次 't' 时得到的是垃圾值？

Question

我正在尝试在 MSP430G2553 上实现一个温度传感器，用于读取和输出温度读数以及时间戳。基本要求如下:

• t:显示系统当前时间。以 hhmmss 的形式将系统当前时间输出到串行终端，其中 hh 是 24 小时格式的两位数小时，mm 是小时中的两位数分钟，ss 是分钟中的两位数秒。

• s:设置系统当前时间。将系统的当前时间设置为提供的参数。命令及其参数之间不会有空格。参数的格式为 hhmmss，其中 hh 是 24 小时制的两位数小时，mm 是小时中的两位数分钟，ss 是分钟中的两位数秒。

• o:显示最早的温度读数及其时间戳。将最旧的温度读数及其时间戳输出到串行终端。输出的格式必须为 hhmmss: T，其中 hh 是 24 小时格式中的两位数小时，mm 是小时中的两位数分钟，ss 是分钟中的两位数秒，T 是测量的温度。必须从列表中删除显示的条目。如果未执行任何读数，则会显示消息“未记录温度”。必须显示。

• l:显示所有温度读数及其时间戳。将所有温度读数及其时间戳输出到串行终端。输出应按时间顺序排列，最旧的在前。输出的格式必须为 hhmmss: T，其中 hh 是 24 小时格式中的两位数小时，mm 是小时中的两位数分钟，ss 是分钟中的两位数秒，T 是测量的温度。所有条目都必须从列表中删除。如果未执行任何读数，则会显示消息“未记录温度”。必须显示。您的系统还必须满足以下要求:

• 您必须每五 (5) 分钟捕获一个新的温度读数，并至少存储 32 个温度读数。如果已达到最大温度读数，请丢弃最旧的温度读数，然后再存储新的温度读数。

输入“t”两次后出现错误。下次我输入“t”时，它会输出 3 或 4 个垃圾值。当我注释掉“s”大小写或将“s”更改为不同的变量时，switch语句中的“t”大小写运行得非常好，但是分配的大小写需要在“s”输入到uart中的情况下。到目前为止我尝试过的包括:更改变量，注释掉第二种情况，更改 S 情况的值以确保由于该情况而产生垃圾值，对代码的某些部分进行括号括起来，修改t 案例的整个第一部分更加有效，但仍然出现相同的错误，等等。任何建议都会非常有帮助。 Picture of Garbage Values

#include "msp430g2553.h"

volatile short unsigned int i=0,j=0,k=0;//This the integer that track the number of cycles
volatile short unsigned int timercount=0;//This the integer that track the number of cycles
volatile unsigned int GT=2600;//variable that sets the global time in seconds of the system
volatile char inputcharacters[16];//Input character storage array
volatile unsigned int tempature[32];//
volatile unsigned int tempaturetime[32];
volatile unsigned int d[6]={0,0,0,0,0,0},test[7];
volatile unsigned int h=0;
volatile unsigned int m=0;
volatile unsigned int ss=0;
int temp=0;

void main(void)
{
    WDTCTL = WDTPW + WDTHOLD;//Disabling watchdogtimer

    P1DIR |= BIT0;//setting p1 bit 1 to outy
    P1OUT &= ~BIT0;//Shutting off the LED

    /* UART config for 9600 baud with SMCLK*/
    IE2 = 0x00;//interrupts off on UART
    UCA0CTL1 = 0x81;//using SMCLK(1Mhz) and software reset enable
    UCA0CTL0 = 0x00;//eight data bits, no parity, one stop bit
    UCA0MCTL = 0x02;//Setting UCBRSx to 1 to set clock rate to 9600
    UCA0BR0 = 0x68;//set clock rate to 9600 from 1Mhz
    UCA0BR1 = 0x00;//set clock rate to 9600
    UCA0STAT = 0x00;//error check and parity, and frame error check all off

    TACTL = 0x1D0;// sets timer a to use the 32.768kh clock with 8 divisor and in up mode
    TACCR0 = 0x1001;// setting the timer to count up to 4097 to get a 1hz(1s) oscillation
    TACCTL0 &= 0xFFEF;// disable capture compare interrupts

    P1SEL|=0X06;// According to slas735j both p1.1(0X02) and p1.2(0X04) need to be set on to receive and send on the UART
    P1SEL2|=0X06;

    ADC10CTL0 &= ENC;
    ADC10CTL0 = SREF_1 | ADC10SHT_3 | REFON | ADC10ON;
    ADC10CTL1 = INCH_10 | ADC10DIV_2 | ADC10SSEL_3;
    //ADC10CTL0 = 0x3030;
    //ADC10CTL1 = 0xA038;
    ADC10CTL0 &= 0xFFFC;

    /* Enable USCI logic and Enable interrupts */
    UCA0CTL1 &= 0xFE;//turning on USCI logic
    IE2 = 0x01;// Sets UART to receive interrupt
    TACCTL0 = CCIE;// set timerA interupts on done last
    __bis_SR_register(LPM3_bits + GIE);//found on TI wiki
}

#pragma vector=USCIAB0RX_VECTOR// This vector name was pulled from the webcources code
__interrupt void uartinput(void)//this form found on TI's wiki
{
    inputcharacters[i] = UCA0RXBUF;
            UCA0TXBUF = UCA0RXBUF;
            if((i>14)||(inputcharacters[i]=='\r'))
                    {if(inputcharacters[i]=='\r'){inputcharacters[i]=0;}//to reduce i, the number of inputs so that we dont count the \r
                      for(j=0;j<i;j++){switch(inputcharacters[j]) {//switch statement that check the input characters and outputs the correct morse
                      case 't' :

                          h=GT/3600;
                          m=(GT-h*3600)/60;
                          ss=GT-h*3600-m*60;

                          d[0]=(h/10);
                          d[1]=(h-(d[0]*10));
                          d[2]=(m/10);
                          d[3]=(m-(d[2]*10));
                          d[4]=(ss/10);
                          d[5]=(ss-(d[4]*10));
                          for(k=0;k<6;k++)
                          {while((IFG2 & 0x02)==0)
                          {;}
                          UCA0TXBUF=d[k]+48;
                          test[k]=d[k]+48;}
                          while((IFG2 & 0x02)==0)
                                {;}
                          UCA0TXBUF='\r';
                          break;
                      case 'x' ://If this break case is not set to 's' it will not cause the error in the 't' case, no idea why
                          GT=0;
                          temp=(inputcharacters[j+1]-0x30)*10;//
                          temp+=(inputcharacters[j+2]-0x30);
                          GT+=(temp*3600);
                          temp=(inputcharacters[j+3]-0x30)*10;
                          temp+=(inputcharacters[j+4]-0x30);
                          GT+=(temp*60);
                          temp=(inputcharacters[j+5]-0x30)*10;
                          temp+=(inputcharacters[j+6]-0x30);
                          GT+=temp;
                          j+=7;
                         break;
                      case 'o' :
                          {;}
                          break;
                      case 'l' :
                          {;}
                          break;
                      case '\r' :
                          {;}
                          break;
                      default :
                          {i++;}
                          }}
                      i=0;
                      j=0;
                      }
            else{i++;}}

#pragma vector=TIMER0_A0_VECTOR
__interrupt void Timer_interupt(void)
{
    if(GT<86459)//24:00:59 in seconds
        {GT++;}//add a second
    else
        {GT=60;}//00:01:00
    if(timercount<299)//300s=5min
    {timercount++;}
    else{timercount=0;
         ADC10CTL0 |= 0x3;// turn on ADC
         while(ADC10CTL1 & ADC10BUSY)//loop to alow for ADC to finish sampeling
                 {;;}
         ADC10CTL0 & 0xFFFE;//Shuting off the ADC after sampeling
         tempature[i]=ADC10MEM;//Storing the tempature
         tempaturetime[i]=GT;// storing the time
         if(i<31)
             {i++;}
         else
             {i=0;}
        }
}

Answer 1

所发布代码的一个非常明显的问题是，当发现当前正在检查的输入字符为x时，必须在用户输入后输入新的日期戳(作为6位数字) x 但在用户输入 \r 之前，这可以工作，但没有代码强制执行该输入序列。

一般来说，uart 输入函数应包含多个“状态和子状态”，其中命令字符导致代码切换到“状态”，并且该状态将具有实际时间戳的子状态。因此，当用户输入命令(例如 x)时，会设置一个模式，其中输入的接下来的 6 位数字是新的时间戳。

建议设计一个“上下文敏感”状态图，然后实现该图。

除此之外，输入缓冲区的大小将减少到 1 个字节

在许多排序问题中，有一个事实是，当积累一些数字时，会说这样的话

int temp = 0;
temp += inputchar[x] - 0x30;
temp += inputchar[x+1] - 0x30;
...

不会产生预期的结果。

产生预期结果的是:

int temp = 0;
temp += inputchar[x] - 0x30;
temp = temp*10 + (inputchar[x+1] - 0x30);
....
temp = temp*10 + (inputchar[x+5] - 0x30);
GT = temp;

当然，在尝试将该输入应用于 temp 的计算之前，使用 if( isdigit( inputchar[x] ) 测试每个 inputchar

为了便于开发，强烈建议一次只尝试执行一个命令并且当该命令正常工作时(广泛使用调试器以使测试变得容易。例​​如jtag该芯片的调试器。)然后继续执行下一个要实现的命令。

变量i正在两个中断处理程序中被修改。因此，当一个中断处理程序修改 i 时，另一个中断的值就会被破坏。

一整天(24 小时)是 878400 秒，因此值 86459 不正确。另外，这是代码中的一个“神奇”数字。魔法数字没有依据。 “神奇”的数字使代码更加难以理解、调试和维护。建议使用 enum 语句或 #define 语句为该“神奇”数字指定一个有意义的名称，然后在整个代码中使用该有意义的名称。注意:将 GT 计数器重置为 60 是不正确的，在发布的代码中应将其重置为 0。

对于“神奇”数字 300 也存在类似的考虑。

当输入命令为l时，发布的代码没有任何 Action ，因此我们无法分析这样的(不存在的)代码来确定它出了什么问题。

在t命令的处理中:

GT 变量在访问之间可能会发生修改。建议保存到"new"变量并从该新变量进行所有计算。

h、m 和 s 的计算正在对相同的中间变量执行大量重新计算。建议:

int myGT = GT;
h = myGT / 3600;
myGT %= 3600;
m = myGT / 60;
s = myGT % 60;

数组test[]已设置但未使用。这是生产代码中的资源浪费。建议其存在和设置由某些-D编译器参数确定。