Question

我一直在为学校编写一个C ++代码项目，希望我从命令行参数中读取文本文件并输出它，并在任何文章[a，an，the]之后添加在命令行中声明的形容词。这是我第一次对任何项目使用命令行。

我已经设法从命令行获取并读取文本文件但我的问题是当我想获取一个字符串作为函数isArticle（）的参数时，我在UNIX shell中不断收到以下消息：

./test-program1[38]: eval: line 1: 7704: Memory fault
0a1,4

我不怀疑问题出在isArticle()函数上，但这里是：

bool isArticle(string n)
{
    string article[]={"a","an","the","A","An","aN","AN","The","tHe","thE","THe","tHE","ThE","THE"};

    for(int i=0;i<sizeof(article);i++)
    {
        if(n.compare(article[i])==0)
            return true;
    }

    return false;
}

虽然它还没有完成，但是我用一些测试代码来查看isArticle（）函数是否正常工作：

int main(int argc, char *argv[])
{ 
    istream *br;
    ifstream file;

    if(argc == 1)
        br = &cin;
    else if(argc==3)
    {
        file.open(argv[2]);
        if(file.is_open())//put work here
        {
            br=&file;
            string word;
            string output[sizeof(file)];
            while(file>>word)
            {
                if(isArticle(word)==true)
                {
                    cout<<word;
                }
            }
        }
        else
        {
            usage(argv[2],"Cannot open "+string(argv[2]));
            return 1;
        }
    }
    else
    {
        usage(argv[1], "More than one filename was given");
        return 1;
    }
    return 0;
}

Answer 1

正如其他人已经指出的那样，sizeof 不会给出这个数字数组中的元素;这样做的惯用方法是使用 std::end( article ) - std::begin( article )。在预C ++ 11中，大多数经验丰富的C ++程序员也会这样做，使用的是来自他们工具包的begin和end。

整个isArticle功能非常单一。在C ++ 11中，整个函数将是一个单独的库调用：

bool
isArticle( std::string const& word )
{
    static std::string const articles[] = {
        "a", "an", "the", "A", "An", "aN", "AN",
        "The", "tHe", "thE", "THe", "tHE", "ThE", "THE",
    };
    return std::any_of(
            std::begin( articles ), std::end( articles ),
            [=]( std::string const& target ) { return word == target; } );
}

在早期的C ++中，我们写过：

return std::find( begin( articles ), end( articles ), word )
        != end( articles );

（来自我们常用工具包的begin和end）。如果我们想要（为教学原因）自己写循环，这将是一些东西像：

std::string::const_iterator current = std::begin( articles );
std::string::const_iterator end = std::end( articles );
while ( current != end && *current != word ) {
    ++ current;
}
return current != end;

可能与您的直接问题无关的几点值得一提：

类类型通常通过引用传递给const，而不是通过引用传递值。可以说，这是过早的优化，但实际上是这样无处不在，其他任何东西让人不知道为什么。
应声明函数中未更改的值const 和 static。
std::string支持==和!=;如果你正在寻找平等，那是你应该使用的。 compare函数应该是真的仅用于词典排序。
从循环中间返回是您通常想要的东西避免。当然，当功能如此简单时，它实际上并非如此但是，这是一个坏习惯。

这只涉及有问题的功能。在main中，您也是遇到sizeof的问题。在这种情况下，它看起来像你试图用它来确定文件中的单词数。那没有实际读取文件就无法完成。你需要的是什么 std::vector<std::string>，而不是C样式数组（其大小必须是在编译时已知。）

当然：if所需的类型是bool。 isArticle 返回bool，因此不需要任何其他内容。写作 isArtile( word ) == true强烈暗示你不知道是什么 bool类型是。（提示：表达式的类型 isArtile( word ) == true也是bool。）

最后一个建议是：如果程序没有参数，那么你什么都不做我不认为是意图。通常 Unix下命令行进程的解决方案（它也很普遍在Windows下）是将所有实际工作放在一个函数中，然后编写类似的东西：

int
main( int argc, char** argv )
{
    if ( argc == 1 ) {
        process( std::cin );
    } else {
        for ( int i = 1; i != argc; ++ i ) {
            std::ifstream in( argv[i] );
            if ( ! in ) {
                //  Output error message and set global flag for
                //  return value...
            } else {
                process( in );
            }
        }
    }
    return globalFlagWithReturnValue;
}

函数process以std::istream&为参数允许它读取std::cin或已打开的std::istream。

Answer 2

sizeof运算符不返回数组中的元素数。它返回数组占用的内存空间。 char数组的sizeof数组与int数组不同。

对于您的数组，您可以通过以下方式确定编译时的元素数量：

const unsigned int elements_in_array =
    sizeof(article) / sizeof(article[0]);

在C ++程序中读取命令行参数时出现内存错误

2 个答案: