LCD打印字符两次？答案

【问题标题】：LCD prints the character twice?LCD打印字符两次？
【发布时间】：2020-09-20 09:09:49
【问题描述】：

我的 8051 项目有这个问题。我想用键盘输入数字和操作，然后用 16x2 LCD 打印它们，结果使用 8051 微控制器。一切正常，我可以进行基本操作并查看结果。但是当我从键盘按字符时，LCD 会打印两次字符。我看不出问题出在哪里。如果你能帮助我，我会很高兴。谢谢你。

#include <reg51.h>

void lcdcmd(unsigned char);
void lcddata(unsigned char);
void MSDelay(unsigned int);
void disp_num(float num);
int get_num(char ch);
void lcdinit();
char scan_key(void);

sfr ldata = 0xA0;
sbit rs = P3^7;
sbit rw = P3^5;
sbit en = P3^6;
sbit r0=P1^0;
sbit r1=P1^1;
sbit r2=P1^2;
sbit r3=P1^3;
sbit c0=P1^4;
sbit c1=P1^5;
sbit c2=P1^6;
sbit c3=P1^7;

void lcdinit(){
    MSDelay(15000);
    lcdcmd(0x30);
    MSDelay(4500);
    lcdcmd(0x30);
    MSDelay(300);
    lcdcmd(0x30);
    MSDelay(600);
    lcdcmd(0x38);
    lcdcmd(0x0F);
    lcdcmd(0x01);
    lcdcmd(0x06);
    lcdcmd(0x80);
}

int main (void)
{
    while(1){
        unsigned int k=0, m=0, n=0;
        int k2, k1;
        char key, key1;
        unsigned char ch2,ch3;

        lcdinit();

        key = scan_key();
        k2 = get_num(key); // sayiya cevirdik. 
        lcddata(key); // lcd de goruntule. 

        ch2 = scan_key();
        lcddata(ch2);

        key1 = scan_key();
        k1 = get_num(key1);
        lcddata(key1);

        ch3 = scan_key();
        lcddata(ch3);
        lcdcmd(0x01);

        switch(ch2)
        {
        case 'A':
            disp_num(k1 + k2);
            break;

        case 'B':
            disp_num(k2 - k1);
            break;

        case 'C':
            disp_num(k2 * k1);
            break;

        case 'D':
            disp_num(k2 / k1);
            break;
        }
        return 0;
    }
}

void lcdcmd(unsigned char value)
{
    ldata = value;      
    rs = 0;
    rw = 0;
    en = 1;            
    MSDelay(50);
    en = 0;
    MSDelay(50);
}

void lcddata(unsigned char value)
{
    ldata = value;  
    rs = 1;
    rw = 0;
    en = 1;          
    MSDelay(50);
    en = 0;
    MSDelay(50);
}

void MSDelay(unsigned int itime)
{
    unsigned int i, j;

    for(i = 0;i < itime; i++)           
        for(j = 0; j < 5; j++);       
}

