Skip to main content
se añadieron 535 caracteres en el cuerpo
Origen Enlace
Candid Moe
  • 26.5k
  • 10
  • 23
  • 47

Casi lo hicistes bien.

Estas usando una variable global (mayor), que se confunde con el argumento de la función. Por lo demás, es innecesario pasar "mayor" como parámetro si lo tienes definido como global.

Solución:

  1. Elimina "mayor" de la lista de argumentos.

  2. Estas usando los mismos nombres para variables globales ("valores") y argumentos de la función. Esto generar confunsiónconfusión y te traeratraerá problemas sin fin a la hora de depurar tu programa. Renombra las variables para hacerlas diferentes.

  3. Declara "global mayor" dentro de la función. De otro modo, "mayor" sólo sería una variable local, distinta al "mayor" que pones fuera.

    n=3
    mayor=0
    valores_in=[];
    
    def Mayor(valores,i):
        global mayor
        if valores[i]>mayor:
            mayor=valores[i]
        return mayor
    
    for i in range(n):
        print("Valores ingresados:",i+1,"/",n)
        print("Ingrese",n,"valores que desee..")
        x=int(input("Valor: "))
        valores_in.append(x)
        Mayor(valores_in,i)
    
    print("El valor mas alto ingresado es",mayor)
    

genera:

Valores ingresados: 1 / 3
Ingrese 3 valores que desee..
Valor: 3
Valores ingresados: 2 / 3
Ingrese 3 valores que desee..
Valor: 6
Valores ingresados: 3 / 3
Ingrese 3 valores que desee..
Valor: 3
El valor mas alto ingresado es 6

Otras observaciones

No tiene mucho sentido llamar a la función tres veces, pasando la lista "valores_in" y el indice "i". Podrías pasar la lista completa y resolver el tema de una vez, o mejor, usar la función "max"

n=3
valores_in=[]

for i in range(n):
    print("Valores ingresados:",i+1,"/",n)
    print("Ingrese",n,"valores que desee..")
    x=int(input("Valor: "))
    valores_in.append(x)

mayor = max(valores_in)
print("El valor mas alto ingresado es",mayor)

Casi lo hicistes bien.

Estas usando una variable global (mayor), que se confunde con el argumento de la función. Por lo demás, es innecesario pasar "mayor" como parámetro si lo tienes definido como global.

Solución:

  1. Elimina "mayor" de la lista de argumentos.

  2. Estas usando los mismos nombres para variables globales ("valores") y argumentos de la función. Esto generar confunsión y te traera problemas sin fin a la hora de depurar tu programa. Renombra las variables para hacerlas diferentes.

  3. Declara "global mayor" dentro de la función. De otro modo, "mayor" sólo sería una variable local, distinta al "mayor" que pones fuera.

    n=3
    mayor=0
    valores_in=[];
    
    def Mayor(valores,i):
        global mayor
        if valores[i]>mayor:
            mayor=valores[i]
        return mayor
    
    for i in range(n):
        print("Valores ingresados:",i+1,"/",n)
        print("Ingrese",n,"valores que desee..")
        x=int(input("Valor: "))
        valores_in.append(x)
        Mayor(valores_in,i)
    
    print("El valor mas alto ingresado es",mayor)
    

genera:

Valores ingresados: 1 / 3
Ingrese 3 valores que desee..
Valor: 3
Valores ingresados: 2 / 3
Ingrese 3 valores que desee..
Valor: 6
Valores ingresados: 3 / 3
Ingrese 3 valores que desee..
Valor: 3
El valor mas alto ingresado es 6

Casi lo hicistes bien.

Estas usando una variable global (mayor), que se confunde con el argumento de la función. Por lo demás, es innecesario pasar "mayor" como parámetro si lo tienes definido como global.

Solución:

  1. Elimina "mayor" de la lista de argumentos.

  2. Estas usando los mismos nombres para variables globales ("valores") y argumentos de la función. Esto generar confusión y te traerá problemas sin fin a la hora de depurar tu programa. Renombra las variables para hacerlas diferentes.

  3. Declara "global mayor" dentro de la función. De otro modo, "mayor" sólo sería una variable local, distinta al "mayor" que pones fuera.

    n=3
    mayor=0
    valores_in=[];
    
    def Mayor(valores,i):
        global mayor
        if valores[i]>mayor:
            mayor=valores[i]
        return mayor
    
    for i in range(n):
        print("Valores ingresados:",i+1,"/",n)
        print("Ingrese",n,"valores que desee..")
        x=int(input("Valor: "))
        valores_in.append(x)
        Mayor(valores_in,i)
    
    print("El valor mas alto ingresado es",mayor)
    

genera:

Valores ingresados: 1 / 3
Ingrese 3 valores que desee..
Valor: 3
Valores ingresados: 2 / 3
Ingrese 3 valores que desee..
Valor: 6
Valores ingresados: 3 / 3
Ingrese 3 valores que desee..
Valor: 3
El valor mas alto ingresado es 6

Otras observaciones

No tiene mucho sentido llamar a la función tres veces, pasando la lista "valores_in" y el indice "i". Podrías pasar la lista completa y resolver el tema de una vez, o mejor, usar la función "max"

n=3
valores_in=[]

for i in range(n):
    print("Valores ingresados:",i+1,"/",n)
    print("Ingrese",n,"valores que desee..")
    x=int(input("Valor: "))
    valores_in.append(x)

mayor = max(valores_in)
print("El valor mas alto ingresado es",mayor)
Origen Enlace
Candid Moe
  • 26.5k
  • 10
  • 23
  • 47

Casi lo hicistes bien.

Estas usando una variable global (mayor), que se confunde con el argumento de la función. Por lo demás, es innecesario pasar "mayor" como parámetro si lo tienes definido como global.

Solución:

  1. Elimina "mayor" de la lista de argumentos.

  2. Estas usando los mismos nombres para variables globales ("valores") y argumentos de la función. Esto generar confunsión y te traera problemas sin fin a la hora de depurar tu programa. Renombra las variables para hacerlas diferentes.

  3. Declara "global mayor" dentro de la función. De otro modo, "mayor" sólo sería una variable local, distinta al "mayor" que pones fuera.

    n=3
    mayor=0
    valores_in=[];
    
    def Mayor(valores,i):
        global mayor
        if valores[i]>mayor:
            mayor=valores[i]
        return mayor
    
    for i in range(n):
        print("Valores ingresados:",i+1,"/",n)
        print("Ingrese",n,"valores que desee..")
        x=int(input("Valor: "))
        valores_in.append(x)
        Mayor(valores_in,i)
    
    print("El valor mas alto ingresado es",mayor)
    

genera:

Valores ingresados: 1 / 3
Ingrese 3 valores que desee..
Valor: 3
Valores ingresados: 2 / 3
Ingrese 3 valores que desee..
Valor: 6
Valores ingresados: 3 / 3
Ingrese 3 valores que desee..
Valor: 3
El valor mas alto ingresado es 6