klanderfri / klanderfri/CardReaderLibrary

Put configuration values together

Offen
#19 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
fix
Vorherrschende Sprache
C++
Sterne
10
Forks
3
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The program has literals that are of a configurative nature. Regard the code below:
`do {`
` threshold(workOriginal, binaryImage, binaryThreshold, 255, THRESH_BINARY);`
` binaryThreshold -= 20;`
` } while (!ImageHelper::IsBlackTextWhiteBackground(binaryImage) && binaryThreshold > 0);`

The 255 and the 0 are literals, and should be so. The 20 however might need to be changed in the future since it is a literal that might need to be configured.

- [ ] Compile all configurative values in one single class.

In the future it might be useful to load these values from an external file which the user can edit (if needed).

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Durchsuche die C++-Quellen nach binaryThreshold, dem Threshold-Aufruf und anderen konfigurierbaren Literalen. Beginne mit dem gezeigten Ablauf von ImageHelper::IsBlackTextWhiteBackground und ermittle dann, wo die Konfigurationsklasse gemeinsam verwendet werden sollte. Die Aufgabe ist erledigt, wenn die konfigurierbaren Werte in einer Klasse gesammelt sind, ohne absichtlich verwendete Literale wie 255 und 0 zu ersetzen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp, opencv
Bereich
computer-vision
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.