用指针算术打印字符串[关闭]

Question

我正在尝试使用 rand() 打印随机单词。 我认为我在指针算法上犯了一个错误，因为我得到了一个奇怪的输出。

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

    int main ()
    {
        srand(time(0));
        const int randomBit = rand() % 2;
        char * sz[] ={"good", "bad"};
    
        switch (randomBit)
        {
        case 0:
            for (char *i=*sz; *i<(sz+3); i++)
            {
                printf("%c", *i);
            }
            break;
    
        case 1:
            for (char *i=*(sz+3); *i!=0x0; i++)
            {
                printf("%c", *i);
            }
        break;
        
        default:
            break;
        }
    
        return 0;
    }

我的错误是什么？ 谢谢。

Answer 1

鉴于此声明...

        char * sz[] ={"good", "bad"};

...此代码具有未定义的行为：

            for (char *i=*(sz+3); *i!=0x0; i++)

表达式*(sz+3)等价于sz[3]，但数组sz只有两个元素，所以最大索引为1。

同样，表达式*sz 等价于sz[0]。那个在语义上还可以，但在风格上很差。同样，风格很差的是将两个不同的情况分开，而您可以使用 sz[randomBit] 选择要打印的字符串来使用相同的代码覆盖这两种情况。

此外，还不清楚您为什么要逐个字符地打印。也许这是任务的一部分，但在现实世界中，程序员可能会写......

    printf("%s", sz[randomBit]);

... 而不是整个 switch 语句。

Answer 2

您的代码非常复杂（而且是错误的）。

这是怎么回事：

int main()
{
  srand(time(0));
  const int randomBit = rand() % 2;
  char* sz[] = { "good", "bad" };

  printf("%s\n", sz[randomBit]);    
  return 0;
}

或者如果您不允许在作业中使用 %s 格式说明符：

  for (char* i = sz[randomBit]; *i != 0;  i++)
  {
    printf("%c", *i);  // or putchar(*i);
  }