Flow-Launcher/Flow.Launcher

BUG: Calculator can't multiply numbers with more than 19 digits

Open

#2 673 ouverte le 25 avr. 2024

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C# (598 forks)batch import
bughelp wantedkeep-fresh

Métriques du dépôt

Stars
 (14 678 stars)
Métriques de merge PR
 (Merge moyen 17j 12h) (23 PRs mergées en 30 j)

Description

Checks

  • I have checked that this issue has not already been reported.

  • I am using the latest version of Flow Launcher.

Problem Description

Calculator is providing wrong result if one of the two numbers being multiplied is more than 19 digits. Examples:

  1. 1.2*0.83333333333333333333
  2. 2453*23534245034253633353
  3. 1*0.22222222222222222222
  4. 1*24353532424240053022
  5. 1*1111111111111111111111
  6. 1324*30000000000000000000

It's easiest to see this issue in action if you copy and paste the above examples and then remove just the last digit. We will notice a large change in the result.

But there are some instances where this doesn't happen. I am unable to determine the logic behind when this issue occurs and when it does not. Examples:

  1. 1*0.11111111111111111111
  2. 1324*10000000000000000000
  3. 1324*40000000000000000000

To Reproduce

No response

Screenshots

No response

Flow Launcher Version

1.18.0

Windows Build Number

10.0.22631.3447

Error Log

No response

Guide contributeur

Direction de recherche
Examinez le code du plugin de la calculatrice pour déterminer comment il gère les opérations arithmétiques, en particulier la multiplication. Recherchez l'utilisation de types double ou float qui peuvent perdre en précision pour les nombres à plusieurs chiffres. Considérez si le plugin devrait utiliser decimal ou BigInteger pour l'arithmétique de précision arbitraire.
Stack technique
csharp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Understanding of numeric precision
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65