Buen día,
Antes de pasar a la parte funcional, el código tiene algunos problemas de concepto.
Cuando tienes un ciclo for
, por ejemplo for i in range(len(numeros)):
el iterador i
incrementa de forma automática, no hace falta que agregues i += 1
.
En el caso de utilizar un rango, el rango que creas va a ir de cero a la longitud de la lista - 1, por lo que tu iterador se inicializará en cero cada vez que vaya a ejecutarse nuevamente el for
, por lo que la línea i = 1
está de más.
Ahora, la razón por la que no funciona es porque adentro del while
nunca reasignas el valor de mayor
una vez que el programa encuentra el valor mayor de una lista lo conserva por siempre y por eso en la segunda ocación que intenta eliminarlo de numeros
ya no existe. Un ejemplo para que se entienda
Teniendo la siguiente lista [5, 10, 1, 2]
, adentro del while
la variable mayor
obtendrá el valor 10
pero cuando se ha eliminado de la lista con numeros.remove(mayor)
la variable mayor
nunca se sobreescribe y conserva el valor de 10
, por eso en la segunda vez que se ejecuta el while
aparece el error que mencionas
Una propuesta de solución. En lugar de asignar mayor
al primer valor de tu lista podrías asignarlo a None
y en el if
que está adentro del for
revisar si mayor is None
o si mayor < numeros[i]
Ejemplo completo:
digito = 1
i = 1
numeros = []
# Toma de datos y calculos
while(digito != 0):
digito = int(input(f"Digite el {i}º numero de la primitiva (0 para acabar): "))
if(digito == 0):
print()
else:
numeros.append(digito)
i += 1
print("\n")
lista_ordenada = []
mayor = None
while(numeros != []):
for i in range(len(numeros)):
if (mayor is None) or (mayor < numeros[i]):
mayor = numeros[i]
lista_ordenada.append(mayor)
numeros.remove(mayor)
mayor = None
print(lista_ordenada)
Ejemplo de ejecución:
Digite el 1º numero de la primitiva (0 para acabar): 5
Digite el 2º numero de la primitiva (0 para acabar): 1
Digite el 3º numero de la primitiva (0 para acabar): 10
Digite el 4º numero de la primitiva (0 para acabar): 2
Digite el 5º numero de la primitiva (0 para acabar): 0
[10, 5, 2, 1]