microsoft/PowerToys

Powertoys run calculator weird behavior with space-separated thousands

Open

#34 145 ouverte le 4 août 2024

Voir sur GitHub
 (7 commentaires) (1 réaction) (0 assignés)C# (7 978 forks)batch import
Cost-SmallHelp WantedIdea-EnhancementIssue-BugProduct-PowerToys RunRun-Plugin

Métriques du dépôt

Stars
 (133 154 stars)
Métriques de merge PR
 (Merge moyen 14j 20h) (117 PRs mergées en 30 j)

Description

Microsoft PowerToys version

0.83.0

Installation method

PowerToys auto-update

Running as admin

No

Area(s) with issue?

PowerToys Run

Steps to reproduce

  1. Open powertoys run
  2. Type in 100 500/100 - the result is incorrect (500 instead of 1005)
  3. If we type in 100500/100 (without the separator), we get the correct result - 1005.

I believe this is because the whitespace will be treated as implied multiplication, so the equation becomes 100(500/100) = 500

✔️ Expected Behavior

Whitespace can be used for thousands separator.

According to Wikipedia, space is the internationally recommended thousands separator.

❌ Actual Behavior

Doing calculations with large numbers formatted with whitespace as thousand separator give misleading results.

Other Software

No response

Guide contributeur

Direction de recherche
Examinez comment PowerToys Run analyse les expressions contenant des espaces. Le problème réside probablement dans la logique de la calculatrice qui traite les espaces comme une multiplication implicite. Pour corriger, les espaces entre les chiffres doivent être ignorés ou traités comme des séparateurs de milliers. Recherchez le code d'analyse des expressions, probablement dans le module de calcul de PowerToys Run. Envisagez d'ajouter une vérification pour les espaces dans les nombres.
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
GitC#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65