{"id":2392,"date":"2018-10-30T14:16:11","date_gmt":"2018-10-30T13:16:11","guid":{"rendered":"https:\/\/www.entropywins.wtf\/blog\/?p=2392"},"modified":"2018-10-30T14:16:11","modified_gmt":"2018-10-30T13:16:11","slug":"readable-functions-do-one-thing","status":"publish","type":"post","link":"https:\/\/www.entropywins.wtf\/blog\/2018\/10\/30\/readable-functions-do-one-thing\/","title":{"rendered":"Readable Functions: Do One Thing"},"content":{"rendered":"

Several tricks and heuristics that I apply to write easy to understand functions keep coming up when I look at other people their code. This post outlines the second key principle. The first principle is Minimize State<\/a>. Following posts will contain specific tips and tricks, often building on these two principles.<\/p>\n

Do One Thing<\/h3>\n

Often functions become less readable because they are doing multiple things. If your function Foo needs to do tasks A, B and C, then create a function for each of the tasks and call those from Foo.<\/p>\n

Having small functions is OK<\/strong>. Having just 3 calls to other functions in your function is OK. Don’t be afraid of “too simple” code or of this style making things harder to follow (it does not (unless you are splitting things up stupidly)). And don’t be afraid of performance. In most programs the amount of function calls have effectively 0 impact on performance. There are exceptions, though unless you know you are dealing with one of these, don’t mash things together for performance reasons.<\/p>\n

One Level of Abstraction<\/h3>\n

Doing one thing includes dealing with a single level of abstraction. For instance, suppose you have a function in which in some cases a message needs to be logged to the file system.<\/p>\n

function doThing() {\n    if (condition) {\n        low_level_write_api_call('write-mode', $this->somePath, 'Some error message');\n    }\n}<\/pre>\n
Here the details of how the error message is logged are on a lower level of abstraction than the rest of the function. This makes it harder to tell what the function is actually doing, because details that don’t matter on the higher level of abstraction clutter the high level logic. These details should be in their own function.<\/p>\n
function doThing() {\n    if (condition) {\n        $this->log('Some error message');\n    }\n}\n\n\nprivate function log(string $message) {\n    low_level_write_api_call('write-mode', $this->somePath, $message);\n}<\/pre>\n
See also<\/h3>\n