O problema está mesmo no próprio for
. Ele deveria ter outra instrução de inicio assim como outra de fim. Desta forma:
//de 0 até ao tamanho - 1 para parar no ultimo
for(int i = 0; i < tamanho - 1; i++){ //de 0 até ao tamanho - 1 para parar no ultimo
aux = aux.getProximo();
}
Códigos de listas são normalmente escritos com while
e normalmentecostumam executam até apanhar o fim da lista, o (null
). Mesmo com uma restrição de usar um for
podemos usar a mesma lógica, o que simplifica:
else {
Node aux;
//escrito desta maneira fica sem conteúdo, servindo so para navegar até ao fimultimo
for(aux = inicio; aux.getProximo() != null ; aux = aux.getProximo());
aux.setProximo(novoNode);
}
Neste ultimo exemplo começamos a navegar no inicio
, e navegamos até que o próximo seja null
, que é o caso de paragem. Quando o próximo é null
significa que estamos no ultimo elemento que é onde queremos parar. Para cada navegação vamos andando para o próximo elemento com aux = aux.getProximo();
como já era feito anteriormente.