[Proyecto I]

[Orlih226]

Nombre: Orlando Valdes Montejo
Grupo: C121
Repositorio: https://github.com/Orlih226/Proyecto_25-26_Archivos_De_Hallowneest

Descripción

Proyecto dedicado a la gestion de eventos y recursos a lo largo del tiempo

(Añadir una breve descripción de su proyecto).

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/Orlih226/Proyecto_25-26_Archivos_De_Hallowneest

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

⚠️ report.md encontrado pero con solo 1,523 palabras (mínimo: 2,000).

3. Proyecto Python

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

Archivos Python (4, 1,118 líneas en total):

• Main.py
• TrabajoConJson.py
• Trabajar.py
• TrabajoConDateTime.py

4. Punto de Entrada

✅ Punto de entrada detectado: Main.py

5. Ejecución

✅ El proyecto se ejecuta.

Traceback (most recent call last):
  File "/tmp/tmpk2zgry2b/repo/Main.py", line 1, in <module>
    import customtkinter as tk
ModuleNotFoundError: No module named 'customtkinter'

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.

[Orlih226]

Hice un nuevo commit, añadí más palabras y arregle requirements que estaba mal escrito

[apiad]

🔍 Revisión de Código — Claude Code

Repositorio: https://github.com/Orlih226/Proyecto_25-26_Archivos_De_Hallowneest

Qué hace tu proyecto

Construiste una aplicación de escritorio (con customtkinter y tkcalendar) para planificar eventos de un reino ambientado en Hollow Knight: cada evento consume recursos —personajes, lugares— en una franja horaria, y tu programa detecta cuándo dos eventos chocan y reajusta automáticamente los horarios para resolver la colisión. Tiene un flujo de cuatro interfaces (menú con calendario, creación, resolución de colisiones con un gráfico de pastel dibujado a mano, e inscripción selectiva con búsqueda binaria). Es un proyecto ambicioso y claramente por encima de la media de primer año.

Ejecución

Preparé un entorno aislado con Python 3.12 e instalé tus dependencias desde requirements.txt sin problemas. La GUI no pude recorrerla: al lanzar python Main.py el servidor X del entorno de evaluación abortó con un fallo de XCB/threading (XInitThreads, xcb_io.c). Es importante que sepas que esto no es un bug tuyo — es un conflicto del entorno donde te evalué (no había un display libre ni Xvfb disponible), y el traceback es 100% de la capa gráfica del sistema, nunca de tu código.

Como tu motor de validación y colisión es Python puro e independiente de la interfaz, lo ejecuté directamente alimentándolo con eventos en el mismo formato que produce tu GUI. Pasó todos los casos que probé: carga de Inventario.json y EventosEjec.json, detección de solapamiento horario (12-14 vs 13-15 colisiona; 12-13 vs 14-15 no), inserción de eventos válidos en la estructura anidada año→mes, reprogramación recursiva de un evento que colisionaba (12:30-13:30 → 13:00-14:00, justo tras el conflicto), expansión de una lista de días [1,2,3] en tres eventos, y una hora malformada 99:99 que lanza ValueError pero que tu try/except (Main.py:678-694) captura sin tumbar la app. El motor hace exactamente lo que promete el issue.

Lo que está bien

• Buena arquitectura para primer año: separaste responsabilidades en módulos claros — TrabajoConJson.py (E/S), TrabajoConDateTime.py (tiempo), Trabajar.py (motor de colisión), Main.py (GUI). Ese instinto de separar la lógica de la interfaz es justo lo que te permitió que yo pudiera probar el motor sin la GUI.
• Estructura de datos bien pensada y justificada: el diccionario anidado año → mes → [eventos] te da acceso directo a un mes sin recorrer toda la base. Y explicas el porqué en el informe. Es una decisión de diseño madura.
• El motor de colisión funciona de verdad, incluida la reprogramación recursiva. Verifiqué que reajusta horarios y no rompe con entradas malformadas.
• Usaste copy.deepcopy para no mutar la base "fantasma" (Main.py:432): eso muestra que entendiste el problema de aliasing de listas, un tropiezo clásico que a muchos se les escapa.
• Validación de entradas presente y razonable: horas malas, meses > 12, días fuera del rango real del mes (ValidarFecha con calendar.monthrange), años < 2025 — todo genera mensajes en vez de reventar.

Qué mejorar

• Indentación inconsistente (mezclas 1, 2 y 3 espacios en casi todos los archivos). Funciona, pero cuesta leerlo. Pasa un formateador como black una sola vez y queda todo uniforme sin que muevas una coma a mano.
• Interfaz() es enorme (~220 líneas montando las 4 interfaces de un tirón, Main.py:14). Divídelo en un método por interfaz: será más fácil de leer y de arreglar cuando algo falle en una sola pantalla.
• Repetición masiva de widgets: los opcion1..opcion22 con sus lambda (Main.py:116-166) y los 7 checkboxes de tipo son casi idénticos. Podrías generarlos con un bucle sobre una lista de nombres — menos código y menos sitios donde equivocarte.
• Evita eval: en inscribir_seleccionados usas eval(text[1]) (Main.py:852). Aquí el texto lo genera tu propio programa, así que no es peligroso hoy, pero es un hábito a erradicar; casi siempre hay una alternativa segura (int(...), json.loads, etc.).
• Los métodos de Trabajar.py (Entrada, Validar, Comparador…) están sin self ni decorador: funcionan porque los llamas como RequisitosDeEventos.X(...), pero lo correcto es marcarlos @staticmethod.
• except: desnudos (Main.py:319,495,573) silencian cualquier error, incluso los que querrías ver. Sé específico: except (KeyError, ValueError):.
• Detalle pequeño: Dias() (Main.py:468) y Meses() (Main.py:479) calculan una lista filtrada y luego retornan la original sin filtrar — código muerto que parece un filtrado que quedó a medio conectar. Vale la pena revisarlo.

Sobre el informe

Tu report.md es de los mejores en explicar el porqué de tus decisiones: el formato del evento como lista, la base anidada, la copia "fantasma", el problema de aliasing, la migración de tkinter a customtkinter, e incluso la pérdida de código por pull mal hechos. No exagera nada — todo lo que afirma se sostiene en el código que verifiqué. La sección de "Experiencias" es especialmente valiosa como reflexión de aprendizaje. La única discrepancia es trivial: la clave del JSON es "Eventos en ejecucion" (sin acento) mientras el informe la escribe con mayúscula; el código usa la correcta.

Trabajo sólido y ambicioso. Enfócate en la legibilidad —indentación, dividir Interfaz(), quitar eval y los except desnudos— y darás un salto grande sin tocar la lógica, que ya funciona. Buen proyecto.