参数检查

Question

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

int main (int argc, char* argv[]) 
{ 
    string STRING; 
    ifstream infile;  
    STRING = argv[1]; 
    infile.open(argv[1]); 
    if (infile.fail())// covers a miss spelling of a fail name 
    { 
     cout << "ERROR. Did you make a mistake in the Spelling of the File\n"; 
     return 1; 
    } 
    else 
    { 
     while(!infile.eof()) 
     { 
      getline(infile,STRING); // Get the line 
      cout<<STRING + "\n"; // Prints out File line 
     } 
     infile.close(); 
     return 0; 
    } 
} 

如果用户只运行没有文件名的程序（我认为是被称为参数）如./displayfile然后我得到我有这个计划天晴，一个问题

工作分段故障

我将如何修改我的代码，使程序将与沿行的错误消息退出“添加文件名”

我首先想到的是沿

线的东西

if (!argc=2) 
{ 
    cout << "ERROR. Enter a file name"; 
    return 1; 
} 

新增： 万一这个问题我使用编译 G ++ displayfile.cpp -o displayfile

Answer 1

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

int main (int argc, char* argv[]) { 
    if(argc != 2) { 
     cout << "You need to supply one argument to this program."; 
     return -1; 
    } 

    string STRING; 
    ifstream infile;  
    STRING = argv[1]; 
    infile.open(argv[1]); 
    if (infile.fail())// covers a miss spelling of a fail name { 
     cout << "ERROR. Did you make a mistake in the Spelling of the File\n"; 
     return 1; 
    } 
    else { 
     while(!infile.eof()) { 
     getline(infile,STRING); // Get the line 
     cout<<STRING + "\n"; // Prints out File line 
     } 
     infile.close(); 
     return 0; 
    } 
} 

Answer 2

变化STRING = argv[1];到if (argv[1] != null) STRING = argv[1];不知道的，所以你得先测试一下。

Answer 3

除了为argc != 2明显的支票我不由固定的一些糟糕的代码和明显的错误：

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

int main (int argc, char* argv[]) 
{ 
    if (argc != 2) 
    { 
     cout << "ERROR. Invalid number of arguments\n"; 
     return 1; 
    } 

    ifstream infile(argv[1]); 
    if (!infile)  // covers a miss spelling of a fail name 
    { 
     cout << "ERROR. Did you make a mistake in the Spelling of the File\n"; 
     return 1; 
    } 

    string STRING; 
    while(getline(infile, STRING)) 
     cout << STRING << '\n';  // Prints out file line 
    return 0; 
} 

你并不需要调用ifstream::open，只是使用构造函数，同样你也不需要这么早声明STRING，也不需要将它初始化为文件名，因为你不使用它。不要忘记，这不是C，你不需要在每个函数的开头都有一大堆声明。

其次，检查流的标志通常是一个坏主意，只需检查!infile以查找任何错误。但真正的错误是在while条件中检查infile.eof，因为它只会在getline试图读取文件末尾时才设置，所以您实际上会打印一个（可能是空的）行太多。只需检查getline的返回值即可找到任何错误或文件结尾。

输出时不要将换行符添加到字符串上，只是在字符串之后放出。最后但并非最不重要的一点，不需要infile.close，因为无论如何析构函数都会调用它。