将用户输入传递给fopen（）参数c

Question

我试图让用户输入文件的路径。然后我将这个文件用fopen（）打开。这里是我正在尝试的一个例子：

char filename[80] = " "; 
printf("enter filenme:"); 
scanf("%s",&filename); 
FILE *fp=fopen(filename,"r"); 

我不断收到段错误：11作为终端错误。

Answer 1

您正在传递文件名开头地址的地址。不要使用&。它是一个数组，所以它已经是一个指向存储地址的指针。

Answer 2

当然，你会得到一个错误：你是路过的地址指针到scanf而不是指针本身：

scanf("%s",&filename); 

将其更改为

scanf("%s", filename); 

或

scanf("%s",&(filename[0])); 

你的问题应该消失了。

Answer 3

既然你标记这个C++，你应该考虑使用std::getline读取文件名：

string filename; 
std::cout << "Enter filename: "; 
std::getline(std::cin, filename); 
std::ifstream input_file(filename, std::ifstream::in); 
// ... read from input_file .... 

如果你要读的文件名到一个固定大小的缓冲区，请确保您指定的那个长度阅读时缓冲区;否则，您的程序中存在缓冲区溢出漏洞。

Answer 4

尽管传递给scanf()的参数不太正确，但我不认为这实际上是问题所在：数组地址和数组第一个字符的地址是相同的。更有可能fopen()失败，并且您从fopen()获得一个空指针，然后您可以访问它。也就是说，尝试这样的事情：

char filename[80] = {}; 
if (scanf("%79s", filename) != 1) { 
    printf("failed to read a filename\n"); 
} 
else if (FILE* fp = fopen(filename, "r")) { 
    printf("reading file '%s'\n", filename); 
} 
else { 
    printf("failed to open file '%s' for reading\n", filename); 
} 

（基于它似乎"%79s"的意见是限制读太多的字符到80个字符数组格式正确，但我不是一个C程序员，即我可能搞砸了正确的格式...）

等效采用C++会是这样这可能是可取：

std::string filename; 
if (!(std::cin >> filename)) { 
    std::cout << "failed to read a filename\n"; 
} 
else if (std::ifstream in{filename}) { 
    std::cout << "reading file '" << filename << "'\n"; 
} 
else { 
    std::cout << "failed to open file '" << filename << "' for reading\n"; 
}