Конвертировать строку в верхний регистр - C ++

Question

У меня проблемы с переводом строк в верхний регистр для сортировки по алфавиту. Моя программа на самом деле начинает тормозить и зависать после третьего набора слов. Что я делаю не так?

string iName = list[i]->GetLastName(); // This just returns a string of a name
string jName = list[j]->GetLastName();

for(unsigned int k = 0; k < iName.length(); k++)
  {
    iName[k] = toupper(iName[k]);
  }

for(unsigned int l = 0; l < jName.length(); l++)
  {
    iName[l] = toupper(jName[l]);
  }

Answer 1

Использовать библиотеку STL algorithm:

std::for_each(iName.begin(), iName.end(), std::toupper);

или (предложено @ Kerrek SB )

std::transform(s.begin(), s.end(), s.begin(), std::toupper);

Answer 2

как говорили другие, вы смешали iname и jname .... и почему вы это сделали?

Потому что вы скопировали вставленный!

Итак, хороший ранний урок программирования - попытаться избежать копирования! вместо этого попробуйте создать функции .....

в вашем случае ...

void stringToUpper(string &s)
{
   for(unsigned int l = 0; l < s.length(); l++)
  {
    s[l] = toupper(s[l]);
  }
}

тогда можно сделать

stringToUpper(iName);
stringToUpper(jName);

Этот подход сокращает ОЧЕНЬ много ошибок, связанных с копированием, и в целом помогает сделать ваши программы намного более модульными

Answer 3

Стандарт предполагает, что вы не должны использовать оператор [] в строке для изменения ее содержимого. Строка задана как неизменный объект, и ваш код нарушает этот стандарт. Поэтому я думаю, что ответ заключается в том, что трудно сказать, что может происходить, потому что вы используете класс так, как он не был разработан. Это может быть испортить какой-то внутренний индекс или что-то еще. Мы не знаем, что может делать класс строки внутри.

Тем не менее, похоже, что это должно работать:)