Para llegar al final de la lista no debes parar en aux==nullptr
sino en aux->SgteNodo()==nullptr
. El motivo es que aux
no es más que una variable que almacena una dirección de memoria y modificar esa variable no va a afectar lo más mínimo a la lista.
Nodo *aux = primerNodo;
while (aux->SgteNodo() != nullptr)
{
aux = aux->SgteNodo();
}
// Ya tienes el último nodo localizado
Nodo *agregar = new Nodo(nuevo);
aux->SgteNodo(agregar);
}
Ojo a la última línea, en el ejemplo que tu has puesto has usado
aux->SgteNodo(*agregar);
Es como si esa función no admitiese un puntero sino un objeto por valor o por referencia ... cuidado con eso que puedes acabar haciendo copias de objetos sin querer, lo que te provocará fugas de memoria.
de que forma podría corregir el método SgteNodo?
Tienes que repasar un poco el funcionamiento de los punteros. La función bien podría quedar así:
class Nodo
{
Nodo* siguiente; // O como se llame tu variable miembro
public:
void SgteNodo(Nodo* nodo)
{ siguiente = nodo; }
Nodo * SgteNodo()
{ return siguiente; }
};
En cualquier caso, buena parte del problema la tiene el hecho de que estás mezclando responsabilidades.
Dado que ya tienes una clase Lista
, la misma debería ocultar la existencia de los objetos de tipo Nodo
. En consecuencia, únicamente Lista
está habilitada para crear y mantener una estructura de objetos de tipo Nodo
. Bajo esta situación, y dada la nula complejidad de los objetos de tipo Nodo
, no hay razón para que los miembros de Nodo
no sean todos públicos:
struct Nodo
{
int dato;
Nodo * siguiente;
Nodo(int valor)
: dato{valor}
, siguiente{nullptr}
{ }
};
Este diseño evita una complejidad innecesaria y mantiene tu código más limpio.