geekcomputers / geekcomputers/Python
Term with mixed case NOT Captured
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 35.4k
- Forks
- 12.9k
- Ø Merge
- 2 Std. 37 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
If you have term = "Sin(20) + cOs(30)", term.replace(function, withmath) on LINE 50 above would not work except the whole term has been converted to lower case. Hence input validation is required.
Possible Solution:
Add term = term.lower() prior to the loop on LINE 47. Hence, this will remove the need for having .lower() on LINE 48's conditional statement.
@jwadel @Victory-Uchenna
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in calculator.py ungefähr bei den Zeilen 44–50, insbesondere bei der Schleife zur Verarbeitung der Terme und der Ersetzungsbedingung. Führe den calculator mit gemischtschreibenden Funktionsnamen wie Sin(20) + cOs(30) aus und bestätige, dass die Eingabe vor der Ersetzung normalisiert wird und das erwartete Ergebnis liefert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- cli
- Issue-Typ
- Bug
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 58/100