tylernt:
Either you write 50 case statements (with the code inline, not calling 50 different functions) or you write 50 functions. Either way it's a lot of typing and a lot of scrolling, they seem to be about equal from a finger-cramp standpoint.Now, if some or all of those 50 functions could be used for other additional things, outside of and aside from parsing your serial data, then I agree it makes more sense to have them as functions and function pointers instead of inlining them in the switch/case block. You can take advantage of code re-use that way.
Perhaps those 50 functions represent various states in a state machine, and sending the serial command isn't the only way to call them? Maybe the functions are included from somewhere else, and aren't something you wrote yourself?
That's the detriment of simple examples: assumptions have to be made about the reasons for other, less relevant portions of the code. Since you're more interested in picking apart the examples than looking at the main points, I won't bother with anymore of them.