In many cases, yes, you are right. But in this case, at least in my opinion, the usage is quite clear. The switch could either be on (1) or off(0) or we don't know yet (-1 = number that could never happen otherwise). And i even bothered to document it:
my $lastswitch = -1;
The whole $lastswitch thing could just be set to 0 on startup in theory (guessing the button isn't pressed on startup). Or i could just replicate the Device::BCM2835::gpio_lev($testpinnumber) call at the beginning.
I just kept this one-off script for my one-off device to my usual style guide of 15+ years of Perl coding. If i see that construct, i instantly know i'm dealing with an uninitialized boolean. Same goes for other "magic" numbers. 3600 is seconds per hour, 86400 is seconds per day and 365.25 is (average) days per year.
Those kinds of magic numbers i wouldn't call "bad", it's more of a grey area.
I can completely understand your point of view, and in most cases i'd support you all the way, but i hereby invoke my right of TIMTOWTDI 😉
|Replies are listed 'Best First'.|
Re^3: Implementing a 2-mode "Audiobutton" on the Raspberry Pi
by hippo (Chancellor) on Oct 29, 2018 at 16:08 UTC