microsoft/PowerToys

Powertoys run calculator weird behavior with space-separated thousands

Open

#34.145 geöffnet am 4. Aug. 2024

Auf GitHub ansehen
 (7 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C# (7.978 Forks)batch import
Cost-SmallHelp WantedIdea-EnhancementIssue-BugProduct-PowerToys RunRun-Plugin

Repository-Metriken

Stars
 (133.154 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 14T 20h) (117 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie, wie PowerToys Run Ausdrücke mit Leerzeichen parst. Das Problem liegt wahrscheinlich in der Rechnerlogik, die Leerzeichen als implizite Multiplikation behandelt. Zur Behebung sollten Leerzeichen zwischen Ziffern ignoriert oder als Tausendertrennzeichen behandelt werden. Suchen Sie nach dem Code zur Ausdrucksanalyse, wahrscheinlich im Berechnungsmodul von PowerToys Run. Erwägen Sie, eine Prüfung auf Leerzeichen innerhalb von Zahlen hinzuzufügen.
Tech Stack
csharp
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65