有没有可能用 sprintf 来简化这个语句?
unsigned char text[50];
void decode (unsigned char* msg)
{
// ON LED
if (strstr (msg, "DO1=1") == msg)
{
LATGbits.LATG1 =1 ; //led1 on
sprintf(text,"OK"); // Send the result in HyperTerminal
sendString(text) ;
}
else if (strstr (msg, "DO2=1") == msg)
{
LATGbits.LATG0 = 0 ; //LED 2 on
sprintf(text,"OK"); // Send the result in HyperTerminal
sendString(text) ;
}
else if (strstr (msg, "DO3=1") == msg)
{
LATGbits.LATG13 = 0 ; //LED 3 on
sprintf(text,"OK"); // Send the result in HyperTerminal
sendString(text) ;
}
else if (strstr (msg, "DO4=1") == msg)
{
LATGbits.LATG14 = 0 ; //LED 4 on
sprintf(text,"OK"); // Send the result in HyperTerminal
sendString(text) ;
}
else if (strstr (msg, "DO5=1") == msg)
{
LATGbits.LATG12 = 0 ; //LED 5 on
sprintf(text,"OK"); // Send the result in HyperTerminal
sendString(text) ;
}
// OFF LED
else if (strstr (msg, "DO1=0") == msg)
{
LATGbits.LATG1 =0 ; //LED 1 off
}
else if (strstr (msg, "DO2=0") == msg)
{
LATGbits.LATG0 = 1 ; //LED 2 off
}
else if (strstr (msg, "DO3=0") == msg)
{
LATGbits.LATG13 = 1 ; //LED 3 off
}
else if (strstr (msg, "DO4=0") == msg)
{
LATGbits.LATG14 = 1 ; //LED 4 off
}
else if (strstr (msg, "DO5=0") == msg)
{
LATGbits.LATG12 = 1 ; //LED 5 off
}
}
从上面的代码可以看出,例如,当在 super 终端中输入单词"DO1=1"
时,LED1 会点亮并且"DO1=0"
将关闭 LED。与 DO1 至 DO5 相同。答案取决于 LATGbits
类型的定义。
最佳答案
由于它完全由相互排斥的选项组成,您可以通过一些明智的重构来摆脱所有的 else if
位。
此外,您比较字符串的方式意味着它们必须位于开头,因此您可以使用 strncmp
。
而且,无需将字符串文字复制到缓冲区然后发送缓冲区,您只需发送字符串文字即可。
进行这些更改会产生如下代码:
#define okayReturn { sendString("OK"); return; }
void decode (unsigned char* msg) {
if (strncmp (msg, "DO1=1", 5) == 0) {
LATGbits.LATG1 = 1;
okayReturn;
}
if (strncmp (msg, "DO2=1", 5) == 0) {
LATGbits.LATG0 = 0;
okayReturn;
}
:
if (strncmp (msg, "DO5=0", 5) == 0) {
LATGbits.LATG12 = 1;
return;
}
}
另一种方法是将字符串分成三个部分(DO
、n
和=v
,其中n
是数字 1..5
之一,v
是 0
或 1
)。
然后您可以使用变量指针组合数据驱动方法,例如:
#include <ctype.h>
void decode (unsigned char* msg) {
// Variables to change for each n in DOn=v
static int *vars[] = { &LATGbits.LATG1, &LATGbits.LATG0,
&LATGbits.LATG13, &LATGbits.LATG14, &LATGbits.LATG12 };
// Check content, allowing for upper/lower case.
msg[0] = toupper (msg[0]);
msg[1] = toupper (msg[1]);
if ((msg[0] != 'D') && (msg[1] != 'O')) return;
if ((msg[2] < '1') || (msg[2] > '5')) return;
if (msg[3] != '=') return;
if ((msg[4] != '0') && (msg[4] != '1')) return;
// Special case for LED 1 vs 2-5.
int on = (msg[2] == '1') ? 1 : 0;
// All content validated, make change, first turning on.
if (msg[4] == '1') {
*(vars[msg[2] - '1']) = on;
sendString("OK");
return;
}
// Or turning off.
*(vars[msg[2] - '1']) = 1 - on;
}
注意使用 toupper()
强制前两个字符为大写。这允许您输入任何一种情况,并且它将被相同对待。如果您需要区分大小写,只需删除这两行即可。
虽然这有点复杂,但它要短得多,尤其是如果您决定稍后添加更多 LED。
请记住,只有在 vars
中的变量实际上是可寻址的情况下,这才会起作用。如果它们是位字段,则不会是这种情况,但即使那样,您也可以使用上面的一些缩短方法来最小化代码长度,从而使其更易于维护:
#include <ctype.h>
void decode (unsigned char* msg) {
// Check content, allowing for upper/lower case.
msg[0] = toupper (msg[0]);
msg[1] = toupper (msg[1]);
if ((msg[0] != 'D') && (msg[1] != 'O')) return;
if ((msg[2] < '1') || (msg[2] > '5')) return;
if (msg[3] != '=') return;
if ((msg[4] != '0') && (msg[4] != '1')) return;
// All content validated, make change, either turning off.
if (msg[4] == '0') {
switch (msg[2]) {
case '1': LATGbits.LATG1 = 0; break;
case '2': LATGbits.LATG0 = 1; break;
case '3': LATGbits.LATG13 = 1; break;
case '4': LATGbits.LATG14 = 1; break;
case '5': LATGbits.LATG12 = 1; break;
}
return;
}
// Or turning on.
switch (msg[2]) {
case '1': LATGbits.LATG1 = 1; break;
case '2': LATGbits.LATG0 = 0; break;
case '3': LATGbits.LATG13 = 0; break;
case '4': LATGbits.LATG14 = 0; break;
case '5': LATGbits.LATG12 = 0; break;
default: return;
}
sendString("OK");
}
这仍然很短,而且只有一点不太容易扩展,如果您想添加 LED。但是,如果您必须更新位字段,它仍然结构良好且易于理解。
关于c - 如何用sprintf简化语句?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/32323535/