Writing to EEPROM using whitespace instead of spare zeros #1
Conversation
| Serial.println("Programming EEPROM:"); | ||
|
|
||
| // Go over all inputs | ||
| for (byte i = 0; i < 256; i++) { |
There was a problem hiding this comment.
BUG: The comparison of the i variable with 256 is always true due to limited range of data type.
| writeEEPROM(i + 512, (d2 == 0) ? 0 : digits[d2]); // write the digit or clear if 0 | ||
| writeEEPROM(i + 256, (d2 == 0 && d3 == 0) ? 0 : digits[d3]); // write the digit or clear if d3 and d2 are 0 | ||
| writeEEPROM(i, digits[i % 10]); // write the digit even 0 | ||
| } |
There was a problem hiding this comment.
It's easier to show powers of two, when using hex notation, like this:
writeEEPROM(i + 0x300, blank); // clear first 7segment display
writeEEPROM(i + 0x200, (d2 == 0) ? blank : digits[d2]); // write the digit or clear if 0
writeEEPROM(i + 0x100, (d2 == 0 && d3 == 0) ? blank : digits[d3]); // write the digit or clear if d3 and d2 are 0
writeEEPROM(i + 0x000, digits[i % 10]); // write the digit even 0
| void programmEEPROM() { | ||
| byte digits[] = { 0x7e, 0x30, 0x6d, 0x79, 0x33, 0x5b, 0x5f, 0x70, 0x7f, 0x7b }; | ||
| byte minusSign = 0x01; | ||
|
|
There was a problem hiding this comment.
let's define a white space for the display
byte blank = 0x00;
Then the code is more readable, where we compare d2 == 0, write blank.
| Serial.println("%"); | ||
| } | ||
|
|
||
| // Non two's complement part |
There was a problem hiding this comment.
How about labeling this as "Unsigned byte part"?
I've implemented a better-looking version for the EEPROM programmer using whitespace. So instead of "-009" it should be " -9". " 023" should now be " 23" and so on. Couldn't test it tho.