No sé si es por el foro, pero tienes que arreglar mucho el interlineado. Tienes que dejar más espacios y separaciones.
Los nombres de las variables SON MUY CONFUSOS. Me ha costado mucho entender para que servía cada cosa, tienes que usar nombres concretos, claros y evitar ambiguedades (estacion y estaci???).
Sé puede tener la mesa desordenada, se puede tener el armario patas arriba... Pero en la programación no se puede ser desordenado, de ese modo te costará mucho más tiempo identificar los errores... Y no digamos una persona externa T_T.
Luego también se podrían añadir anotaciones y demás... Aunque eso ya es más segundario.
En cuanto al error, se encuentra en esta función:
void recotren(TTrenes &Tren,int x,int nest,int pos){
int est;
cout<<"Intoduce las estaciones"<<endl;
f[b]or(int p=0;p<nest;p++)[/b]{
cin>>est;
if(est>x){
while(est>x){
cout<<"Error,numero no permitido"<<endl;
cin>>est;
}
}
[b]Tren.paratren[p]=est;[/b]
}
almacenaestaciones(Tren,nest,x,pos);
Tren.ntrenes++;
}
Cada vez que se usa la funcion, P se resetea y siempre se le da los mismos valores (entre 0 y nest).
La solución estaría en guardar cada tren en una variable diferente... O mejor aún, hacer un array de trenes:
TTrenes Tren[100];
Y luego le pasas a la función el numero de tren que estas editando:
int contador = 0;
do{
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
recotren(Tren[contador],x,nest,pos);
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
contador++;
}while(opc!='n');
Esas líneas irían en sustitución de:
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
while(opc!='n'){
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
recotren(Tren,x,nest,pos);
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
}
Y con eso iría... Ah. Te hago una lista con algunas cosas que se podrían corregir en tu programa:
if((x<2)||(x>MAXT)){
cout<<"Error, numero no permitido,introduzca un numero valido"<<endl;
cout<<"Numero total de estaciones? "<<endl;
cin>>x;
}
Aquí tendría que ir un while en vez de un if (supongo, que querras que se repita hasta que le des una coordenada buena).
inicialiazarestaciones(Tren,x);
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
recotren(Tren,x,nest,pos);
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
while(opc!='n'){
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
[u][b]recotren(Tren,x,nest,pos,fin);[/b][/u]
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
}
Esa función no tiene 5 parametros sino 4. Supongo que te confundirias.
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
recotren(Tren,x,nest,pos);
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
while(opc!='n'){
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
recotren(Tren,x,nest,pos);
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
}
Lo que entiendo aquí, es que quieres introducir un estaciones hasta que el usuario diga no... No tienes que poner el codigo dos veces, basta con que uses un do while:
do
{
cout<<"Introduce el numero de estaciones del recorrido del tren "<<endl;
cin>>nest;
recotren(Tren,x,nest,pos);
cout<<"Quiere introducir el recorrido de otro tren?"<<endl;
cin>>opc;
}while(opc!='n');
La condición no se comprueba hasta el final, menos código y más claro queda
void inicializartrenes(TTrenes &Tren){
Tren.ntrenes=0;
}
Haces una función solo para eso? T_T
Mejor metelo dentro de otra, como la de "InicializarEstaciones".
PD: No te ofendas, yo tampoco era mejor cuando empezé
![[carcajad]](/images/smilies/nuevos/risa_ani2.gif "carcajada")
