优化这段代码 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 ) { LCD_DisString((i%9)+1, 16, (char *)gcszLcdComPcl[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } else 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 if( i == COM_NET_DIR ) { LCD_DisString((i%9)+1, 16, (char *)gcszStateName1[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } else if( i == COM_NET_YXM || i == COM_NET_YKM) { LCD_DisString((i%9)+1, 16, (char *)gcszPoint[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } else if( i == COM_NET_YCM ) { LCD_DisString((i%9)+1, 16, (char *)gcszData[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } else if( i == COM_NET_JM ) { LCD_DisString((i%9)+1, 16, (char *)gcszLcdJmMode[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } else if( i == COM_NET_AREA ) { LCD_DisString((i%9)+1, 16, (char *)gcszLcdSynctmMode[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } else if( i == COM_NET_SYNCTM ) { LCD_DisString((i%9)+1, 18, (char *)gcszLcdRSMode[(int)gRunPara.COMMS_NetInfo[netid][i].val]); } } 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); } }
时间: 2023-12-14 14:38:35 浏览: 56
可以尝试以下优化:
1. 将重复的代码提取成函数,以减少冗余代码和提高可读性。
2. 用数组来存储不同的处理方式,而不是使用大量的 if-else 语句,以提高代码的可维护性和可扩展性。
3. 在循环之前计算出需要处理的数据数量,以避免无用的循环迭代。
4. 避免在循环内部重复执行相同的操作,例如多次将同一个变量转换为字符串。
5. 为常量字符串和数组添加 const 关键字,以确保它们不会被修改。
下面是一个可能的优化版本:
```c
const char* const kLcdComPcl[] = { "0", "1", "1.5", "2" };
const char* const kLcdStateName1[] = { "DISABLE", "ENABLE" };
const char* const kLcdPoint[] = { "CLOSE", "OPEN" };
const char* const kLcdData[] = { "SET", "RESET" };
const char* const kLcdJmMode[] = {"RS232", "RS485"};
const char* const kLcdSynctmMode[] = {"LOCAL", "NETWORK"};
const char* const kLcdRSMode[] = {"MASTER", "SLAVE"};
void PrintIpValue(float value, char* buffer) {
unsigned char bytesforIP[4];
*(float*)bytesforIP = value;
sprintf(buffer, "%d.%d.%d.%d", bytesforIP[0], bytesforIP[1], bytesforIP[2], bytesforIP[3]);
}
void PrintLcdValue(int index, int row, float value, int not, int netid, int j) {
char szVal[16];
char buf[20];
memset(szVal, 0, sizeof(szVal));
sprintf(szVal, "%s", gRunPara.COMMS_NetInfo[netid][index].szName);
LCD_DisString((index % 9) + 1, row, szVal);
if (index >= COMMS_NET_TOTALSUM) {
return;
}
if (((index >= COM_NET_PCL) && (index <= COM_NET_DIR)) || ((index >= COM_NET_YXM) && (index <= COM_NET_SNTP)) || (index == COM_NET_SYNCTM)) {
if (index == COM_NET_PCL) {
LCD_DisString((index % 9) + 1, 16, kLcdComPcl[(int)value]);
} else if (((index > COM_NET_PCL) && (index < COM_NET_DIR)) || index == COM_NET_SNTP) {
if (index == COM_NET_IP) {
PrintIpValue(value, buf);
LCD_DisString((index % 9) + 1, 10, buf);
int len = strlen(buf);
if (not == 2) {
Lcd_IP_Not(netid, index, j, len, buf);
}
} else {
PrintIpValue(value, szVal);
LCD_DisString((index % 9) + 1, 10, szVal);
}
} else if (index == COM_NET_DIR) {
LCD_DisString((index % 9) + 1, 16, kLcdStateName1[(int)value]);
} else if (index == COM_NET_YXM || index == COM_NET_YKM) {
LCD_DisString((index % 9) + 1, 16, kLcdPoint[(int)value]);
} else if (index == COM_NET_YCM) {
LCD_DisString((index % 9) + 1, 16, kLcdData[(int)value]);
} else if (index == COM_NET_JM) {
LCD_DisString((index % 9) + 1, 16, kLcdJmMode[(int)value]);
} else if (index == COM_NET_AREA) {
LCD_DisString((index % 9) + 1, 16, kLcdSynctmMode[(int)value]);
} else if (index == COM_NET_SYNCTM) {
LCD_DisString((index % 9) + 1, 18, kLcdRSMode[(int)value]);
}
} else {
sprintf(szVal, "%.0f", value);
if (index >= 12 && index <= 14) {
LCD_DisString((index % 9) + 1, 16, szVal);
} else {
LCD_DisString((index % 9) + 1, 18, szVal);
}
}
}
int num_values = (COMMS_NET_TOTALSUM < 9) ? COMMS_NET_TOTALSUM : 9;
for (int i = 0; i < num_values; i++) {
memset(szVal, 0, sizeof(szVal));
PrintLcdValue(i, 0, gRunPara.COMMS_NetInfo[netid][i].val, not, netid, j);
}
```
阅读全文