Merge pull request #46 from DanielYanes-YH25001/fix/correccion-prs-33-34-42-y-44

Correción de pull requests 33, 34, 42 y 44

Author: DanielYanes-YH25001

models/asignacion.py

@@ -1,7 +1,7 @@
 class Asignacion:
-  """Clase para crear la asignacion de proyecto, con las validaciones"""
+  """Clase para modelar una asignación"""
 
-  def __init__(self, id, id_tarea, id_empleado, horas=0.0):
+  def __init__(self, id, id_tarea, id_empleado, horas = 0.0):
     if not isinstance(id, int) or id <= 0:
       raise ValueError("El ID de la asignación debe ser un entero positivo.")
     if not isinstance(id_tarea, int) or id_tarea <= 0:
@@ -18,29 +18,23 @@ def __init__(self, id, id_tarea, id_empleado, horas=0.0):
 
   def to_dict(self):
     """Método para guardar los datos en JSON"""
-
     return {
       "id": self.id,
       "id_tarea": self.id_tarea,
       "id_empleado": self.id_empleado,
-      "horas": self.horas,
+      "horas": self.horas
     }
 
   @classmethod
   def from_dict(cls, data):
-    """Método de clase para traer una asigancion"""
-
+    """Método de clase para traer una asignación"""
     return cls(
-      id=data["id"],
-      id_tarea=data["id_tarea"],
-      id_empleado=data["id_empleado"],
-      horas=data.get("horas", 0.0),
+      id = data["id"],
+      id_tarea = data["id_tarea"],
+      id_empleado = data["id_empleado"],
+      horas = data.get("horas", 0.0)
     )
 
   def __str__(self):
-    """Método para imprimir los atributos de la asignacion"""
-
-    return (
-      f"Asignacion(id={self.id}, id_tarea={self.id_tarea}, "
-      f"id_empleado={self.id_empleado}, horas={self.horas})"
-    )
+    """Método para imprimir los atributos de la asignación"""
+    return f"Asignacion(id={self.id}, id_tarea={self.id_tarea}, id_empleado={self.id_empleado}, horas={self.horas})"

models/empleado.py

@@ -1,5 +1,5 @@
 class Empleado:
-  """Clase para crear el empleado, con las validaciones"""
+  """Clase para modelar un empleado"""
 
   def __init__(self, id, nombre, rol):
     if not isinstance(id, int) or id <= 0:
@@ -15,24 +15,21 @@ def __init__(self, id, nombre, rol):
 
   def to_dict(self):
     """Método para guardar los datos en JSON"""
-
     return {
       "id": self.id,
       "nombre": self.nombre,
-      "rol": self.rol,
+      "rol": self.rol
     }
 
   @classmethod
   def from_dict(cls, data):
     """Método de clase para traer un empleado"""
-
     return cls(
-      id=data["id"],
-      nombre=data["nombre"],
-      rol=data["rol"],
+      id = data["id"],
+      nombre = data["nombre"],
+      rol = data["rol"]
     )
 
   def __str__(self):
     """Método para imprimir los datos de un empleado"""
-    
     return f"Empleado(id={self.id}, nombre='{self.nombre}', rol='{self.rol}')"

models/proyecto.py

@@ -1,40 +1,35 @@
 class Proyecto:
-  """clase para crear los proyectos, con las validaciones para id, nomre y estado"""
+  """Clase para modelar un proyecto"""
 
   def __init__(self, id, nombre, estado="Activo"):
     if not isinstance(id, int) or id <= 0:
       raise ValueError("El ID del proyecto debe ser un entero positivo.")
     if not nombre or not isinstance(nombre, str):
       raise ValueError("El nombre del proyecto debe ser una cadena no vacía.")
     if estado not in {"Activo", "Finalizado", "Pausado"}:
-      raise ValueError(
-        "El estado del proyecto debe ser 'Activo', 'Finalizado' o 'Pausado'."
-      )
+      raise ValueError("El estado del proyecto debe ser 'Activo', 'Finalizado' o 'Pausado'.")
 
     self.id = id
     self.nombre = nombre.strip()
     self.estado = estado
 
   def to_dict(self):
-    """Método para guardar los datos en JSON"""  
-
+    """Método para guardar los datos en JSON"""
     return {
       "id": self.id,
       "nombre": self.nombre,
-      "estado": self.estado,
+      "estado": self.estado
     }
 
   @classmethod
   def from_dict(cls, data):
     """Método de clase para traer un proyecto"""
-
     return cls(
-      id=data["id"],
-      nombre=data["nombre"],
-      estado=data.get("estado", "Activo"),
+      id = data["id"],
+      nombre = data["nombre"],
+      estado = data.get("estado", "Activo")
     )
- 
+
   def __str__(self):
     """Método para imprimir los atributos de un proyecto"""
