8个IF语句

2万 · 2022-6-11 15:30

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

1万 · 2022-6-11 15:32

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

2万 · 2022-6-11 15:34

原意是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万 · 2022-6-11 15:36

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;
            }

1万 · 2022-6-11 15:39

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

1万 · 2022-6-11 15:41

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

1万 · 2022-6-11 15:44

来个for循环。。。

2万 · 2022-6-11 15:46

#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万 · 2022-6-11 15:48

高手啊，牛X

1万 · 2022-6-11 15:50

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

只看该作者 · 2022-6-11 15:50

for(int i=0;i<8;i++)
{
uin8_t mask = 1<<i;
if((err_pra[i].err_name_date != 0) && ((err_chl_date[i].chl_date &= mask) == 0x01))
{
g_AckBuf[35] |= mask;
}
else
{
g_AckBuf[35] &= ~mask;
}
}

1万 · 2022-6-11 15:53

用switch语句试试

1万 · 2022-6-11 16:00

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

1万 · 2022-6-11 16:02

当然可以了

1万 · 2022-6-11 16:04

if里边嵌套for就可以

1万 · 2022-6-11 16:07

if 嵌套for 然后在for里边嵌套if就行了

1万 · 2022-6-11 16:10

嵌套多了会不会速度变慢

1万 · 2022-6-11 16:13

这种有规律的直接用查表法，和函数指针打包成一个结构体，代码清晰干净

2万 · 2022-6-11 16:16

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

只看该作者 · 2022-6-19 14:34

if少用，容易头晕

8个IF语句

精华达人奖章

湍急之河流

七世轮回

技术导师奖章