No se si se buena practica, pero te recomendaria que crearas un atributo puntero de tipo nodo en tu clase que apunte al ultimo nodo insertado para ahorrarte ese ciclo, además creo que el while podría mejorar si lo colocaras como:
while(aux->SgteNodo() != NULL) { //Esta condición sera cierta hasta que encuentre un nodo cuya siguiente dirección apunte a NULL
aux = aux->SgteNodo();
}
Nodo *aux2 = new Nodo(nuevo);
aux->insertarSgte(aux);
Espero haber sido de ayuda
:)
EDIT:
Supongamos que tenemos la siguiente clase Nodo,
class Nodo {
public:
Nodo(){};
Nodo(int dato){
this->dato = dato;
this->sgte = NULL; //tambien puedes escribir nullptr lo pongo así porque es la palabra que ocupas
}
Nodo* SgteNodo() { return sgte;}
int getDato() { return dato; }
void setDato(int dato) { this->dato = dato; }
void insertarSgte(Nodo *sgte){
this->sgte = sgte;
}
//Agregamos una sobrecarga de operador ostream para imprimir nuestro nodo con mas facilidad
ostream& operator<<(ostream& os, Nodo n){
return os<<n.getDato(); //Solo imprimimos el contenido del nodo
}
private:
int dato;
Nodo *sgte;
};
Considerando lo anterior y suponiendo que quieres mostrar el contenido de la lista, puedes tener como sugerencia lo siguiente, desconozco el nombre del método pero lo siguiente debe ir dentro de esa funcion.
Nodo *aux = this->primerNodo;
while(aux->SgteNodo() != NULL)
{
//cout << aux->getDato() << endl; //Opcion A ocupas el metodo getDato de la clase nodo para mostrar el contenido
cout << *aux << endl; //Opcion B muestras el objeto de la siguiente forma con ayuda de la sobrecarga del operador mostrada en la clase anterior
aux = aux->SgteNodo();
}
Como puedes observar el funcionamiento del condicional dentro del while es revisar si el siguiente nodo no sea NULL, te recomiendo que practiques un poco más de punteros, otra cosa que me sirve mucho es que a los atributos de tu clase les crees un getter y un setter junto con el nombre del atributo, eso sirve para dos cosas, uno cuando llames un metodo de cualquier objeto ya no te preocupas como se llama el metodo, simplemente piensas "oh quiero obtener el atributo tal de este objeto" y solo llamas al metodo getAtributo() o si lo quieres cambiar con el setAtributo(), lo menciono esto porque vi que ocupaste SgteNodo() tanto para mostrar y guardar el nodo ocupas el mismo metodo, se puede pero si en un futuro quieres reciclar las clases tienes que recordar una particularidad y puedes meterte en una situacion un poco complicada, por ello haciendo eso puedes mantener las cosas un poco más simples, lo mencionado es solo una opinión y algo basado en mi poca experiencia, pero que me ha resultado útil en varios proyectos de la universidad. Exito en tu trabajo.
Nota si quieres crear un estructura más reciclable utiliza templates.