Lessons from “Clean Code”

So if you want to go fast, if you want to get done quickly, if you want your code to be easy to write, make it easy to read. — Robert C. Martin

Usually, there are two types of programmers clever ones and professionals. Both of them can get the job done but there are a lot of differences between the two. Professionals understand that clarity is the king. They use their powers to write clean code that others can understand and work on it by writing clean code. But what exactly is clean code? What are its characteristics and how can one write clean code? These were the questions in my mind when I started reading Clean Code written by Robert C. Martin and gladly this book didn’t disappoint me at all. This book has so many takeaways for clean code and I think every programmer should read it especially at the start of their career.

In this article, we will learn

• variable names should reveal its intent and should not include data type

• class and function name should be a noun and verb respectively

• functions should do only one thing and number of arguments should be minimum

• comments should be avoided as much as possible

• parts of code linked together should be placed next to each other

• the code should be formatted in a way that the eyeballing should be minimum while reading it

All the code in this article is written in Python so if you are unfamiliar with the language it might be a bit abstract for you.

Characteristics of Clean Code?

Robert C. Martin aka Uncle Bob asked some very experienced and well-known programmers that what they thought about clean code. As per them here are a few of the characteristics of clean code.

I like my code to be elegant and efficient. Logic should be straightforward to make it hard for bugs to hide. Clean code does one thing well. — Bjarne Stroustrup(inventor of C++)

Clean code can be read, and enhanced by a developer other than its original author. It has unit and acceptance tests. It has meaningful names — Dave Thomas

Clean code is simple and direct. Clean code reads like well-written prose — *Grady Booch* (author of Object-Oriented Analysis and Design with Applications)

So, If I outline these characteristics, the clean code should be

• Elegant: It should be simple and easy to read. It should make you smile while reading it.

• Focused: Each function/module should expose a single-minded attitude. It should only focus on getting one thing done.

• Tests: If it doesn’t have tests then it’s not clean no matter how elegant/readable it is.

Writing Clean Code

Meaningful Name

If a name requires a comment, then the name does not reveal its intent. — Uncle Bob

Names are important as they reveal the purpose. Coming up with good and meaningful names takes time but believe me it’s worth it because it saves time we have to read/overview it later.

Variable Name

It should describe its nature without any comment and if it’s not, then the name should be changed. i.e.

A name should also be searchable, for that, longer names are preferred over shorter ones. Also, a single character name should not be used for a large scope. It should only be used in a shorter scope like a small function/loop.

Function & Class Name

A function name should be a verb and it should also reveal the action along with the value, like extract_age, delete_user_info. *On the other hand, a class name should not be a verb. It should be a noun or a noun phrase name. i.e. *Customer, Account and WeatherRecord.

Don’t make it hard for yourself by adding unnecessary context. Don’t prefix every class/function with context unless they represent a different context. For example in an application of “Green Pizza Hut”, it will be really bad to prefix every function/class with “gph” as there will be a lot of redundant characters. Shorter names are better as far as they are clear.

Cuteness

If you are cute, great that’s good for you but please don’t make the code cute make it rather professional. I know it sounds very cute to name a variable after your pet or favorite character’s name but how on earth one can know what is “Bubbles” or “Dexter”.

Don’t use slang words while naming your variables, functions or classes. For example, don’t use merk() to mean kill().

One Word per Concept

Pick one word for a concept and stick with it. For example, we should not use *get, retrieve, extract *and *fetch as a function name *for the same concept of different classes. It will be very difficult to remember which class has which function name. In the same way, it’s also confusing to have *Manager, Controller *in the codebase as both serve the same purpose.

Functions

Functions should do one thing. They should do it well. They should do it only.— Uncle Bob

Size does matter in functions, the first rule of a function is that it should be small and then it should be smaller than that.

Do One Thing

There is no advantage of writing a function if it’s doing more than one thing. As the main purpose of writing functions is to decompose a larger concept into a set of steps at the next level of abstraction. Functions should not be large enough to hold nested structures. Also, the indent level of a function should not be greater than one or two. It will make the function easier to read and understand.

Arguments

There should be no more than three arguments for a function. An ideal number of arguments should be zero. More arguments take more conceptual power. A lesser number of arguments make easier to write test cases.

Flag arguments are ugly. Passing a boolean into a function is a truly terrible practice. It immediately complicates the signature of the function, loudly proclaiming that this function does more than one thing. It does one thing if the flag is true and another if the flag is false.

Comments

Nothing can be quite so helpful as a well-placed comment. Nothing can be quite so damaging as an old crufty comment that propagates lies and misinformation. — Uncle Bob

Comments are always failures. We have them because we cannot always figure out how to express ourselves without them, but their use is not a cause for celebration. The purpose of a comment should not be to explain the intent. We write a module and we know it is confusing and disorganized. We know it’s a mess. So we say to ourselves, “Ooh, I’d better comment that!” No! You should better clean it!

There are two ways in the following code; a messy code with a comment to explain it and other is clean code explaining itself without any comment.

Unfortunately we, the programmers, don’t realistically maintain the comments. Requirement changes and so does the code, a chunk of it moves from here and there but the comment stays there and creates the mess and misunderstanding. There should be no commented code or position markers. It’s very rare when position markers make sense to gather certain function together but in general, they should be removed.

An example of position maker is given below.

*# Actions #######*

Good Comments

Not all comments are bad, sometimes it’s necessary to add comments. For example for legal reasons, it’s good to add comments for copyrights, authorship at the start of the file. It’s also reasonable to add a warning of consequences and “to do” comments.

Sometimes a comment goes beyond just useful information about the implementation and provides the reason for writing this code. For example, in the following example the developer expressing the reason for adding download delay.

Formatting

Code formatting is important. It’s too important to ignore and it’s important to treat religiously.

The source file should be like a newspaper article. The filename should be simple but meaningful enough to tell us whether we are in the right file or not just like a newspaper article’s title. As the start of an article gives an abstract of the article and details increases as we go in. In the same way, high-level concepts should be at the top of a file and details should increase as we go down. Low-level function/details should be at the end of the file.

Vertical Formatting

Similar logical lines should be separated from other by a single blank line. The blank line is a visual cue that identifies a new and separate concept.

If one function calls another, they should be vertically close, and the caller should be above the callee, if at all possible. The stronger that affinity, the less vertical distance there should be between them. This gives the program a natural flow. Also, variables should be declared as close to their usage as possible.

Horizontal Formatting

We use horizontal white space to associate things that are strongly related and disassociate things that are weakly related. We surround the assignment operators with white space to accentuate them. The spaces make that separation obvious. But we don’t put spaces between the function names and the opening parenthesis. This is because the function and its arguments are closely related.

In python, we also wrap comparisons, mathematical and boolean operators with a single space but if there are multiple operators either in a single line then we wrap the operator with the lower priority. An example is given below.

The size of a line should be near to the length of its neighboring lines. So that the eyeballing should be minimized as much as possible while reading the code. It will make reading easier a lot.

For example in the code below, the size length is not near to its neighboring lines

1. Variable name should be meaningful, pronounceable, searchable and should reveal its intent. The data type should not be a part of it.

2. Class names should be a noun and have a clear description of the things inside it.

3. Function names should be a verb and do only one thing and should not be too long.

4. Functions should have the minimum number of arguments.

5. Comments can be written as an absolute last way of explaining what you are trying to accomplish and should be well thought.

6. Parts of codes linked together should be formatted next to each other.

7. The code should have as less eye-balling as possible.

If you have some questions or your perspective of this please do reach out to me through LinkedIn.

Happy Clean Coding