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.