Как исправить алгоритм кодирования строки?

Question

[olkhovich]

Вроде простая задачка, но уже долго не могу додумать решение.
Условие:
Каждый символ кодируется своим обозначением и количеством повторений. Например, для строки «aaaaaabbbbbaccccddeeeee» результатом кодирования будет строка «a6b5a1c4d2e5»

Мой вариант

std::string codingString(std::string inputString) {
    std::string str;
    int count = 1;
    for (int i = 0; i < inputString.length(); i++) {
        if (inputString[i] == inputString[i + 1]) {
            count++;
            str += operator+(inputString[i], std::to_string(count));
        }
        else {
            count = 1;
        }
    }
    return str;
}

Не могу понять, как избавиться от лишнего вывода вне цикла.

Answer 1

[0xD34F]

Вы записываете результат при равенстве соседних символов, а надо - наоборот.

std::string encodeString(std::string str) {
  std::string result = "";
  int count = 0;
  int len = str.length();

  for (int i = 0; i < len; i++) {
    count++;

    if (i == len - 1 || str[i] != str[i + 1]) {
      result += str[i] + std::to_string(count);
      count = 0;
    }
  }

  return result;
}

Comments

[poniyur]

Корректно ли ваше решение?
Просто вы выходите за пределы строки в условии if на последнем кругу цикла.

[0xD34F]

poniyur, да, меньше надо ширяться js'ом. Что там бывает при обращении за границу? UB? На всякий случай поправил.

[poniyur]

0xD34F, я только недавно начал изучать с++, поэтому и спросил)
А вообще, все говорят, что поведение непредсказуемо. Может отработать, может не отработать, может крашнуться.

[Антон Жилин]

poniyur, именно чтобы не было UB, и добавлено условие слева в if

[poniyur]

Антон Жилин, то он уже отредактировал, там раньше решение другим было