ToLower std::vector<std:: string>

Это связано с вопросом:

Строковый массив для функции C++

Хотя сейчас все работает нормально, единственное, чего я не могу сделать, - это расширить пользовательский ввод, когда я получаю сообщение об ошибке:

функция

bool lookupTerm(const std::string& term, const std::vector<std::string>& possible_names) {

    transform(term.begin(), term.end(), term.begin(), ::tolower);
    for (const std::string &possible_name : possible_names)
    {
        if (possible_name.compare(term) == 0)
            return true;
    }
    return false;
}

параметры

const std::vector<std::string> possible_asterisk         = { "star" , 
                                                              "asterisk" , 
                                                              "tilde"};
string term = "SoMeWorD";

ошибка

 In file included from /usr/include/c++/7.2.0/algorithm:62:0,
                 from jdoodle.cpp:5:
/usr/include/c++/7.2.0/bits/stl_algo.h: In instantiation of '_OIter std::transform(_IIter, _IIter, _OIter, _UnaryOperation) [with _IIter = __gnu_cxx::__normal_iterator<const char*, std::__cxx11::basic_string<char> >; _OIter = __gnu_cxx::__normal_iterator<const char*, std::__cxx11::basic_string<char> >; _UnaryOperation = int (*)(int) throw ()]':
jdoodle.cpp:40:64:   required from here
/usr/include/c++/7.2.0/bits/stl_algo.h:4306:12: error: assignment of read-only location '__result.__gnu_cxx::__normal_iterator<const char*, std::__cxx11::basic_string<char> >::operator*()'
  *__result = __unary_op(*__first);

Я знаю, что преобразование должно получать строку. Как я могу на мгновение преобразовать std::vector в просто строку, чтобы я мог перевести это слово в нижний регистр?

2 ответа

Решение

Это потому что term является const ссылка. Сделайте копию перед преобразованием ее в нижний регистр:

bool lookupTerm(const std::string& term, const std::vector<std::string>& possible_names) {
    std::string lower(term);
    transform(lower.begin(), lower.end(), lower.begin(), ::tolower);
    for (const std::string &possible_name : possible_names)
    {
        if (possible_name.compare(lower) == 0)
            return true;
    }
    return false;
}

Вы также можете достичь того же эффекта, удалив constи принимая параметр по значению:

bool lookupTerm(std::string term, const std::vector<std::string>& possible_names) {

std::transform должен иметь возможность изменять содержимое любого третьего аргумента, на который ссылается.

Это не работает в вашем случае, так как term это const объект.

Вы можете создать функцию локального объекта для хранения преобразованной строки.

std::string lowercaseTerm(term);
transform(term.begin(), term.end(), lowercaseTerm.begin(), ::tolower);

а затем использовать lowercaseTerm в следующей строке.

  if (possible_name.compare(lowercaseTerm) == 0)
Другие вопросы по тегам