Hola, ¿como les va?
estoy haciendo un programa para copiar una lista simplemente enlazada y imprimirla, la cuestion es que me tira 2 errores que deben ser sencillos, pero no logro resolverlos..

Agradeceria mucho su ayuda.

#include <stdio.h>
#include <stdlib.h>
 
struct registro
{
    int valor;
    struct registro *sig;
};
 
typedef struct registro _nodo;
 
_nodo *crearlista(_nodo *apuntador)
{
    return (apuntador=NULL);
}
 
_nodo *insertarEnLista(int numero, _nodo *apuntador )
{
    _nodo *registroNuevo, *apuntadorAuxiliar;
 
    registroNuevo = (_nodo*)/*casteo*/ malloc(sizeof(_nodo));
 
    if(registroNuevo!=NULL)
    {/*lleno el nuevo nodo suelto*/
        registroNuevo->valor=numero;
        registroNuevo->sig=NULL;
 
        if(apuntador==NULL)
            apuntador=registroNuevo;
        else
        {
            apuntadorAuxiliar = apuntador;
 
            while(apuntadorAuxiliar->sig != NULL)
                apuntadorAuxiliar = apuntadorAuxiliar->sig;
 
            apuntadorAuxiliar->sig = registroNuevo;
        }
    }
 
    return apuntador;
}
 
void imprimirLista(_nodo *apuntador)
{
    _nodo *apuntadorAuxiliar;
 
    apuntadorAuxiliar=apuntador;
 
    printf("Contenido de la lista: inicioLista->");
 
    while(apuntadorAuxiliar != NULL)
    {
        printf("%d -> ", apuntadorAuxiliar->valor);
        apuntadorAuxiliar = apuntadorAuxiliar->sig;
    }
    printf("NULL\n");
 
    return;
}
 
_nodo *apuntador2 s_copy(_nodo *apuntador)
{
    _nodo *registroAuxiliar, _nodo *apuntador2 , *apuntadorAuxiliar , *apuntadorAuxiliar2;
    apuntador2*=crearlista(*apuntador2);
 
 
      if(apuntador==NULL)
            printf("la lista que copio estaba vacia");
            apuntador2=apuntador;
 
      else
        {
            apuntadorAuxiliar = apuntador;
            apuntadorAuxiliar2 = apuntador2;
 
            while(apuntadorAuxiliar->sig != NULL)
                {
                    *apuntadorAuxiliar2=*apuntadorAuxiliar;
 
                    apuntadorAuxiliar2=apuntadorAuxiliar2->sig;
                    apuntadorAuxiliar = apuntadorAuxiliar->sig;
                }
        }
 
        return apuntador2;
}
 
 
int main()
{
    _nodo *inicioLista;
    _nodo *inicioLista2;
 
    inicioLista=crearlista(inicioLista);
    inicioLista2=crearlista(inicioLista2);
 
 
    inicioLista = insertarEnLista(5, inicioLista);
    inicioLista = insertarEnLista(6, inicioLista);
    inicioLista = insertarEnLista(7, inicioLista);
    inicioLista = insertarEnLista(8, inicioLista);
    inicioLista = insertarEnLista(9, inicioLista);
 
    inicioLista2 = s_copy(inicioLista);
 
    imprimirLista(inicioLista2);
 
 
 
    return EXIT_SUCCESS;
}
 
 

muchas gracias amigo, ya no me da mas ese problema, ahora solo obtengo la violacion de segmento de la que hablabas, en cuanto la solucione subo el codigo final, por si alguien lo precisa en el futuro. un saludo.

¡Buenas!

_nodo *insertarEnLista(int numero, _nodo *apuntador )
 

No puedes modificar un puntero exterior si no tienes un doble puntero. Si cuando llamas a la funcion el puntero que usas en la llamada tiene el valor NULL, copiaras dicho valor en el parametro que utiliza la funcion internamente, pero anque cambies la direccion a la que apunta el puntero interno de la funcion, no estaras cambiando el externo, por lo tanto seguira siendo NULL cuando salga de la funcion:

#include <stdio.h>
 
void puntero(int *ptr)
{
    int d;
 
    ptr = &d;
}
 
int main(int argc, char *argv[])
{
    int *ptr = NULL;
 
    puntero(ptr);
 
    printf("%p\n",ptr);
 
    return 0;
}
 

Si quieres cambiar el puntero tendras que mandar un puntero al puntero:

#include <stdio.h>
 
void puntero(int **ptr)
{
    int d;
 
    //accedemos al puntero exterior para poder modificarlo
    (*ptr) = &d;
}
 
int main(int argc, char *argv[])
{
    int *ptr = NULL;
 
    puntero(&ptr);
 
    printf("%p\n",ptr);
 
    return 0;
}
 

Por lo tanto el prototipo de tu nueva funcion es:

_nodo *insertarEnLista(int numero, _nodo **apuntador )
 

Y tendras que modificar el resto del codigo para que funcione de forma correcta.

Y con esta funcion no se que quieres hacer, pero realmente no esta haciendo nada efectivo.

 
 
_nodo * s_copy(_nodo *apuntador)
{
    _nodo *registroAuxiliar, *apuntador2 , *apuntadorAuxiliar , *apuntadorAuxiliar2;
    apuntador2=crearlista(apuntador2);
 
 
      if(apuntador==NULL) {
            printf("la lista que copio estaba vacia");
            apuntador2=apuntador;
      }
 
      else
        {
            apuntadorAuxiliar = apuntador;
            apuntadorAuxiliar2 = apuntador2;
 
            while(apuntadorAuxiliar->sig != NULL)
                {
                    *apuntadorAuxiliar2=*apuntadorAuxiliar;
 
                    apuntadorAuxiliar2=apuntadorAuxiliar2->sig;
                    apuntadorAuxiliar = apuntadorAuxiliar->sig;
                }
        }
 
        return apuntador2;
}
 

Si nos cuentas cual es tu intencion con esta funcion te podremos ayudar.

¡Saludos!

Cierto. Estaba con el chip de usar el propio puntero a la raiz como parametro de entrada/salida y usar el valor de retorno para informar de posibles errores...