将字符串传递给函数

Question

我正在尝试将字符串传递给 C 中的函数。该字符串将由用户输入，然后传递给函数以写入文本文件。我知道这看起来很基础，但我只是在学习 C。

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

void read() {

        char text[50][30],buffer[150];

        int i=0;
        FILE *file_in;
        file_in=fopen("test.txt","r");

        if (file_in == NULL) {
                printf("Error opening file\n");
        }

        while (fgets(buffer,150,file_in)) {
                strcpy(text[i],buffer);
                printf("line %d: %s\n",i,text[i]);
                i++;
        }

        getchar();

        fclose(file_in);
}

void write(char str[])
{
        FILE *file_in;
        file_in=fopen("test.txt","a");

        if (file_in == NULL) {
                printf("Error opening file\n");
        }

        //write to the file
        fprintf(file_in, "\n%s", str);
//      fputs(str, file_in);

        fclose(file_in);
}

int main()
{
        read();

        char msg[50];

        printf("Enter some text: ");
        puts(msg);

        write(msg); 

        return 0;
}

它写入文件，但写入的是奇怪的字符，而不是我实际键入的字符。我做错了什么？

Answer 1

您似乎将gets 与puts 混淆了。 puts 将字符串写入控制台。 gets 从控制台读取一个字符串。把它们关掉，你的程序应该可以工作了。

Microsoft 的编译器经常针对不安全或已弃用的函数发出警告，例如 gets。您可以改用fgets，因为它不允许缓冲区溢出。

这是一个例子：

fgets(msg, 50, stdin);

或

fgets(msg, sizeof(msg), stdin);

Answer 2

首先：不要调用你的函数read() 和write()——选择更具体的。函数名read() 和write() 已经被系统用于低级文件操作，尝试自己定义它们会导致意外行为。

第二：你永远不会初始化msg 变量的内容或读取任何数据，所以它的内容是随机的。 （请记住，puts() 打印数据；它不读取任何内容。）

Answer 3

当您编写char msg[50]; 时，它包含不确定的值。它不是零初始化或任何东西。

puts(msg); 行将该垃圾写出，然后write(msg); 将该垃圾写入文件。

我猜您打算在printf 之后和puts 之前添加一些用于输入文本的代码。

注意。在您的 read() 函数（您还没有调用）中，您应该使 fgets 缓冲区大小与您的数组的宽度相匹配，并且您应该检查如果文件没有跑出数组的末尾有很多行。

另外，最好将您的函数命名为 read 和 write 以外的其他名称，因为在 POSIX 环境中已经存在使用该名称的函数可能会发生冲突。

Answer 4

解决办法如下：

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

int read() {

        char text[50][30],buffer[150];

        int i=0;
        FILE *file_in;
        file_in=fopen("test.txt","r");

        if (file_in == NULL) {
                printf("Error opening file\n");
        }

        while (fgets(buffer,150,file_in)) {
                strcpy(text[i],buffer);
                printf("line %d: %s\n",i,text[i]);
                i++;
        }

       // getchar();why you were using this?

        fclose(file_in);
      //  return 0;
}

void write(char str[])
{
        FILE *file_in;
        file_in=fopen("test.txt","a");

        if (file_in == NULL) {
                printf("Error opening file\n");
        }

        //write to the file
        fprintf(file_in, "\n%s", str);
//      fputs(str, file_in);

        fclose(file_in);
}

int main()
{
        char msg[50];
        read();



        printf("Enter some text: ");
       // getchar();
        gets(msg);//It reads in msg 

        write(msg); 

        return 0;
}