Sehe das als kritische Würdigung und MEINE persönliche Meinung.
Andere können das anders sehen:
1a) ich kann ich mit deinem t_ prefix nichts anfangen. Du versteckst da etwas in die Variablennamensgebung hinein. "Ungarische Notation" oder ähnliches macht man meiner Meinung schon lange nicht mehr.
1b) den Underscore verwende ich zur Worttrennung wenn ich WAS_IN_GROSSBUCHSTABEN habe, z.B. Makros. ansonsten camelCase, SnakeCase
1c) t_func vs func ... da habe ich viel spass daran im code zwei ähnlich lautende Variablen zu verwenden
1d) t_Action ... ist ein "Name" - verwende ich dann als type alias/Signatur. Ich würde das wie Typen mit großem Buchstaben beginnen lassen.
Using-declaration - cppreference.com
Type alias, alias template (since C++11) - cppreference.com
technisch hältst du dort einen Functionpointer, hier für ein Callback, wird auch gern mit CB abgekürzt...
-
wenn ich einen Setter zum "Aktivieren" und "Deaktiveren" brauche, dann nenn ich ihn auch activate, deactivate, setActive, setInactive ...
ich mache mir eine Funktion die auf Grund einen Parameters aktiviert und deaktiviert. Meine setActive/setInactive ruft dann diese Funktion mit dem richtigen Parameter auf.
default parameter bool val = false könnte man auch setzen. -
activTimer, onTimer, setTimer, ticTimer ... generell, wozu den (fast) Klassennamen noch mal bei der function wiederholen.
Macht aus meiner Sicht keinen Sinn.onTimer ... würde bei mir eher richtung setCallback heißen oder setTimeoutCB,... also was soll passieren wenn der Timer feuert.
setTimer ... wäre bei mir ganz klar ein setter, vieleicht setInterval (t_msec ist ja sowieso auch nicht sprechend - gehts um die einheit oder was der Wert representiert)
ticTimer ... wäre bei mir (und in vieeelen) Arduino Libraries ein update() evtl. ein run(), ja die OneButton lib hat auch z.B. ein tick() aber kein tickOneButton. -
BUM: wenn ich vergesse den callback im setup zu setzen kann das grauslich bum machen. ich würde das (und habe das) mit einem if abgefangen
dein onTimer wenn ich den mal alleine lese:
verstehe fast gar nicht was mit der Fahrradkette - sorry func passieren soll-
@MicroBahner hat sich auch schon zu den führenden underscores geäußert. Ich sehe trailing underscores ähnlich unelegant
In deinem ersten Sketch hattest du ja eh schon den Zugriff auf gleichlautende member mit this-> behalte das - das ist gut!