Question

远未完成，但现在我试图让这个程序询问文件名并将其存储在字符串中然后转换为ifstream，然后通过调用单独的函数isValid来检查文件是否有效，如果检查它将返回true如果没有，则有效，如果有效，则主函数将退出＆＃34;文件有效＆＃34;。然后它将继续重复此操作，直到输入退出。但它每次都会返回false，我不知道出了什么问题。我会很乐意为你提供任何帮助。

# include <iostream>
#include <string>
#include<fstream>
using namespace std;

bool isValid(ifstream& file)
{
    if (file.good())
    {

        return true;
    }
    else
    {
        return false;
    }
}



int main()
{
    string file_name;
    cout <<"please enter a HTML file name or hit 'exit' to quit and if you want to clear file please enter 'clear': ";
    cin >> file_name;
    ifstream my_file(file_name.c_str());



    while (file_name != "exit")
    {
        if ((isValid(my_file)) == true)
        {
            cout << "Hello" << endl;
        }
        string file_name;
        cout <<"please enter a HTML file name or hit 'exit' to quit and if you want to clear file please enter 'clear': ";
        cin >> file_name;
        ifstream my_file(file_name.c_str());
    }
}

Answer 1

您遇到了一个名为“阴影”的问题。

int main() {
    int i = 0;
    while (i == 0) {
        int i = 1;  // declares a new variable that
        // "shadows" (obscures) the outer i inside this scope
    }  // shadow i goes away, original i returns
}

以上代码将永远运行，因为while循环上下文中的i是在main中声明的i。

您的代码执行此操作：

int main()
{
    // ...
    ifstream my_file(file_name.c_str());

    while (file_name != "exit")
    {
        if ((isValid(my_file)) == true)  // << outer my_file
        // ...
        ifstream my_file(file_name.c_str()); // << scope-local variable
    }  // << scope local my_file goes away
}

您可能需要考虑重构代码以避免重复并简化它：

#include <iostream>
#include <fstream>
#include <string>

int main() {
    for (;;) {  // infinite loop
        std::string file_name;
        std::cout <<"please enter a HTML file name or hit 'exit' to quit and if you want to clear file please enter 'clear': " << std::flush;
        if (!std::getline(std::cin, file_name))
            break;
        if (file_name == "exit")
            break;

        std::ifstream my_file(file_name);
        if (!my_file.is_open()) {
            std::cerr << "unable to open file " << file_name << '\n';
            continue;
        }

        std::cout << "hello\n";
    }
}

我把它作为练习留给你重新介绍你的isValid函数。

在函数中验证文件并在main中调用它并输出＆＃34;文件是有效的＆＃34;如果它是有效的

1 个答案: