我认为的strtok问题

Question

想不通，有什么帮助吗？我不认为这是 strtok，我很确定这是我的代码。我想不通是怎么想的。 Get 和 Set 导致 sigsevg。如果我在 num = strtof 等之后放置 printf()，则 num 是正确的，但其他命令的解释不正确。

 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

typedef struct
{
    float height;
    float width;
    float length;
}Box;

void usage(void)
{

    printf("\nUsage: [command] [parameter] [amount]\n");
    printf("commands:\n\tSet\n\tGet\n");
    printf("parameters:\n\theight\n\twidth\n\tlength\n");
}


int main()
{
    usage();
    Box box1 = {0,0,0};
    int loop = 1;
    float num;
    char cp[65], *delims =  " !@#$%^&*():;/><.,\\?\"";
    char *tok1, *tok2, *tok3, *temp;

beginning:
    while(loop)
    {

        //Read the command from standard input
        char str[65];
        fgets(str, 64, stdin);
        str[64] = 0;

        //Tokenize the string
        strncpy(cp, str, 64);
        tok1 = strtok(cp, delims);
        tok2 = strtok(NULL, delims);
        tok3 = strtok(NULL, delims);

        //Check if tok3 is float
        num = strtof(tok3, &temp);
        if(num != 0)
        {

        }
        else
        {
            usage();
            goto beginning;
        }
        if(tok1 == 'Get' && tok2 == 'height')
        {
            printf("%f", box1.height);
        }
        else if(tok1 == 'Get' && tok2 == 'width')
        {
          printf("%f", box1.width);
        }
        else if(tok1 == 'Get' && tok2 == 'length')
        {
          printf("%f", box1.length);
        }
        else if(tok1 == 'Get')
        {
           usage();
           goto beginning;
        }

        if(tok1 == 'Set' && tok2 == 'height')
        {
          box1.height = num;
          printf("%f", box1.height);
        }
        else if(tok1 == 'Set' && tok2 == 'width')
        {
          box1.width = num;
        }
        else if(tok1 == 'Set' && tok2 == 'length')
        {
          box1.length = num;
        }
        else if(tok1 == 'Set')
       {
         usage();
         goto beginning;
       }

    }
     return 0;
}

Answer 1

if(tok1 == 'Get' && tok2 == 'height')

C 字符串必须使用双引号，你不能用== 测试它们是否相等，你应该使用strcmp：

if(strcmp(tok1, "Get")==0 && strcmp(tok2, "height")==0)

关于strtof：

num = strtof(tok3, &temp);

如果您不必使用temp，请使用空指针：

num = strtof(tok3, NULL);

还有一段代码使用goto:

if(num != 0)
{

}
else
{
    usage();
    goto beginning;
}

goto 很难看，请改用continue：

if(num == 0)
{
    usage();
    continue;
}