[Proyecto I] Reservation Manager

[lacuentadeadrian57-netizen]

Nombre: Adrian Martinez Borrego
Grupo: C121
Repositorio: https://github.com/lacuentadeadrian57-netizen/Reservation-Manager

Descripción

Una aplicación de manejo de reservas que plantea una interfaz donde el usuario puede controlar sus locales, recursos y actividades a lo largo de un periodo de tiempo

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/lacuentadeadrian57-netizen/Reservation-Manager

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

✅ report.md — 2,028 palabras (mínimo: 2,000).

3. Proyecto Python

⚠️ Sin pyproject.toml.
✅ requirements.txt encontrado.
Dependencias declaradas: streamlit, pandas

Archivos Python (3, 547 líneas en total):

• manager.py
• loader.py
• main.py

4. Punto de Entrada

✅ Punto de entrada detectado: main.py

5. Ejecución

✅ El proyecto se ejecuta.

2026-06-26 14:53:53.089 WARNING streamlit.runtime.scriptrunner_utils.script_run_context: Thread 'MainThread': missing ScriptRunContext! This warning can be ignored when running in bare mode.
2026-06-26 14:53:53.089 WARNING streamlit.runtime.state.session_state_proxy: Session state does not function when running a script without `streamlit run`
2026-06-26 14:53:53.090 WARNING streamlit.runtime.scri

Resumen

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

6/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/lacuentadeadrian57-netizen/Reservation-Manager

Qué hace tu proyecto

Construiste un gestor de reservas en Streamlit donde el usuario administra locales, recursos y reservas a lo largo del tiempo, con cálculo automático de precio y persistencia en JSON. Lo más valioso —y lo más difícil del enunciado— es tu motor de resolución de colisiones: cuando una reserva nueva se solapa con otra en el mismo local, o compite por un recurso escaso con reservas en otros locales, tu programa la reprograma automáticamente al primer hueco libre en vez de rechazarla. Eso está muy bien pensado.

Ejecución

Corrí tu lógica de negocio directamente sobre save.json (la clase Manager está limpiamente separada de la GUI, lo que me permitió testearla en aislamiento) y también arranqué el servidor Streamlit en modo headless: arrancó sin errores. Todos los módulos compilan (py_compile OK; necesita Python 3.12 por las f-strings con comillas anidadas). Verifiqué reservas válidas, cálculo de precio, todas las validaciones de entrada, y las dos formas de reprogramación por colisión. Todo eso funciona.

Lo que está bien

• Separación de capas ejemplar: manager.py (lógica), main.py (GUI), loader.py (persistencia). Que la lógica no dependa de Streamlit es una decisión de diseño excelente para tu nivel.
• Cálculo de precio correcto: para una reserva de Conference Room + Whiteboard por 3 días me dio exactamente 1185 = (200 + 30 + 50 + 100 + 15) × 3.
• Todas las validaciones funcionan: inicio > fin, fecha en el pasado, opcional inexistente, y pares de recursos que se excluyen (Mirrors/Lighting controls) — todas rechazadas con mensajes claros.
• La reprogramación por colisión funciona de verdad, incluso el caso difícil: dos locales distintos peleando por el único Ribbon Microphone se reprograman sin solaparse.
• Escritura atómica a disco con archivo temporal + os.replace (loader.py): un patrón sólido y poco común a primer año.

Qué mejorar

• Bug en update_quantity (manager.py:192-196): primero asignas self.quantity[name] = value y después comparas if value < self.quantity[name], que ya siempre es False — así que refresh() nunca se llama al bajar una cantidad. Compara antes de asignar (guarda el valor viejo en una variable). Lo comprobé: al bajar Projector de 5 a 2, no se revalidan las colisiones.
• Bug en delete_resource (manager.py:130-152): al borrar un recurso que es requisito de varios locales, tu código también borra esos locales. Lo verifiqué: al borrar 'Projector' me quedó solo 'Rehearsal Room'. Borrar un recurso no debería eliminar los locales que lo usan; revisa la rama del del self.requisites[location].
• add_resource con una exclusión que no existe lanza KeyError (manager.py:125). No pasa desde la interfaz (las exclusiones vienen de un multiselect), pero conviene validar que el recurso exista antes de tocarlo.
• except: desnudo en loader.py: atrapa absolutamente todo (incluido Ctrl+C). Usa except OSError o except Exception.
• El método refresh funciona pero es denso y sin comentarios — justo ahí viven los dos bugs sin ser evidentes. Partirlo en funciones pequeñas con nombres explicativos te facilitaría encontrarlos.

Sobre el informe

Tres cosas concretas:

1. Está duplicado: las líneas 1-179 se repiten idénticas en 180-359. Así llegó a ~2000 palabras, pero el contenido único es la mitad — probablemente un error al pegar. Bórralo.
2. Mencionas "horarios de apertura" como funcionalidad, pero el código no implementa horarios en ningún lado.
3. El ejemplo de colisión usa horas (10:00-12:00), pero tu sistema trabaja a granularidad de día (date, no datetime). El informe sobreestima la resolución temporal real.

Lo demás describe bien la reprogramación por colisiones y capacidad, que efectivamente están implementadas.

Buen trabajo con lo esencial — el corazón del proyecto es difícil y lo resolviste. Enfócate en testear las operaciones de edición (borrado y cambio de cantidad) con casos límite, y en alinear el informe con lo que el código realmente hace.