Skip to main content
4 de 4
se añadieron 20 caracteres en el cuerpo
Christian
  • 10.1k
  • 9
  • 21
  • 40

¡Hola!

He notado varios problemas en el código, primero que nada, en esta parte del código:

if pr_nom in compras:
    print(f"Hay {compras.value(pr_nom)} {pr_nom.title()}")
    if compras.value(pr_nom) in compras >= 1:
        resta = compras.value[pr_nom] - pr_can

Estas usando como condiciones las variables pr_nom y pr_can, variables que no tienen ningún valor asignado, si le has asignado un valor, pero dentro de una función, y si esa función no es ejecutada, entonces no tienen valor, deberás integrar el if a la función, así, al ejecutarla, deberás especificar el elemento y la cantidad antes, para posteriormente poder sacar las compras, pero deberás primero arreglar otros errores más como el problema de indentación que hay, recuerda que al usar el signo : debe haber un espacio de indentación, pasaría de esto:

def sacar_compras():
pr_nom = input("Ingrese un elemento a remover: ")
pr_can = int(input("Ingrese una cantidad a remover: "))

a esto:

def sacar_compras():
    pr_nom = input("Ingrese un elemento a remover: ")
    pr_can = int(input("Ingrese una cantidad a remover: "))

Otra cosa, para ejecutar el if, deberás antes asignarle un valor a pr_nom y a pc_can, has hecho un input, pero está en una función externa al procedimiento, siendo el valor que reciba, una variable local, yo te recomendaría integrar el procedimiento a la función misma, quedaría así:

if pr_nom in compras:
    print(f"Hay {compras.value(pr_nom)} {pr_nom.title()}")
else:
    print("No hay stock suficiente")
if compras.value(pr_nom) in compras >= 1:
    resta = compras.value[pr_nom] - pr_can
    print(f"Te quedan {resta}")
else:
    print("No hay stock suficiente")

Como veras, descarté uno de los else que pusiste, porque te dará un error:

else:
    print("Este elemento no estaba ingresado en la lista de compras")

El else como tal te es importante, pero, recuerda que no puede haber un else sin un if, no puede haber 2 else para un mismo if, por lo tanto, o creas otro if, que quedará mal, o lo agregas a otro else, que quedará mal, pero también puedes hacer un bloque try/except, que es lo que yo te recomiendo:

try:
    if pr_nom in compras:
        print(f"Hay {compras(pr_nom)} {pr_nom.title()}")
    else:
        print("No hay stock suficiente")
    if pr_nom in compras >= 1:
        resta = compras[pr_nom] - pr_can
        print(f"Te quedan {resta}")
    else:
        print("No hay stock suficiente")
except:
    print("Este elemento no estaba ingresado en la lista de compras")

Por último, te habrás dado cuenta que yo cambié

if compras.value(pr_nom) in compras >= 1:

Por:

if pr_nom in compras >= 1:

Esto es porque, primero, no necesitas usar .value para acceder al valor de algo en un diccionario, tampoco necesitas referirte a compras dos veces, eso te dará error.

Espero haberte aclarado

Kalio
  • 302
  • 1
  • 9