使用strtok_r函数失败

Question

请帮我修复此代码。我现在迷失了 strtok 功能。我收到关于“if (tokens[0] == "A")”行的消息“ISO C++ 禁止指针和整数之间的比较”

if (started && ended)
{


  char *p = inData;
  char *tokens[50];
  int i = 0;

  while (i < 50) {
    tokens[i] = strtok_r(p,",",&p);
    if (tokens[i] == NULL) {
      break;
    }
    i++;
  }


  if (tokens[0] == 'A'){
    pinMode(atoi(tokens[1]),OUTPUT);       
    analogWrite(atoi(tokens[1]),atoi(tokens[2]));
  }

  else if (tokens[0] == 'D')
  {
   if (atoi(tokens[2]) == 1)
   {
     pinMode(atoi(tokens[1]),OUTPUT);
     digitalWrite(atoi(tokens[1]),HIGH);
   }
   else if (atoi(tokens[2]) == 0)
   {
     pinMode (atoi(tokens[1]),OUTPUT);
     digitalWrite(atoi(tokens[1]),LOW);
   }       
 }



  started = false;
  ended = false;
  index = 0; 
}

Answer 1

在 if (tokens[0] == 'A') 行中，字符常量 'A' 只是 A - 65 或 0x41 的 ASCII 值，而 tokens[0]< /code> 是一个 char *。因此，您有“指针和整数之间的比较”。

你到底想做什么？要检查令牌的第一个字符是否是大写 A，请编写

if (tokens[0][0] == 'A')

要检查它是否是字符串“A”，请编写

if (strcmp(tokens[0], "A") == 0)

另外，您对 strtok_r 的使用是错误的。您正在使用输入缓冲区来存储上下文信息，因此会覆盖它。 （您真的需要使用可重入版本吗？标准 strtok 不需要上下文存储。）它应该看起来像

char *token;
char *context;

for (token = strtok(p, ",", &context);
    token;
    token = strtok(NULL, ",", &context)) {

  tokens[i++] = token;
  if (i >= 50) break;
}

此外，您最好使用 strsep > 代替 strtok 像这样

char *token;
while (token = strsep(&p, ",")) {
  tokens[i++] token;
  if (i >= 50) break;
}

Answer 2

好吧，那行（我在你的代码中找不到）确实是错误的。 tokens[0] 是 char*，'A' 是 char。如果要比较所有字符串，请使用strcmp。如果您想检查 tokens[0] 的第一个字符，请使用 tokens[0][0]。

Answer 3

您说错误发生在该行上，

if (tokens[0] == "A")

但您向我们展示的代码中没有这样的行。

我假设您实际上的意思如下：

if (tokens[0] == 'A')

上面不起作用的原因是 tokens[0] 是 char* 类型，并且您无法比较 < code>char* 到这样的 char 。如果要比较两个字符串是否相等，则需要进行字符串比较：

if (strcmp(tokens[0], "A") == 0)