¿Por qué este bucle, usado para agregar caracteres adyacentes en un vector, produce una salida extraña?

Samuel Maskell ha explicado el problema con precisión. Solo estoy agregando una solución de muestra

int len ​​= holder.size ();
para (int i = 0; i <(len / 2); ++ i)
cout << (titular [2 * i] + titular [2 * i + 1]) << "";
if (len% 2! = 0)
cout << titular [len - 1];

Ahora las otras respuestas aquí abordan con precisión el problema. Has complicado demasiado tu programa con tus dos bucles, lo que ha creado una salida no deseada debido a algunos problemas de anidamiento.

Antes de escribir un programa, con frecuencia dibujaré un diagrama lógico para tener una idea de cómo fluirá el programa. Alternativamente, me preguntaré qué estoy tratando de hacer que el programa haga y aborde las múltiples formas en que esto se puede hacer. Entrar en algunos hábitos para ayudar a visualizar el camino lógico que toma el programa puede ayudarlo a llegar a la raíz de sus problemas con bastante facilidad.

Tiene un bucle anidado, que es un bucle dentro de otro bucle. Para cada paso del bucle externo, el bucle interno ejecutará holder.size () – 1. Considere

para (int i = 1; i <= 5; ++ i) {
para (j = 1; j <= 5; ++ j) {
cout << i << ',' << j << endl;
}
}
Esto imprimirá, 1,1 1,2 1,3 1,4 1,5 2,1 2,2 2,3 2,4 2,5 etc.

En su caso, no necesita un bucle anidado, sí lo hará un solo bucle. Utilizar

para (decltype (holder.size ()) i1 = 0; i1 <= holder.size () - 2; i1 + = 2) {
holder.size ()) i2 = i1 + 1;
…
}