-    
     return f"Proyecto(id={self.id}, nombre='{self.nombre}', estado='{self.estado}')"

models/tarea.py

@@ -15,24 +15,21 @@ def __init__(self, id, id_proyecto, descripcion):
 
   def to_dict(self):
     """Método para guardar los datos en JSON"""
-
     return {
       "id": self.id,
       "id_proyecto": self.id_proyecto,
-      "descripcion": self.descripcion,
+      "descripcion": self.descripcion
     }
-  
+
   @classmethod
   def from_dict(cls, data):
     """Método de clase para traer una tarea"""
-
     return cls(
-      id=data["id"],
-      id_proyecto=data["id_proyecto"],
-      descripcion=data["descripcion"],
+      id = data["id"],
+      id_proyecto = data["id_proyecto"],
+      descripcion = data["descripcion"]
     )
- 
+
   def __str__(self):
     """Método para imprimir los datos de una tarea"""
-    
     return f"Tarea(id={self.id}, id_proyecto={self.id_proyecto}, descripcion='{self.descripcion}')"

modules/empleados.py

@@ -9,7 +9,7 @@ def menu_empleados(empleado_service):
   Args:
     empleado_service (EmpleadoService): Servicio de empleado con operaciones CRUD
   """
-  
+
   while True:
     render_menu(
       "Gestión de Empleados",

modules/persistence.py

@@ -22,7 +22,7 @@ def _crear_archivo_vacio(self):
 
   def cargar_datos(self):
     """Carga el archivo JSON, creando un archivo vacío si no existe"""
-    
+
     if not self.filepath.exists():
       self._crear_archivo_vacio()
 

modules/ui.py

@@ -31,7 +31,7 @@ def render_menu(title, options):
 
   Args:
     title (str): Título del menú
-    options (list[str]): Lista de opciones a mostrar 
+    options (list[str]): Lista de opciones a mostrar
   """
 
   table = Table(show_header = False, box = box.SIMPLE, pad_edge = False)

services/asignacion_service.py

@@ -2,22 +2,22 @@
 
 
 class AsignacionService:
-  # Servicio para gestionar asignaciones de empleados a tareas
-  
+  """Servicio para gestionar asignaciones de tareas a empleados"""
+
   def __init__(self, manejador_persistencia):
-    # Constructor: recibe el manejador de persistencia
     self.manejador_persistencia = manejador_persistencia
 
   def _cargar(self):
-    # Carga los datos desde el manejador de persistencia
+    """Carga los datos desde el manejador de persistencia"""
     return self.manejador_persistencia.cargar_datos()
 
   def _guardar(self, datos):
-    # Guarda los datos en el manejador de persistencia
+    """Guarda los datos en el manejador de persistencia"""
     self.manejador_persistencia.guardar_datos(datos)
 
   def crear(self, id_tarea, id_empleado):
-    # Crea una nueva asignación con horas iniciales en 0
+    """Crea una nueva asignación con horas iniciales en 0"""
+
     datos = self._cargar()
     nuevo_id = self.manejador_persistencia.obtener_proximo_id(datos["asignaciones"])
     asignacion = Asignacion(nuevo_id, id_tarea, id_empleado, horas=0.0)
@@ -26,18 +26,21 @@ def crear(self, id_tarea, id_empleado):
     return asignacion
 
   def obtener_todos(self):
-    # Retorna todas las asignaciones
+    """Retorna todas las asignaciones"""
+
     datos = self._cargar()
     return [Asignacion.from_dict(a) for a in datos["asignaciones"]]
 
   def obtener_por_id(self, id_asignacion):
-    # Obtiene una asignación por su ID
+    """Obtiene una asignación por su ID"""
+
     datos = self._cargar()
     asignacion = next((a for a in datos["asignaciones"] if a["id"] == id_asignacion), None)
     return Asignacion.from_dict(asignacion) if asignacion else None
 
   def obtener_por_tarea(self, id_tarea):
-    # Obtiene todas las asignaciones de una tarea específica
+    """Obtiene todas las asignaciones de una tarea específica"""
+
     datos = self._cargar()
     return [
       Asignacion.from_dict(a)
@@ -46,20 +49,22 @@ def obtener_por_tarea(self, id_tarea):
     ]
 
   def obtener_por_tarea_y_empleado(self, id_tarea, id_empleado):
-    # Obtiene una asignación por tarea y empleado
+    """Obtiene una asignación por tarea y empleado"""
+
     datos = self._cargar()
     asignacion = next(
       (
         a
         for a in datos["asignaciones"]
         if a["id_tarea"] == id_tarea and a["id_empleado"] == id_empleado
       ),
-      None,
+      None
     )
     return Asignacion.from_dict(asignacion) if asignacion else None
 
   def obtener_por_empleado(self, id_empleado):
