gibi77 me puse a revisar tu código. Y tengo un par de sugerencias, y una para arreglar tu error de ejecución.
Las sugerencias:
La primera:
Cuando defines e implementas la función LeeValor(int) lo haces asi
char LeeValor(int)
//(LA IMPLEMENTACION)
char CLDL::LeeValor(int que){
char Valor[30];
printf("Escriba el valor a %s \n",que?"Eliminar":"Dar de alta");
scanf("%s",Valor);
return Valor[30];
}
¿Porque
char LeeValor(int)? deberia ser
char* LeerValor(int) al igual que en la implementación, ya que lo que supuse que tu querías devolver era una cadena de caracteres no un solo caracter.
Segunda sugerencia:
Siempre que uses scanf utiliza fflush para asegurarte que el buffer de entrada se limpie.
Tercera sugerencia:
return Valor[30];
Estarías devolviendo el caracter en la posición 30. Deberias hacerlo asi
return Valor;
Aunque igual este erroneo porque estas pasando un puntero de una variable que se va destruir en seguida termine. Sabiendo que utilizas C++ usa string de la STL para no complicarte con los punteros, aunque igual se puede con chars.
Cuarta sugerencia:
Nunca te olvides de inicializar todos los miembros privados de tus clases, aunque a veces de igual, otras es crucial. Asi que mejor siempre hacerlo a menos que el lenguaje te lo haga por ti pero este no es el caso, con los punteros, por ejemplo, si no le pones un constructor a NodoLDL (la unica función que se le puede poner a los constructores, si mal no recuerdo) para que SIG y ANT marquen NULL al igual que Ini y Act (que se que los pusiste como NULL o 0 que es lo mismo).Eso te sirve ademas para comprobar si hay siguiente, si hay anterior, ya que si no tienen NULL no se sabe.
Quita sugerencia:
Para la funcion AltaLDL que tambien le pusiste como argumento char debiendo ser char*
Nunca compies una cadena a sangre fria así. Jeje
NVO->Renglon[30]=Valor;
Si quisieras copiar un caracter eso esta bien, pero no es lo que supongo que querías ya que la estructura NodoLDL tiene a renglon como cadena no como caracter. Yo lo haria así. Dime si me equivoco.
strcpy(NVO->Renglon,Valor)
Sexta sugerencia:
En la misma función AltaLDL
NodoLDL*NVO= new NodoLDL;
if(!NVO)return;
NVO->Renglon[30]=Valor;
NVO->SIG=Act->SIG;
NVO->ANT=Act;
Act->SIG->ANT=NVO;
Act->SIG=NVO;
Act=NVO;
Salta el error en ejecucion ahi porque estas intentando modificar el contenido de una porcion de la memoria que no tiene nada asignado. Estoy hablando de Act cuando recien ejecutas el programa Act tiene NULL y no tiene una estructura NodoLDL asignada ni nada. Por eso es que cuando haces Act->SIG finaliza, porque no tiene la estructura en esa porcion de memoria para referirse ni a un SIG ni a un ANT. Y despues hay un par de errores como no fijarte justamente eso, si la lista esta vacía, si esta vacía tienes que proceder de otra forma para insertar un nodo. Bueno en fin te pongo como yo lo escribi.
void CLDL::AltaLDL(char Valor[30]) {
NodoLDL*NVO=new NoodoLDL;
///ACA TENDRIAS QUE EN REALIDAD MANDAR UN MENSAJE DE ERROR PERO COMO DIJISTE QUE NO LO TERMINASTE, NO HAY PROB
if(!NVO) return;
strcpy(NVO->Renglon,Valor);
//esto,nada para volver al primer nodo
if(Act) while(Act->ANT) Act = Act->ANT;
//Lo que te decia, fijate si existe el primer nodo
if(!Act) {
NVO->ANT = NULL;
NVO->SIG = Act;
Act = NVO;
} else {
NVO->SIG = Act->SIG;
Act->SIG=NVO;
NVO->ANT=Act;
Act = NVO;
if(NVO->SIG) NVO->SIG->ANT = NVO;
}
}
Lo podrías escribir mejor pero yo lo hice así no más, en cuanto a la ejecucion me fije si no se interrumpia y no se interrumpio, pero en cuanto al funcionamiento no me fije mucho. Ah, y me olvide algo te conviene que cuando rebobinas el puntero actual
trabajes con una copia para devolverlo a su lugar luego.
Espero haberte ayudado, disculpá si mi comentario fue muy largo, es que cuando tengo sueño redundo, jeje. Siento si me fije en todo,
soy muy obsesivo. Debería irme a dormir. Bueno, en fin saludos.
Mostrar Mensajes