C 中的 gets() 问题

Question

我编写了以下代码：

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

#define SIZE 128

int main ()

{
    char mychar , string [SIZE];
    int i;
    int const count =0 ;    

    printf ("Please enter your string: \n\n");
    fgets (string, SIZE, stdin);

    printf ("Please enter char to find: ");
    mychar = getchar();

    for (i=0 ; (string[i] == '\0') ; i++ )
        if ( string[i]  == mychar )
            count++;

    printf ("The char %c appears %d times" ,mychar ,count);

    return 0;
}

问题是 gcc 给了我一个关于 'int const count' 的错误：“只读变量 'count' 的增量”。

好像出了什么问题？

感谢 ！

Answer 1

尝试使用 fgets 代替：

fgets (string, SIZE, stdin);

为什么 gets不安全，已在 SO 上多次回答。您可以看到这个。

Answer 2

始终使用 fgets() 而不是 gets。还有很多东西需要修复。您不应该使用标准库函数来创建用户界面。标准库确实不是为此设计的。相反，您应该使用 curses 库 或类似的东西。您还可以编写程序来接受参数作为输入。

正确使用标准库的简短示例。该版本没有任何错误检查，因此假设用户输入是正确的。

#include <stdio.h>

int main(int artc, char *argv[])
{
    /* arguments are strings so assign only the first characte of the
     * third argument string. Remember that the first argument ( argv[0] ) 
     * is the name of the program. 
     */
    char  mychar = argv[2][0];
    char *string = argv[1];
    int i, count = 0;

    /* count the occurences of the given character */
    for(; *string != '\0'; ++string)
        if(*string == mychar) ++count;

    printf("The char ‘%c’ appears %d times.\n", mychar, count);

    return 0;
}

用法： ./count "Hello, World!" l

输出： 字符“l”出现3次。

---

编辑：至于原始代码。将 == 更改为 !=。

for (i=0 ; (string[i] == '\0') ; i++ )

to:

for (i=0 ; (string[i] != '\0') ; i++ )

比较是错误的。

Answer 3

要使此示例正常工作，您还应该将以下行：

if(*string == mychar) ++count;

更改为

if(string[i] == mychar) ++count;< /code>

完整的工作示例现在是：

#include <stdio.h>

int main(int artc, char *argv[])
{
/* arguments are strings so assign only the first characte of the
 * third argument string. Remember that the first argument ( argv[0] ) 
 * is the name of the program. 
 */
char  mychar = argv[2][0];
char *string = argv[1];
int i, count = 0;

/* count the occurences of the given character */
for (i=0 ; (string[i] != '\0') ; i++ )
    if(string[i] == mychar) ++count;

printf("The char ‘%c’ appears %d times in the sentence: %s\n", mychar, count, string);

return 0;
}

Answer 4

考虑替换为“scanf(“%s”, &string)”。

Answer 5

gets 是危险的，因为它允许您读入的数据多于分配的空间，您可以使用 fgets 来指定要读入的字符数，并在发现换行符时停止。

Answer 6

gets 是危险的，因为它可以获取比变量大小更多的数据。从而使系统遭受攻击并损害安全性。
应该使用 fgets，因为它限制了数量。要读取的字符数。

Answer 7

这会做：

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

#define SIZE 128

int main()
{
  char mychar, string[SIZE];
  int i;
  int count=0;    

  printf("Please enter your string: ");
  fgets(string, SIZE, stdin);

  printf("Please enter char to find: ");
  mychar = getchar();

  for (i = 0; (string[i] != '\0'); i++)
    if (string[i] == mychar) ++count;

  printf("The char %c appears %d times in the sentence: %s" ,mychar ,count, string);

  return 0;
}