优化这段代码 for(i = page; i<line; i++) { if(i >= COMMS_NET_TOTALSUM) break; memset(szVal, 0, sizeof(szVal)); sprintf(szVal, "%s", gRunPara.COMMS_NetInfo[netid][i].szName); LCD_DisString((i%9)+1, 0, szVal); if( ((i >= COM_NET_PCL) && (i <= COM_NET_DIR)) || ((i >= COM_NET_YXM) && (i <= COM_NET_SNTP)) || (i == COM_NET_SYNCTM)) { if(((i > COM_NET_PCL) && (i < COM_NET_DIR)) || i == COM_NET_SNTP ) { unsigned char bytesforIP[4]; if (i == COM_NET_IP) { *(float*)bytesforIP = gRunPara.COMMS_NetInfo[netid][i].val; sprintf(buf,"%d%d%d.%d%d%d.%d%d%d.%d%d%d",bytesforIP[0]/100,bytesforIP[0]%100/10,bytesforIP[0]%10,bytesforIP[1]/100,bytesforIP[1]%100/10,bytesforIP[1]%10, bytesforIP[2]/100,bytesforIP[2]%100/10,bytesforIP[2]%10,bytesforIP[3]/100,bytesforIP[3]%100/10,bytesforIP[3]%10); LCD_DisString((i%9)+1, 10, buf); len = strlen(buf); if (not == 2) Lcd_IP_Not(netid,i,j,len,buf); } else { if (i == COM_NET_SNTP ) { *(float*)bytesforIP = gRunPara.COMMS_NetInfo[netid][i].val; sprintf(szVal,"%d.%d.%d.%d",bytesforIP[0],bytesforIP[1],bytesforIP[2],bytesforIP[3]); LCD_DisString((i%9)+1, 14, szVal); } else { *(float*)bytesforIP = gRunPara.COMMS_NetInfo[netid][i].val; sprintf(szVal,"%d.%d.%d.%d",bytesforIP[0],bytesforIP[1],bytesforIP[2],bytesforIP[3]); LCD_DisString((i%9)+1, 10, szVal); } } } else displayNetInfo(LCD_DisString,netid,i); } else { if (i >= 12 && i <= 14) { sprintf(szVal, "%.0f", gRunPara.COMMS_NetInfo[netid][i].val); LCD_DisString((i%9)+1, 16, szVal); } else { sprintf(szVal, "%.0f", gRunPara.COMMS_NetInfo[netid][i].val); LCD_DisString((i%9)+1, 18, szVal); } } }
时间: 2024-02-01 14:13:12 浏览: 18
可以尝试如下优化:
1. 使用局部变量代替多次访问的全局变量,可以提高代码的执行效率。
2. 对于需要多次使用的值,可以在循环外部计算并存储起来,避免重复计算。
3. 可以使用更简洁的语法代替 if-else 语句,例如使用三目运算符。
4. 可以使用更简洁的字符串格式化方法,例如使用 snprintf 函数代替 sprintf 函数。
5. 可以使用更具可读性的变量名,让代码更易于理解。
下面是一个可能的优化版本:
```c
for (int i = page; i < line && i < COMMS_NET_TOTALSUM; i++) {
char* name = gRunPara.COMMS_NetInfo[netid][i].szName;
if (name == NULL) continue; // 防止空指针
memset(szVal, 0, sizeof(szVal));
strcpy(szVal, name);
LCD_DisString((i % 9) + 1, 0, szVal);
if (i >= COM_NET_PCL && i <= COM_NET_DIR ||
i >= COM_NET_YXM && i <= COM_NET_SNTP ||
i == COM_NET_SYNCTM) {
if (i > COM_NET_PCL && i < COM_NET_DIR || i == COM_NET_SNTP) {
unsigned char bytesforIP[4];
float val = gRunPara.COMMS_NetInfo[netid][i].val;
*(float*)bytesforIP = val;
snprintf(buf, sizeof(buf), "%d%d%d.%d%d%d.%d%d%d.%d%d%d",
bytesforIP[0] / 100, bytesforIP[0] % 100 / 10, bytesforIP[0] % 10,
bytesforIP[1] / 100, bytesforIP[1] % 100 / 10, bytesforIP[1] % 10,
bytesforIP[2] / 100, bytesforIP[2] % 100 / 10, bytesforIP[2] % 10,
bytesforIP[3] / 100, bytesforIP[3] % 100 / 10, bytesforIP[3] % 10);
LCD_DisString((i % 9) + 1, i == COM_NET_IP ? 10 : 14, buf);
int len = strlen(buf);
if (not == 2) Lcd_IP_Not(netid, i, j, len, buf);
} else {
displayNetInfo(LCD_DisString, netid, i);
}
} else {
float val = gRunPara.COMMS_NetInfo[netid][i].val;
int row = i >= 12 && i <= 14 ? 16 : 18;
snprintf(szVal, sizeof(szVal), "%.0f", val);
LCD_DisString((i % 9) + 1, row, szVal);
}
}
```