无法创建反转字符串的程序

Question

我正在使用Linux。 我正在尝试用 c 编写一个程序，该程序将向后打印字符串。 这是我的代码：

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
int main (){
    char string[100];
    printf ("Enter string:\n");
    gets (string);
    int length = strlen (string)-1;
    for (length = length; length>=0; length--){
        puts (string[length]);
    }
}

这是错误：

a.c:10: warning: passing argument 1 of ‘puts’ makes pointer from integer without a cast
/usr/include/stdio.h:668: note: expected ‘const char *’ but argument is of type ‘char’
/tmp/cc5rpeG7.o: In function `main':
a.c:(.text+0x29): warning: the `gets' function is dangerous and should not be used.

我应该做什么？

Answer 1

忘记函数 gets() 的存在——它是致命的。请改用 fgets() （但请注意，它不会删除行尾的换行符）。

您想一次输入一个字符：使用 putchar() 将其写入标准输出。不要忘记在循环后的输出中添加换行符。

另外，for (length = length; length >= 0; length--) 不是惯用的 C. 使用以下之一：

• for (; length >= 0; length--)
• for (length = strlen(string) - 1;
• length >= 0; length--) for (int length = strlen(string) - 1; length >= 0; length--) = 0; length--)

最后一个替代方案使用了 C99 中添加的功能（很久以前就在 C++ 中可用）。

此外，我们还可以讨论 length 是否是变量的适当名称。最好将其重命名为 i 或 pos 或类似的名称，因为虽然它被初始化为输入的长度，但它实际上用作数组索引，而不是用作任何东西的长度。

主观：不要在函数名称与其参数列表之间放置空格。 C 语言的创始人不会这样做——你也不应该这样做。

---

为什么 gets() 是致命的？

第一个互联网蠕虫 - 1988 年的 Morris 蠕虫 - 利用了 fingerd 使用 gets() 而不是 fgets() 的程序。从那时起，许多程序都因为使用 gets() 而不是 fgets() 或其他替代方案而崩溃。

根本问题是 gets() 不知道有多少空间可用于存储它读取的数据。这会导致“缓冲区溢出”，您可以在您最喜欢的搜索引擎中搜索该术语，该术语将返回大量条目。

如果有人在示例程序中输入了 150 个字符，那么 gets() 将在长度为 100 的数组中存储 150 个字符。这永远不会带来幸福 - 它通常会导致核心转储，但是通过精心选择的输入（通常由 Perl 或 Python 脚本生成），您可能可以让程序执行任意其他代码。如果程序将由具有“提升权限”的用户运行，这一点确实很重要。

顺便说一句，gets() 可能会在下一个版本中从标准 C 库中删除（C1x - 请参阅 WG14）。它不会在很长一段时间内（20 年？）从实际的 C 库中消失，但应该用此实现（或类似的东西）替换：

#undef NDEBUG
#include <assert.h>
char *gets(char *buffer)
{
    assert("Probability of using gets() safely" == 0);
}

---

另一个小细节，在评论中部分讨论主要问题。

显示的代码显然是针对 C99 的；函数中 length 的声明在 C89 中无效。鉴于此，main() 函数不显式返回值是“可以”的，因为 C99 标准遵循 C++ 标准的引导，并允许您省略 的返回main() 效果与最后的 return(0); 或 return 0; 相同。

因此，严格来说，这个问题中的程序不能因为最后没有返回而被归咎于错误。然而，我认为这是更特殊的标准化决定之一，并且如果标准将这一规定排除在外，我会更喜欢它 - 或者做一些更激进的事情，例如允许普遍存在但错误的 void main()观察到当控制从那里返回时，结果是成功状态返回到环境。不幸的是，不值得为改变标准的这一方面而奋斗 - 但作为个人风格的决定，我不会利用授予的许可来省略来自 的最终 返回 main()。如果代码必须与 C89 编译器一起使用，则它应该在末尾显式返回 0;（但随后也必须修复 length 的声明）。

Answer 2

您还可以使用递归来完成此操作。我认为使用循环时看起来更好。

只需使用您的字符串调用该方法，并在打印方法中的字符之前，使用相同的字符串再次调用该方法，减去第一个字符。

这将以相反的顺序打印出您的字符串。

Answer 3

第一：

NEVER NEVER NEVER NEVER NEVER使用gets()；它会在您的代码中引入故障点。无法告诉 gets() 目标缓冲区有多大，因此，如果您传递一个大小为容纳 10 个字符的缓冲区，并且输入流中有 100 个字符，gets() code> 会很乐意将这额外的 90 个字符存储在缓冲区末尾之外的内存中，这可能会破坏一些重要的内容。缓冲区溢出是一种简单的恶意软件利用； Morris 蠕虫专门利用了 sendmail 中的 gets() 调用。

使用 fgets() 代替；它允许您指定从输入流读取的最大字符数。但是，与 gets() 不同，fgets() 会将终止换行符保存到缓冲区（如果有空间），因此您必须考虑到这一点：

char string[100]; 
char *newline;
printf("Enter a string: ");
fflush(stdout);
fgets(string, sizeof string, stdin);
newline = strchr(buffer, '\n');      // search for the newline character
if (newline)                         // if it's present
  *newline = 0;                      // set it to zero

现在就这样了顺便说一句...

您的错误来自于 puts() 需要一个 char * 类型的参数，但您传递的是 类型的参数>char，因此出现“来自整数的指针，无需强制转换”消息（char 是整数类型）。要将单个字符写入 stdout，请使用 putchar() 或 fputc()。

Answer 4

您应该使用 putchar 而不是 puts

所以这个循环：

for (length = length; length>=0; length--){
    puts (string[length]);
}

将是：

for (length = length; length>=0; length--){
    putchar (string[length]);
}

putchar 将采用单个字符作为参数并将其打印到 stdout，这就是你想要的。另一方面，puts 会将整个字符串打印到 stdout。因此，当您将单个字符传递给需要整个字符串（字符数组、NULL 终止字符串）的函数时，编译器会感到困惑。

Answer 5

使用 putc 或 putchar，因为 puts 被指定为获取 char* 并且您正在向其提供 >字符。