Extract duplicate isHex/parseHex uit main.cpp en client.cpp #19

New issue

Closed

opened 2026-08-04 18:35:54 +02:00 by eddy · 1 comment

eddy commented

2026-08-04 18:35:54 +02:00

Owner

Probleem:
main.cpp:201-213 en client.cpp:116-128 bevatten identieke isHex() en parseHex() functies.

main.cpp roept hex.toUpperCase() aan vóór isHex().
client.cpp doet dit ook, maar inconsistente documentatie.

Voorstel:

Maak src/serial_utils.h met gedeelde implementatie
Include in beide bestanden
Voeg unit tests toe voor edge cases (lowercase hex, lege string)

Impact:

Minder duplicatie
Consistent gedrag
Makkelijker te onderhouden

Labels: refactor, firmware, tech-debt
Prioriteit: laag

**Probleem:** `main.cpp:201-213` en `client.cpp:116-128` bevatten identieke `isHex()` en `parseHex()` functies. `main.cpp` roept `hex.toUpperCase()` aan vóór `isHex()`. `client.cpp` doet dit ook, maar inconsistente documentatie. **Voorstel:** 1. Maak `src/serial_utils.h` met gedeelde implementatie 2. Include in beide bestanden 3. Voeg unit tests toe voor edge cases (lowercase hex, lege string) **Impact:** - Minder duplicatie - Consistent gedrag - Makkelijker te onderhouden **Labels:** refactor, firmware, tech-debt **Prioriteit:** laag

eddy commented

2026-08-05 08:23:10 +02:00

Author

Owner

Opgelost in commit op main:

Nieuw: src/serial_utils.h — gedeelde static inline isHex() en parseHex()
main.cpp: duplicaten verwijderd (lijnen 214-226), #include "serial_utils.h" toegevoegd
client.cpp: duplicaten verwijderd (lijnen 129-141), #include "serial_utils.h" toegevoegd

Beide PlatformIO environments (s3lite-server, s3lite-client) compileren zonder errors.

Opgelost in commit op `main`: - **Nieuw:** `src/serial_utils.h` — gedeelde `static inline isHex()` en `parseHex()` - **`main.cpp`**: duplicaten verwijderd (lijnen 214-226), `#include "serial_utils.h"` toegevoegd - **`client.cpp`**: duplicaten verwijderd (lijnen 129-141), `#include "serial_utils.h"` toegevoegd Beide PlatformIO environments (`s3lite-server`, `s3lite-client`) compileren zonder errors.

eddy closed this issue

2026-08-05 08:23:14 +02:00

eddy referenced this issue from a commit

2026-08-05 08:23:37 +02:00

refactor: extract duplicate isHex/parseHex into shared serial_utils.h