[Proyecto I] GoEvent

[Unit0code]

Nombre: Marlon Alejandro Rodriguez Gigato
Grupo: C122
Repositorio: https://github.com/Unit0code/GoEvent

Descripción

Proyecto enfocado en la gestion de recursos y tiempo de eventos con tematica de empresa transportista en Cuba.

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/Unit0code/GoEvent

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

❌ No se encontró report.md en el repositorio.

3. Proyecto Python

⚠️ Sin pyproject.toml.
⚠️ Sin requirements.txt.

Archivos Python (10, 1,258 líneas en total):

• Events.py
• Agregar.py
• Eliminar.py
• Recursos.py
• Jsons.py
• user.py
• miscelaneo.py
• recursos_dannados.py
• main.py
• Buscar_hueco.py

4. Punto de Entrada

✅ Punto de entrada detectado: main.py

5. Ejecución

✅ El proyecto se ejecuta.

(el programa espera entrada — timeout normal)

Resumen

• ✅ Repositorio accesible
• ❌ report.md con ≥2000 palabras
• ✅ Archivos Python
• ❌ Configuración de dependencias
• ✅ Punto de entrada
• ✅ Ejecución

4/6 verificaciones pasadas.

---

Verificación automática — los resultados son orientativos. El profesor revisará en detalle.

[apiad]

🔍 Revisión de Código — Claude Code

Repositorio: https://github.com/Unit0code/GoEvent

Qué hace tu proyecto

GoEvent gestiona la agenda de una base de transporte: agendas eventos (12 tipos, de viajes a las provincias a mantenimiento y vacaciones), y el programa valida horarios, colisiones de tiempo, restricciones particulares y de exclusión mutua, y requisitos mínimos de recursos. Además modelaste un sistema de desgaste: los vehículos gastan usos y las personas gastan energía, y los eventos "curativos" los restauran. Todo con cuentas protegidas por contraseña y persistencia en JSON. Es un dominio ambicioso y bien pensado para primer año.

Ejecución

Lo corrí de verdad en un entorno aislado con Python 3.12 (usa solo la biblioteca estándar, así que no hizo falta instalar nada). Recorrí todos los menús con entradas simuladas y también probé la lógica de negocio directamente. Todo funcionó, sin ningún Traceback no controlado. Un detalle: en Agregar.py:125 usas un f-string con comillas simples anidadas, que solo es válido en Python 3.12 o superior — en 3.11 o anterior el archivo ni siquiera compilaría. Bajo 3.12 todo corre limpio.

Lo que está bien

• El sistema de reglas funciona de extremo a extremo. Verifiqué: agregar un "Viaje a la Habana" con Conductor+Vehículo+Guía se agenda ✅; sin las categorías necesarias lo rechaza correctamente; una fecha pasada, una hora muy temprana y una fecha con formato inválido todas dan mensajes claros y re-piden; y el par prohibido Jose+Marlon fue bloqueado. Muy completo.
• El desgaste de recursos funciona. Al expirar un evento, comprobé que Transtur1 bajó de 5 a 4 usos y Juan de 100 a 80 de energía, y el evento se eliminó de la agenda. Es la parte más difícil y la resolviste.
• "Buscar hueco" funciona de verdad. Con dos eventos ocupando Camión1, calculó y fusionó los intervalos y me sugirió correctamente "antes de las 06:40 o después de las 23:40". La fusión recursiva de intervalos está bien.
• Buena arquitectura: 10 módulos separados por responsabilidad, herencia bien usada en los eventos, despacho por diccionario (número→clase) en vez de cadenas de if, y serialización JSON con round-trip completo. Poco común a este nivel.
• Validación de entradas robusta: tu try_option maneja letras, números fuera de rango y negativos sin romperse.

Qué mejorar

• No te tragues los errores con try/except Exception: pass (en main.py, opciones 1, 2 y 5). Si algo falla dentro de "agregar evento", el usuario vuelve al menú sin saber qué pasó, y tú te quedas sin ver el bug. Captura excepciones específicas (p. ej. ValueError) o al menos imprime el error. Esto te ahorrará horas de depuración.
• Renombra los métodos __dict__() (en user.py, Recursos.py, Events.py) a algo como to_dict(). __dict__ ya es un atributo especial de Python (el diccionario interno de la instancia); redefinirlo como método funciona por casualidad porque siempre lo llamas explícitamente, pero es una colisión conceptual que confunde.
• Un typo con consecuencia: en Events.py:29 la restricción del viaje a la Habana dice 'Menedez', pero el recurso se llama 'Menendez' (Recursos.py:52). Por esa letra que falta, esa restricción nunca se activa. Un par que debería estar prohibido puede viajar junto.
• Detalles menores: en main.py:53 escribiste miscelaneo.clean sin (), así que ahí no limpia la pantalla; y events: list = [] como parámetro por defecto en user.py:7 es una trampa clásica de Python (mejor events=None y crear la lista dentro). Tu propio dividir_lista_str es exactamente ', '.join(map(str, lista)) — buen instinto reimplementarlo, esa es la versión idiomática.

Sobre el informe

Traes un Readme.md (por eso la verificación automática no encontró report.md). Es corto (~525 palabras) pero está bien escrito y, lo importante, coincide con el código: cada feature que describe existe y funciona. De hecho lo subestima — ni menciona "Buscar hueco", que sí implementaste. Dos detalles: pones "Python version 3.17" (esa versión no existe; tu código necesita 3.12+), y las rutas de las imágenes usan \ de Windows, así que no se ven en GitHub. Nada grave.

Muy buen trabajo. Modelaste un dominio complejo, lo hiciste funcionar completo y lo organizaste con criterio. Sigue así. 🚌