Skip to main content
se añadieron 334 caracteres en el cuerpo
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

El error sale porque MostrarMenu es una función estática definida dentro de Biblioteca. Al ser una función estática puedes llamarla de dos formas diferentes:

while fin is False:
    Biblioteca.MostrarMenu() # opcion 1 - acceso via el nombre de la clase
    biblioteca.MostrarMenu() # opcion 2 - acceso via instancia de la clase

Bueno, la segunda opción como tal no va a funcionar directamente. Para que funcione hay que declarar previamente el método como estático (Gracias a @abulafia por darse cuenta del detalle):

class Biblioteca:
    @staticmethod
    def MostrarMenu():

El resto de llamadas también están mal ya que son funciones miembros de biblioteca. En este caso no son funciones estáticas, luego necesitas la una instancia de Biblioteca como punto de partida:

if (opcion == 1):
    biblioteca.AñadirLibroABiblioteca()
elif (opcion == 2):
    biblioteca.MostrarBiblioteca()
elif (opcion == 3):
    biblioteca.BorrarLibro()
elif (opcion == 4):
    biblioteca.NumeroLibros()
elif (opcion == 5):
    fin = True

En otro orden de cosas, te sugiero separar las funciones con una o dos líneas en blanco... Tener todo el código compactado no va a facilitar su lectura ni su mantenimiento

El error sale porque MostrarMenu es una función estática definida dentro de Biblioteca. Al ser una función estática puedes llamarla de dos formas diferentes:

while fin is False:
    Biblioteca.MostrarMenu() # opcion 1 - acceso via el nombre de la clase
    biblioteca.MostrarMenu() # opcion 2 - acceso via instancia de la clase

El resto de llamadas también están mal ya que son funciones miembros de biblioteca. En este caso no son funciones estáticas, luego necesitas la una instancia de Biblioteca como punto de partida:

if (opcion == 1):
    biblioteca.AñadirLibroABiblioteca()
elif (opcion == 2):
    biblioteca.MostrarBiblioteca()
elif (opcion == 3):
    biblioteca.BorrarLibro()
elif (opcion == 4):
    biblioteca.NumeroLibros()
elif (opcion == 5):
    fin = True

En otro orden de cosas, te sugiero separar las funciones con una o dos líneas en blanco... Tener todo el código compactado no va a facilitar su lectura ni su mantenimiento

El error sale porque MostrarMenu es una función estática definida dentro de Biblioteca. Al ser una función estática puedes llamarla de dos formas diferentes:

while fin is False:
    Biblioteca.MostrarMenu() # opcion 1 - acceso via el nombre de la clase
    biblioteca.MostrarMenu() # opcion 2 - acceso via instancia de la clase

Bueno, la segunda opción como tal no va a funcionar directamente. Para que funcione hay que declarar previamente el método como estático (Gracias a @abulafia por darse cuenta del detalle):

class Biblioteca:
    @staticmethod
    def MostrarMenu():

El resto de llamadas también están mal ya que son funciones miembros de biblioteca. En este caso no son funciones estáticas, luego necesitas la una instancia de Biblioteca como punto de partida:

if (opcion == 1):
    biblioteca.AñadirLibroABiblioteca()
elif (opcion == 2):
    biblioteca.MostrarBiblioteca()
elif (opcion == 3):
    biblioteca.BorrarLibro()
elif (opcion == 4):
    biblioteca.NumeroLibros()
elif (opcion == 5):
    fin = True

En otro orden de cosas, te sugiero separar las funciones con una o dos líneas en blanco... Tener todo el código compactado no va a facilitar su lectura ni su mantenimiento

Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

El error sale porque MostrarMenu es una función estática definida dentro de Biblioteca. Al ser una función estática puedes llamarla de dos formas diferentes:

while fin is False:
    Biblioteca.MostrarMenu() # opcion 1 - acceso via el nombre de la clase
    biblioteca.MostrarMenu() # opcion 2 - acceso via instancia de la clase

El resto de llamadas también están mal ya que son funciones miembros de biblioteca. En este caso no son funciones estáticas, luego necesitas la una instancia de Biblioteca como punto de partida:

if (opcion == 1):
    biblioteca.AñadirLibroABiblioteca()
elif (opcion == 2):
    biblioteca.MostrarBiblioteca()
elif (opcion == 3):
    biblioteca.BorrarLibro()
elif (opcion == 4):
    biblioteca.NumeroLibros()
elif (opcion == 5):
    fin = True

En otro orden de cosas, te sugiero separar las funciones con una o dos líneas en blanco... Tener todo el código compactado no va a facilitar su lectura ni su mantenimiento