[Proyecto I]Rubi

[RubixxxPdA]

**Nombre:**Rubi Perez de Alejo Lorenzo
**Grupo:**c-121
**Repositorio:**https://github.com/RubixxxPdA/proyectopro-2526/tree/main

Descripción

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

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/RubixxxPdA/proyectopro-2526

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

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

3. Proyecto Python

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

Archivos Python (2, 1,014 líneas en total):

• core.py
• pythonmain.py

4. Punto de Entrada

✅ Punto de entrada detectado: core.py

5. Ejecución

✅ El proyecto se ejecuta.

(sin salida)

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/RubixxxPdA/proyectopro-2526

Qué hace tu proyecto

Construiste un sistema de agendamiento por consola para un salón de uñas ("RR Nails") con dos roles: administrador (con contraseña) que ve la agenda, gestiona el inventario y agrega servicios; y cliente que agenda y elimina citas. Lo más valioso es el motor de disponibilidad: verifica horario laboral, cuenta manicuristas libres, distingue recursos gastables de no gastables, y cuando el hueco pedido no sirve, busca automáticamente el próximo espacio libre entre citas o al día siguiente. Está bien pensado y funciona.

Ejecución

Lo corrí de verdad con Python 3.12, alimentando el menú con entradas simuladas. Agendé una cita válida (marzo 15, 10:00) → se creó y guardó correctamente, y la agenda quedó ordenada por fecha. Pedí un horario fuera de rango (18:00) → me propuso "21 de marzo a las 09:00" y agendó al aceptar. Eliminé una cita → se borró y el cliente desapareció al llegar su contador a 0. Agregué un servicio y un recurso → ambos se guardaron bien. También le lancé entradas basura (mes "february", febrero 30, hora "25:99", servicio "99" y "abc") y ninguna la hizo reventar: todas dieron un mensaje de error controlado. Muy buen trabajo defensivo.

Lo que está bien

• Separaste lógica y presentación: core.py con el negocio y pythonmain.py con el menú. Eso es una decisión de diseño madura para primer año.
• El motor de agendamiento funciona de verdad: solapamiento de horarios, gestión de manicuristas, búsqueda recursiva del siguiente hueco. Verificado ejecutando.
• Manejo de entradas inválidas excelente: mes, día, hora y número de servicio malos, todos capturados sin Traceback.
• Modelo de datos JSON coherente entre código, recursos.json y agenda.json, con acentos preservados (ensure_ascii=False).
• Funciones pequeñas, con nombres claros en español y docstrings.

Qué mejorar

• La opción "Salir" (3) no funciona. En pythonmain.py:69 escribiste elif opcion == 3:, pero opcion es un texto (viene de input()), no un número. Compararlo con el entero 3 nunca es verdadero, así que "Salir" no guarda ni cierra: solo vuelve a mostrar el menú. Cámbialo a elif opcion == "3": (con comillas), como en las otras opciones. Es el detalle más importante a arreglar.
• El consumo de inventario no se guarda. Cuando agendas, sí restas los recursos gastables en memoria (consumir_recursos_gastables), pero luego llamas a guardar_datos(), que solo escribe agenda.json. El archivo recursos.json nunca se actualiza porque falta llamar a guardar_recursos(). Lo comprobé: después de agendar, el stock de "esmalte regular" seguía en 20 y recursos.json quedó idéntico. Añade una llamada a guardar_recursos() después de agendar y el inventario bajará de verdad entre ejecuciones.
• Tienes una función completa sin conectar. ver_detalles_cita (core.py:605) está bien escrita pero no aparece en ningún menú de pythonmain.py, así que nadie puede usarla. O la enganchas en el menú de cliente, o la quitas del informe (que la anuncia como funcionalidad activa).
• Detalle menor: el except: desnudo en ingresar_hora (core.py:258) mejor como except ValueError:, para no capturar por accidente cosas como Ctrl+C.

Sobre el informe

El informe es completo y describe bien el modelo de datos y los algoritmos. Tres cosas no coinciden con el código: dice que el punto de entrada es mainrr_corregido.py (en realidad es pythonmain.py, y ese archivo no existe); lista "Ver detalles de cita" y "Ver recursos por servicio" como funcionalidades activas, pero la primera está desconectada y la segunda no está implementada; y describe el consumo de inventario como efectivo, cuando en realidad no se persiste (por el punto anterior). Son cabos sueltos entre lo que quisiste hacer y lo que quedó cableado, no exageraciones graves. Ajustar esas tres líneas dejaría el informe fiel al código.

Muy buen proyecto, Rubi — el núcleo es sólido y ejecuta. Arregla la comparación de "Salir", persiste el inventario y conecta (o retira) la función de detalles, y queda redondo.