反转用户输入的整数字符串 (C)

Question

我想要做的是反转用户输入的一串数字。发生的情况是它编译并运行，直到我在 scanf 之后按 Enter 键。然后我收到一些 Microsoft 运行时错误...出了什么问题？？？

注意：这是家庭作业，但我已经弄清楚了逻辑。令我困惑的是这个错误。

#include <stdio.h>
int main()
{
    unsigned int giveStr = 0;
    char* charIt;
    printf("Enter a number to be reversed.\t");
    scanf("%d", &giveStr);
    fflush(stdin);
    sprintf(charIt, "%d", giveStr);
    revStr(giveStr);
    getchar();
    return 0;
}

revStr(unsigned int n)
{
      char buffer[100];
      int uselessvar, counter = 0;
      for (; n > 0;)
      {
           uselessvar = sprintf(&buffer[counter], "%d", n);
           counter++;
      }
      for (counter = 0; counter > 0;)
      {
          printf("%c", buffer[counter]);
          counter--;
      }
      return 0;
}

编辑：刷新换行符的标准输入：/以及图像这里只是不那样程序。和我的。

Answer 1

您正在尝试访问未分配的内存：

 sprintf(charIt, "%d", giveStr);

将 char* charIt; 更改为 char charIt[50]; 并且一切都应该很好（嗯，至少分段错误部分）

另外...将 charIt 传递给 revStr，因为 charIt 包含带有我们号码的字符串。
然后，revStr 中的一个简单的 for 循环就可以解决问题（第二个循环的目的是什么？）

void revStr(char *giveStr)
{
  int counter;
  for (counter = strlen(giveStr)-1; counter >= 0; counter--)
  {
    printf("%c", giveStr[counter]);
  }
  printf("\n");
}

这将打印我们的字符表示从最后一个到第一个的每个字符。您应该阅读有关 for 循环的更多内容。

Answer 2

对于家庭作业问题，如果您有K&R 书，请转阅读第 3.5 节并仔细阅读。

请注意函数 reverse() 和 itoa()。他们应该为您提供如何解决问题的好主意。

Answer 3

你的程序如何跳出 for (; n > 0;) 循环？计数器不会简单地增加直到出现总线错误吗？

ED：

恕我直言，我认为“我已经弄清楚了逻辑”的说法有点乐观。 :^) 毫无疑问有人会按照应该做的方式发布
当我写完这篇文章时，但可能值得引起人们注意出了什么问题（除了其他地方提到的内存分配问题）：

您的第一个循环“for (; n > 0;)”很奇怪，因为您将整个数字 n 打印到计数器的缓冲区中。那么为什么您需要多次执行此操作呢？如果您选择单个数字，您可能会选择，但您不会，显然您知道如何执行此操作，因为您已经使用过“sprintf(charIt, “%d”, GiveStr);”。 [旁白：对于无符号整数变量来说，giveStr 并不是一个好名字！]

你的第二个循环也有奇怪的条件：你将 counter 设置为 0，将条件设置为 counter > > 。 0，然后减少里面的计数器。这显然不会按照您想要的方式循环字符。假设您认为第一个循环是逐个字符的，那么您可能会考虑从 counter-1 向下循环到 0？