Skip to content

Fix: Corrección del comportamiento del decimal después del signo menos#55

Merged
alexricardotapiacarita-ai merged 4 commits intodevfrom
fix/decimal-after-minus
Dec 1, 2025
Merged

Fix: Corrección del comportamiento del decimal después del signo menos#55
alexricardotapiacarita-ai merged 4 commits intodevfrom
fix/decimal-after-minus

Conversation

@alexricardotapiacarita-ai
Copy link
Contributor

📋 Descripción

Este Pull Request corrige el error donde la calculadora permitía ingresar un punto decimal de forma incorrecta después de utilizar el signo menos (-).
Ahora la lógica valida correctamente la secuencia para evitar resultados inesperados y mejorar la estabilidad de la expresión numérica.

Se ajustó la función encargada de manejar la construcción del número y se añadieron validaciones adicionales.


🔗 Issue Relacionado

Related to #49


🎯 Tipo de Cambio

  • 🐛 Bug fix (corrección de error)
  • ✨ Nueva funcionalidad
  • 📝 Mejora de documentación
  • ♻️ Refactoring (sin cambios de funcionalidad)
  • 🧪 Tests (agregar o mejorar tests)
  • 🎨 Estilo
  • 🚀 Release

🧪 ¿Cómo se ha probado?

  • Tests unitarios
  • Prueba manual
  • Probado en diferentes sistemas operativos

Casos verificados:

  • Evitar que se inserte un decimal inmediatamente después de un signo menos cuando no corresponde.
  • Validación correcta de nuevas expresiones negativas.
  • Compatibilidad con inputs previos y operaciones estándar.

✅ Checklist

  • Mi código sigue las convenciones del proyecto
  • He realizado self-review de mi código
  • He comentado mi código en áreas complejas
  • He actualizado la documentación correspondiente (si aplica)
  • Mis cambios no generan nuevas advertencias
  • He agregado tests que prueban mis cambios
  • Los tests nuevos y existentes pasan localmente
  • Los commits siguen Conventional Commits

📝 Notas Adicionales

  • Este fix ayuda a prevenir malas interpretaciones en operaciones negativas.
  • Se deja base preparada para futuros ajustes en validaciones de expresiones.

👥 Revisores Sugeridos

@Jhos3ph @Jandres25

@alexricardotapiacarita-ai alexricardotapiacarita-ai added mejora Mejora a funciones existentes funcionalidad Nuevas funcionalidades urgente Requiere atención inmediata gui Funcionalidades de la interfaz gráfica duplicate Issue o PR duplicado labels Dec 1, 2025
@Jandres25 Jandres25 removed the duplicate Issue o PR duplicado label Dec 1, 2025
@Jandres25 Jandres25 added this to the v2.1.0 milestone Dec 1, 2025
Copy link
Contributor

@Jandres25 Jandres25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahora si está chido

@alexricardotapiacarita-ai alexricardotapiacarita-ai added the error Bug o fallo en el código label Dec 1, 2025
@alexricardotapiacarita-ai alexricardotapiacarita-ai merged commit f399f3f into dev Dec 1, 2025
2 checks passed
@alexricardotapiacarita-ai alexricardotapiacarita-ai deleted the fix/decimal-after-minus branch December 1, 2025 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

error Bug o fallo en el código funcionalidad Nuevas funcionalidades gui Funcionalidades de la interfaz gráfica mejora Mejora a funciones existentes urgente Requiere atención inmediata

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error: Manejo confuso de números decimales negativos (secuencia "- . 3")

2 participants