-    # Obtiene todas las asignaciones de un empleado específico
+    """Obtiene todas las asignaciones de un empleado específico"""
+
     datos = self._cargar()
     return [
       Asignacion.from_dict(a)
@@ -68,7 +73,8 @@ def obtener_por_empleado(self, id_empleado):
     ]
 
   def agregar_horas(self, id_tarea, id_empleado, horas):
-    # Agrega horas a una asignación (solo valida horas ingresadas, no el total)
+    """Agrega horas a una asignación"""
+
     if not isinstance(horas, (int, float)):
       raise ValueError("Las horas deben ser un número.")
     if horas < 0 or horas > 12:
@@ -81,7 +87,7 @@ def agregar_horas(self, id_tarea, id_empleado, horas):
         for a in datos["asignaciones"]
         if a["id_tarea"] == id_tarea and a["id_empleado"] == id_empleado
       ),
-      None,
+      None
     )
 
     if asignacion is None:
@@ -98,15 +104,16 @@ def agregar_horas(self, id_tarea, id_empleado, horas):
     return True
 
   def actualizar_horas(self, id_asignacion, horas):
-    # Actualiza el total de horas de una asignación existente
+    """Actualiza el total de horas de una asignación existente"""
+
     if not isinstance(horas, (int, float)):
       raise ValueError("Las horas deben ser un número.")
     if horas < 0 or horas > 12:
       raise ValueError("No se pueden registrar más de 12 horas al día.")
 
     datos = self._cargar()
     asignacion = next((a for a in datos["asignaciones"] if a["id"] == id_asignacion), None)
-    
+
     if not asignacion:
       return False
 
@@ -115,13 +122,14 @@ def actualizar_horas(self, id_asignacion, horas):
     return True
 
   def eliminar(self, id_asignacion):
-    # Elimina una asignación por su ID
+    """Elimina una asignación por su ID"""
+
     datos = self._cargar()
     asignacion = next((a for a in datos["asignaciones"] if a["id"] == id_asignacion), None)
-
     if not asignacion:
       return False
 
+    # Elimina la asignación
     datos["asignaciones"] = [a for a in datos["asignaciones"] if a["id"] != id_asignacion]
     self._guardar(datos)
     return True

services/empleado_service.py

@@ -2,22 +2,22 @@
 
 
 class EmpleadoService:
-  # Servicio para gestionar empleados
-  
+  """Servicio para gestionar empleados"""
+
   def __init__(self, manejador_persistencia):
-    # Constructor: recibe el manejador de persistencia
     self.manejador_persistencia = manejador_persistencia
 
   def _cargar(self):
-    # Carga los datos desde el manejador de persistencia
+    """Carga los datos desde el manejador de persistencia"""
     return self.manejador_persistencia.cargar_datos()
 
   def _guardar(self, datos):
-    # Guarda los datos en el manejador de persistencia
+    """Guarda los datos en el manejador de persistencia"""
     self.manejador_persistencia.guardar_datos(datos)
 
   def crear(self, nombre, rol):
-    # Crea un nuevo empleado
+    """Crea un nuevo empleado"""
+
     datos = self._cargar()
     nuevo_id = self.manejador_persistencia.obtener_proximo_id(datos["empleados"])
     empleado = Empleado(nuevo_id, nombre, rol)
@@ -26,18 +26,21 @@ def crear(self, nombre, rol):
     return empleado
 
   def obtener_todos(self):
-    # Retorna todos los empleados
+    """Retorna todos los empleados"""
+
     datos = self._cargar()
     return [Empleado.from_dict(e) for e in datos["empleados"]]
 
   def obtener_por_id(self, id_empleado):
-    # Obtiene un empleado por su ID
+    """Obtiene un empleado por su ID"""
+
     datos = self._cargar()
     empleado = next((e for e in datos["empleados"] if e["id"] == id_empleado), None)
     return Empleado.from_dict(empleado) if empleado else None
 
   def actualizar(self, id_empleado, nombre=None, rol=None):
-    # Actualiza los datos de un empleado (nombre y/o rol)
+    """Actualiza los datos de un empleado (nombre y/o rol)"""
+
     datos = self._cargar()
     empleado = next((e for e in datos["empleados"] if e["id"] == id_empleado), None)
     if not empleado:
@@ -57,7 +60,8 @@ def actualizar(self, id_empleado, nombre=None, rol=None):
     return True
 
   def eliminar(self, id_empleado):
-    # Elimina un empleado y sus asignaciones relacionadas
+    """Elimina un empleado y sus asignaciones relacionadas"""
+
     datos = self._cargar()
     empleado = next((e for e in datos["empleados"] if e["id"] == id_empleado), None)
     if not empleado: