klanderfri / klanderfri/CardReaderLibrary
Put configuration values together
- 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
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