char scan_key()
{
    unsigned char c;
    c='s';
    while(!(c=='0' && c=='1' && c=='2' && c=='3' && c=='4'
                   && c=='5' && c=='6' && c=='7' && c=='8'
                   && c=='9' && c=='+' && c=='-' && c=='#'
                   && c=='$' && c=='*' && c=='/' ))
    {
      r0 = 0; r1 = 1; r2 = 1; r3 = 1;

      if(c0==0 && r0==0){ lcddata('7'); MSDelay(100000); return c = '7';}
      if(c1==0 && r0==0){ lcddata('8'); MSDelay(100000); return c = '8';}
      if(c2==0 && r0==0){ lcddata('9'); MSDelay(100000); return c = '9';}
      if(c3==0 && r0==0){ lcddata('A'); MSDelay(100000); return c = 'A';}

      r0 = 1; r1 = 0; r2 = 1; r3 = 1;

      if(c0==0 && r1==0){ lcddata('4'); MSDelay(100000); return c= '4';}
      if(c1==0 && r1==0){ lcddata('5'); MSDelay(100000); return c= '5';}
      if(c2==0 && r1==0){ lcddata('6'); MSDelay(100000); return c= '6';}
      if(c3==0 && r1==0){ lcddata('B'); MSDelay(100000); return c= 'B';}

      r0 = 1; r1 = 1; r2 = 0; r3 = 1;

      if(c0==0 && r2==0){ lcddata('1');MSDelay(100000); return c= '1';}
      if(c1==0 && r2==0){ lcddata('2');MSDelay(100000); return c= '2';}
      if(c2==0 && r2==0){ lcddata('3');MSDelay(100000); return c= '3';}
      if(c3==0 && r2==0){ lcddata('C');MSDelay(100000); return c= 'C';}

      r0 = 1; r1 = 1; r2 = 1; r3 = 0;

      if(c0==0 && r3==0){ lcddata('*');MSDelay(100000);return c= '*';}
      if(c1==0 && r3==0){ lcddata('0');MSDelay(100000);return c= '0';}
      if(c2==0 && r3==0){ lcddata('#');MSDelay(100000);return c= '#';}
      if(c3==0 && r3==0){ lcddata('D');MSDelay(100000);return c= 'D';}
    }
    return 0;
}

int get_num(char ch)         //convert char into int
{
    switch(ch)
    {
    case '0': return 0; break;
    case '1': return 1; break;
    case '2': return 2; break;
    case '3': return 3; break;
    case '4': return 4; break;
    case '5': return 5; break;
    case '6': return 6; break;
    case '7': return 7; break;
    case '8': return 8; break;
    case '9': return 9; break;
    }
    return 0;
}

void disp_num(float num)            //displays number on LCD
{
    unsigned char UnitDigit  = 0;  //It will contain unit digit of numb
    unsigned char TenthDigit = 0;  //It will contain 10th position digit of numb
    unsigned char decimal = 0;
    int j;
    int numb;
    j=(int)(num*10);
    numb=(int)num;

    TenthDigit = (numb/10);          // Findout Tenth Digit

    if( TenthDigit != 0)             // If it is zero, then don't display
      lcddata(TenthDigit+0x30);  // Make Char of TenthDigit and then display it on LCD

    UnitDigit = numb - TenthDigit*10;

    lcddata(UnitDigit+0x30);     // Make Char of UnitDigit and then display it on LCD
    MSDelay(2000000);
}

【问题讨论】：

您是否尝试过简化问题或进行任何调试以缩小问题范围？
在scan_key 中，while 循环的条件没有意义。您不能同时等于多个值。另外，在从该函数返回之前为局部变量 c 赋值有什么意义？
c=='0' && c=='1' && ...总是假的。你的意思是“或”这些？
在 scan_key 中，while 循环条件是为此，除非按下某个键，否则它将始终处于该循环中。一旦按下一个键，就会分配局部变量 c 并且它是条件之一并退出循环。不能这样做吗？
不存在“这些条件之一”。这是一个整体条件。如果第一部分是真的，那么所有其他部分都是假的。总的来说，这将永远是错误的。

标签： c microcontroller 8051 microprocessors

【解决方案1】：

我通过从这里删除 lcddata() 函数解决了问题：

key = scan_key();
    k2 = get_num(key); // sayiya cevirdik. 
    lcddata(key); // lcd de goruntule. 

    ch2 = scan_key();
    lcddata(ch2);

    key1 = scan_key();
    k1 = get_num(key1);
    lcddata(key1);

    ch3 = scan_key();
    lcddata(ch3);
    lcdcmd(0x01);

我不明白您对 scan_key() 函数中的 while() 想说什么，它在模拟中运行良好。但是现在我想为这个操作的结果创建一个 20 秒的延迟，我用 MSDelay() 函数尝试过，但没有任何改变。

编辑：对不起，现在我意识到了 while() 中的错误。那是一个巨大的错误，我会解决这个问题，谢谢大家。

【讨论】：