Hola a todos

Tengo un problema que no logro resolver. El usuario debe colocar su nombre y apellido y el script tiene que tomar la primera letra del nombre y agregarle el apellido. Por ejemplo:
Juan Perez
..debe quedar:
JPerez

 #include <iostream>

#include <string>

using namespace std;

/*

Diseñar un programa que permita generar direcciones de correo electrónico. 

El programa recibe el apellido y nombre de un usuario de la facultad (apellido y nombre se asignan a una sola variable) 

y debe retornar la dirección de correo electrónico (e-mail) generada. El dominio asignado a la Facultad para el e-mail es: 

frsf.edu.ar, y el nombre de usuario se forma con la inicial del nombre y a continuación el apellido.

*/

int main(int argc, char *argv[]) {

	int contador=0;

	string nombre;

	string email;

	getline (cin,nombre);

	email[0]=nombre[0];

	for (int i=0;nombre[i]!=' ';i++){

		//busca la posicion del espacio para agregar el apellido al email

		contador=i;

	}

	//comenzamos a buscar el final de linea a partir de la 1ra letra luego del espacio

	for (int a=contador+1;nombre[a]!='\0';a++){

	email+= nombre[a];

}

cout<<email;

	return 0;

}

Lo que esta saliendo en vez de JPerez es 'espacio'Perez.
Porque no toma la J?
Con email[0]=nombre[0]; ya deberia tomarla dado que le asigna la primera letra de Nombre a la primera posicion, sin embargo no funciona.

Alguna idea?

Fijate que tiene nombre[1], nombre[2] y asi, a ver si el resto se esta cargando bien.
Por otro lado, el primer for tiene toda la pinta de que deberia ser un while

(03-03-2016 12:38)Adriano escribió:  Fijate que tiene nombre[1], nombre[2] y asi, a ver si el resto se esta cargando bien.
Por otro lado, el primer for tiene toda la pinta de que deberia ser un while

Si, puede ser reemplazado por un while pero en este caso cual seria la ventaja? La finalidad se cumple igual.. pregunto de onda para saber si me pueden bochar por haber metido un for en lugar de while
Puse un for para buscar las lineas hasta el final..para buscar en un arreglo o un string tengo entendido se usa el for porque vas incrementando por 1 el valor de sus posiciones en cada ciclo

Edit: Hice un cout de email [0] y definitivamente no se muestra la primera letra.

---

Edit 2:

Estoy mas perdido:
cout<<email[0]; muestra la primera letra
cout<<email; me muestra la primera letra como un espacio

 	string nombre;

	string email;

	getline (cin,nombre);

	email[0]=nombre[0];

	cout<<"Cout de la posicion 0 del string email:"<<endl;

	cout<<email[0];

	cout<<endl<<"Cout de todo el string email:"<<endl;

	cout<<email;

El problema es que contador queda con la posición antes del espacio.
por ejemplo:
JUAN PEREZ
Queda contador=3
Es decir que "a" le estas asignando 4, y queda en el espacio.

Lo segundo, es que verifiques la concatenacion, ya que esto: "email+= nombre[a];" me parece que esta mal, y te esta pisando todo (por eso no aparece la primer letra)

(03-03-2016 12:59)rod77 escribió:  Lo segundo, es que verifiques la concatenacion, ya que esto: "email+= nombre[a];" me parece que esta mal, y te esta pisando todo (por eso no aparece la primer letra)

Creo que viene por aca el tema...no se que verificar, la concatenacion tendria que conservar el valor inicial del string y agregarle lo especificado..
Tal como decis, esta pisando todo, como hago entonces para conservar el primer caracter del string?

Declara otro contador que empiece en 1 arriba del segundo for y hace:
email[nuevocontador] = nombre[a]
y antes de salir del for incrementa en uno el nuevocontador.
También acordate de incrementar en uno a tu variable a antes del segundo for para que no empiece en el espacio en blanco.

Lo que te estaba pasando es que al hacer email =+ nombre [] estabas pisando la letra que ya habias asignado por un espacio en blanco.

(03-03-2016 14:54)crackOverflow escribió:  Declara otro contador que empiece en 1 arriba del segundo for y hace:
email[nuevocontador] = nombre[a]
y antes de salir del for incrementa en uno el nuevocontador.
También acordate de incrementar en uno a tu variable a antes del segundo for para que no empiece en el espacio en blanco.

Lo que te estaba pasando es que al hacer email =+ nombre [] estabas pisando la letra que ya habias asignado por un espacio en blanco.

No entendi nada. Por que motivo hacer un nuevo contador?
Si me podrias poner un ejemplo estaria muy agradecido

El tema del for o el while es conceptual mas que nada. Un serrucho te sirve para clavar un clavo? si, pero probablemente no sea lo mejor
El for esta pensado para iterar una N cantidad de veces. El while hasta que X condicion no se cumpla. Un while con un contador adentro es algo perfectamente valido.

Y si queres ir un paso mas alla (tranqui, si estas recien arrancando dejalo para despues), mirate esto: http://www.tutorialspoint.com/c_standard...strtok.htm

Saludos!

La idea es que lo pienses y que veas que hay varias formas de hacerlo, ademas de buscar variantes.

YO crearia una variable mas que sea "apellido" y el 2do for lo grabe ahi.
y al final usar la funcion append.

algo asi:

 int main(int argc, char *argv[]) {

int contador=0;

string nombre;

string apellido;

string email;

getline (cin,nombre);

for (int i=0;nombre[i]!=' ';i++){

//busca la posicion del espacio para agregar el apellido al email

contador=i;

   }

    //comenzamos a buscar el final de linea a partir de la 1ra letra luego del espacio

for (int a=contador+1;nombre[a]!='\0';a++){

apellido+= nombre[a];

}

email.append(nombre[0]);

email.append(apellido);

cout<<email;

return

Igual fijate si hace lo que queres,ya que no se si funciona, y no se si podes usar dicha funcion. Talvez podes hacer algo tipo email = nombre[0]+apellido. Pero como no se en que estas desarrollando se hace dificil.