Ошибка памяти при чтении аргументов командной строки в программе на C ++

Как уже указывали другие, sizeof не дает количество элементов в массиве; идиоматический способ сделать это - использовать std::end( article ) - std::begin( article ). В pre- C++ 11 большинство опытных программистов C++ сделали бы то же самое, используя версии begin и end своего инструментария.

Вся isArticle функция весьма unidiomatic. В 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.