###Los nodos no son listas. He visto en StackOverflow en Español varias veces esta confusión, y me resulta muy curioso que tantos usuarios cometan ese error.
En el código que has facilitado creas un alias de puntero a nodo (nodo*
) y dices que es una lista (lista
), eso es tan erróneo como decir que un escalón es una escalera, sinceramente ¿Te parecen lo mismo?:
Además tu función main
no tiene valor de retorno. Y generalmente no se aconseja usar using namespace std;
(consulta este hilo para saber por qué) y hay que usar std::endl
con cuidado (consulta este hilo para saber por qué).
###Solución.
Si quieres seguir con tu diseño actual (que al igual que akko no te lo aconsejo), deberás cambiar la firma de agregar
y mostrar
para recibir referencias en lugar de copias:
// Referencia a lista ---> v
void agregar(int nuevoDato, lista &unaLista) {
// vvvvvvv <--- literal de puntero nulo, evita usar la macro NULL
if (unaLista == nullptr) {
// vvvvvvvvvvvvvvv <--- inicializacion uniforme de datos simples
unaLista = new nodo{nuevoDato};
std::cout << "Nodo agregado (" << unaLista -> dato << ")\n";
// ^^^^^ <--- espacio de nombres explicito ^^^^^
// evitamos el uso de endl --/
} else {
agregar(nuevoDato, unaLista -> siguiente);
}
std::cout << '\n'; // <-- evitamos el uso de endl
// ^^^^^ <--- espacio de nombres explicito
}
// v <--- Referencia a lista
void mostrar(lista &unaLista) {
// Evitando la recursividad innecesaria
for (auto nodo = unaLista; nodo; nodo = nodo -> siguiente)
std::cout << nodo -> dato << '\n'; // <-- evitamos el uso de endl
// ^^^^^ <--- espacio de nombres explicito
std::cout << "-\n"; // <-- evitamos el uso de endl
// ^^^^^ <--- espacio de nombres explicito
}