C语言问题

2万 · 2018-8-29 08:52

8个IF语句，怎样变简单些？

1万 · 2018-8-29 08:54

楼主程序可以公开吗？贴程序看下吧，这么说看不出什么原因

2万 · 2018-8-29 08:56

原意是8个结构变量只要不为0，并且err_chl_date[0].chl_date的8个不同的位，置1为真，那么数组g_AckBuf[35]的8个位分别置1.要不就为0
if((err_pra[0].err_name_date!=0) && ((err_chl_date[0].chl_date &= 0x01)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x01;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xf7;
                                                                     }

                                                            if((err_pra[1].err_name_date!=0) && ((err_chl_date[1].chl_date &= 0x02)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x02;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xfd;
                                                                     }

            if((err_pra[2].err_name_date!=0) && ((err_chl_date[2].chl_date &= 0x04)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x04;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xfb;
                                                                     }

                                                            if((err_pra[3].err_name_date!=0) && ((err_chl_date[3].chl_date &= 0x08)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x08;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xf7;
                                                                     }

            if((err_pra[4].err_name_date!=0) && ((err_chl_date[4].chl_date &= 0x10)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x10;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xef;
                                                                     }

                                                            if((err_pra[5].err_name_date!=0) && ((err_chl_date[5].chl_date &= 0x20)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x20;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xdf;
                                                                     }

            if((err_pra[6].err_name_date!=0) && ((err_chl_date[6].chl_date &= 0x40)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x40;
                                                                  }
                                                               else {
                     g_AckBuf[35]&=0xbf;
                                                                     }

                                                            if((err_pra[7].err_name_date!=0) && ((err_chl_date[7].chl_date &= 0x80)==0x01))
                                                                  {
                                                                                    g_AckBuf[35]|=0x80;
                                                                  }
                                                         else {
                     g_AckBuf[35]&=0x7f;
                                                                     }

2万 · 2018-8-29 08:59

if((err_pra[0].err_name_date!=0) && ((err_chl_date[0].chl_date &= 0x01)==0x01))
   {
            g_AckBuf[35]|=0x01;
      }
      else {
                     g_AckBuf[35]&=0xf7;
               }
if((err_pra[1].err_name_date!=0) && ((err_chl_date[1].chl_date &= 0x02)==0x02))
      {
            g_AckBuf[35]|=0x02;
               }
      else {
                     g_AckBuf[35]&=0xfd;
               }

if((err_pra[2].err_name_date!=0) && ((err_chl_date[2].chl_date &= 0x04)==0x04))
               {
                              g_AckBuf[35]|=0x04;
               }
      else {
                     g_AckBuf[35]&=0xfb;
               }

if((err_pra[3].err_name_date!=0) && ((err_chl_date[3].chl_date &= 0x08)==0x08))
               {
                     g_AckBuf[35]|=0x08;
                        }
else {
         g_AckBuf[35]&=0xf7;
      }

if((err_pra[4].err_name_date!=0) && ((err_chl_date[4].chl_date &= 0x10)==0x10))
               {
                     g_AckBuf[35]|=0x10;
               }
else {
               g_AckBuf[35]&=0xef;
         }

if((err_pra[5].err_name_date!=0) && ((err_chl_date[5].chl_date &= 0x20)==0x20))
      {
            g_AckBuf[35]|=0x20;
      }
else {
            g_AckBuf[35]&=0xdf;
      }

if((err_pra[6].err_name_date!=0) && ((err_chl_date[6].chl_date &= 0x40)==0x40))
               {
                     g_AckBuf[35]|=0x40;
               }
      else {
                     g_AckBuf[35]&=0xbf;
               }

if((err_pra[7].err_name_date!=0) && ((err_chl_date[7].chl_date &= 0x80)==0x80))
      {
            g_AckBuf[35]|=0x80;
         }
      else {
                     g_AckBuf[35]&=0x7f;
            }

2万 · 2018-8-29 09:01

可以变成一条赋值语句，效率也可以提高一点，但会变得难于理解

1万 · 2018-8-29 09:03

哦。。。。不行。
你的括弧里面有赋值，不能弄掉。。。

2万 · 2018-8-29 09:05

来个for循环。。。

2万 · 2018-8-29 09:08

#define MAX_NUM  (unsigned char)0xff
#define CNT          (unsigned char)0x08

const unsigned char Tab[CNT] = {0x01, 0x02, 0x04 ,0x08, 0x10, 0x20, 0x40, 0x80};
unsigned char u8Loop = 0;

for (; u8Loop < CNT; u8Loop++)
{
      if (0 != err_pra[u8Loop].err_name_date)
      {
               if (Tab[u8Loop]  == (err_chl_date[u8Loop].chl_date & Tab[u8Loop]))
               {
                        g_AckBuf[35] |= ab[u8Loop];
               }
               else
               {
                     g_AckBuf[35] &= (MAX_NUM - ab[u8Loop]);
               }
      }
      else
      {
            g_AckBuf[35] &= (MAX_NUM - ab[u8Loop]);
      }
}

2万 · 2018-8-29 09:12

高手啊，牛X

2万 · 2018-8-29 09:16

楼主写代码发现有很多代码重复的时候首先要考虑到的就是能不能用查表法，自己做个表通过索引值查表

2万 · 2018-8-29 09:19

好的，我明天去单位试一下，多谢各位大侠了哈，结贴了先

只看该作者 · 2018-8-29 11:05

case

只看该作者 · 2018-8-29 11:05

switch case

只看该作者 · 2018-8-30 09:57

用switch语句试试

1万 · 2018-9-3 11:19

用switch语句试试

1万 · 2018-9-3 11:50

或者看看有没有重复项合并下