ya que la pregunta es de OOP, haría mal con responder y no señalar las cosas que se pueden mejorar (desde mi humilde perspectiva). Empecemos...
Sí, definitivamente está mal heredar de una clase para sobreescribir un método al cual estás cambiando sus parámetros de entrada. Con esto estás rompiendo con el Principio de Sustitución de Liskov.
Ahora bien, hiciste bien en separar los concerns al crear una clase que gestione el registro, y otra que represente la entidad Socio. Sin embargo, la clase Socio
parece ser más un DTO que una clase propiamente dicha. Para esto, Python provee las maravillosas namedtuples.
Así, puedes representar a la entidad Socio de la siguiente manera:
from collections import namedtuple
Socio = namedtuple('Socio', 'nombre dni telefono')
Ahora vamos con la parte de gestión de registros. Otro principio que hay que tener en cuenta a la hora de diseñar sistemas bajo el paradigma OO, es que cada clase debe tener una única responsabilidad (SRP). Por lo tanto estaríamos haciendo un error si la clase Gestion_socios
se encarga de en efecto gestionar los registros y además de servir como interfaz al usuario.
¿Qué tal si abstraemos las tareas de gestión de socios y creamos una clase más genérica capaz de añadir y eliminar objetos de tu "base de datos"? Así podemos tener esta clase lista por si en algún momento deseas ya no solo agregar socios, sino empleados, visitantes, etc.
Además, vamos a aprovechar y desacoplamos nuestra "base de datos" de esa clase, y así cumplimos con el Principio de inversión de la dependencia, dando la posibilidad de "inyectar" la base de datos de nuestro sistema a la clase encargada de manejarla (lo cual sería beneficioso si quieres cargar un diccionario ya existente).
Yo sugeriría que esa clase se pareciera a lo siguiente:
class GestorDePersonas:
def __init__(self, database=None):
if database is None:
database = dict()
self.database = database
def add(self, objeto):
if self._existe(objeto):
raise KeyError(f'{objeto.__class__.__name__} ya existente')
self.database[objeto.dni] = objeto
def remove(self, objeto):
if self._existe(objeto):
self.database.pop(objeto.dni)
else:
raise KeyError(f'{objeto.__class__.__name__} no existe')
def get(self, dni):
sentinel = type('', (), {})
sentinel.dni = None
return self.database.get(dni, sentinel)
def _existe(self, objeto):
dni = objeto.dni
return True if dni in self.database else False
Ahora, creemos una clase responsable de ser la interfaz entre el usuario y nuestra aplicación.
class SocioUI:
def __init__(self, gestor):
self.gestor = gestor
def dar_de_alta(self):
try:
nombre = input("Introduce tu nombre: ")
dni = int(input("Introduce tu DNI sin letra: "))
telefono = int(input("Introduce tu número de teléfono: "))
socio = Socio(nombre, dni, telefono)
self.gestor.add(socio)
print('Socio añadido con éxito!')
except KeyError as e:
print(e)
def dar_de_baja(self):
try:
dni = input('Ingresa tu dni')
socio = self.gestor.get(dni)
self.gestor.remove(socio)
except KeyError as e:
print(e)
Y así, finalmente, podemos usar nuestro pequeño sistema como muestro a continuación:
gestor = GestorDePersonas()
ui = SocioUI(gestor)
ui.dar_de_alta()
De esta manera, eso que preguntas de "entrar simplemente para disfrutar del contenido" lo puedes conseguir añadiendo los métodos de ese contenido que deseas mostrar, a la clase SocioUI
(siempre y cuando ese contenido esté bajo la misma responsabilidad que los métodos ya creados en la clase), o puedes crear nuevas clases de UI que se alineen con el contenido deseado, y agrupar todas esas clases similares dentro de otra que sea la principal, o main. De ser así, puedes estudiar cómo aplicar el patrón de Chain of responsability para agrupar estas clases que puedes crear.
Nota, por favor, cómo deben ser nombradas las clases y métodos, tanto por convención, como por semántica (intentar que al nombrar clases, estas sean sustantivos, y los métodos sean verbos o predicados).
También, nota como por lo general conviene favorecer la composición sobre la herencia. No es que la herencia sea mala, sino que hay que saber cuándo usarla y sus implicancias más allá del simple hecho de copiar el código de una clase a otra. Es decir, considerar cosas como el coupling, la rigidez creada, "explosión de clases", etc.)
Gestion_socios
herede deSocio
.self.socios[dni] = Socio(nombre_socio,dni,telefono)
no está mal, pero sí la herencia.