Don't Repeat Yourself (DRY) Design Principle

The idea behind the Don’t-Repeat-Yourself (DRY) design principle is an easy one:

• A piece of logic should only be represented once in an application. 
• Avoiding the repetition of any part of a system is a desirable trait.
• Code that is common to at least two different parts of your system should be factored out into a single location so that both parts call upon it.
• All this means that you should stop doing copy+paste right away in your software.

Your motto should be the following:

 

Repetition is the root of all software evil

• Repetition does not only refer to writing the same piece of logic twice in two different places. It also refers to repetition in your processes – testing, debugging, deployment, etc. 
• Repetition in logic is often solved by abstractions or some common service classes whereas repetition in your process is tackled by automation.
• A lot of tedious processes can be automated by concepts from Continuous Integration and related automation software such as TeamCity. 
• Unit testing can be automated by testing tools such as NUnit and xUnit. 
• DRY is known by other names as well: Once and Only Once, and Duplication is Evil (DIE)

We will take a look at the “logic” side of the DRY with a simple example.

Magic strings

These are hard-coded strings that pop up at different places throughout your code: connection strings, formats, constants, 

internal static class Problem {
    public static void DoSomething() {
        string address = "Isline, US";
        string format = "{0} is {1}, lives in {2}, age {3}";
        Console.WriteLine(format, "Abraham", "a good friend", address, 30);
    }
    public static void DoSomethingAgain() {
        string address = "Isline, US";
        string format = "{0} is {1}, lives in {2}, age {3}";
        Console.WriteLine(format, "Jeethu", "a neighbour", address, 54);
    }
    public static void DoSomethingMore() {
        string address = "Isline, US";
        string format = "{0} is {1}, lives in {2}, age {3}";
        Console.WriteLine(format, "Pramod", "my brother", address, 4);
    }
    public static void DoSomethingExtraordinary() {
        string address = "Isline, US";
        string format = "{0} is {1}, lives in {2}, age {3}";
        Console.WriteLine(format, "Parvathy", "my sister's best friend", address, 4);
    }
}

This is obviously a very simplistic example but imagine that the methods are located in different sections or even different modules in your application. In case you want to change the address you’ll need to find every hard-coded instance of the address. Likewise if you want to change the format you’ll need to update it in several different places.

We can put these values (address and format) into a separate location, such as Constants.cs,

internal class Constants {
    public static readonly string Address = "Isline, US";
    public static readonly string StandardFormat = "{0} is {1}, lives in {2}, age {3}";
}

The updated program looks as follows,

internal class SolutionIteration1 {
    public static void DoSomething() {
        string address = Constants.Address;
        string format = Constants.StandardFormat;
        Console.WriteLine(format, "Abraham", "a good friend", address, 30);
    }
    public static void DoSomethingAgain() {
        string address = Constants.Address;
        string format = Constants.StandardFormat;
        Console.WriteLine(format, "Jeethu", "a neighbour", address, 54);
    }
    public static void DoSomethingMore() {
        string address = Constants.Address;
        string format = Constants.StandardFormat;
        Console.WriteLine(format, "Pramod", "my brother", address, 4);
    }
    public static void DoSomethingExtraordinary() {
        string address = Constants.Address;
        string format = Constants.StandardFormat;
        Console.WriteLine(format, "Parvathy", "my sister's best friend", address, 4);
    }
}

If we change the constants in Constants.cs then the change will be propagated through the application.

However, we still repeat the following two lines over and over again.

string address = Constants.Address;
string format = Constants.StandardFormat;

The VALUES of the constants are now stored in one place, but what if we change the location of our constants to a different file? Or decide to read them from a file or a database? Then again we’ll need to revisit all these locations.

We can move those variables to the class level and use them in our code as follows:

Updated code as below,

internal class SolutionIteration2 {
    private static string address = Constants.Address;
    private static string format = Constants.StandardFormat;
    public static void DoSomething() {
        Console.WriteLine(format, "Abraham", "a good friend", address, 30);
    }
    public static void DoSomethingAgain() {
        Console.WriteLine(format, "Jeethu", "a neighbour", address, 54);
    }
    public static void DoSomethingMore() {
        Console.WriteLine(format, "Pramod", "my brother", address, 4);
    }
    public static void DoSomethingExtraordinary() {
        Console.WriteLine(format, "Parvathy", "my sister's best friend", address, 4);
    }
}

We’ve got rid of the magic string repetition, but we can do better.

Notice that each method performs basically the same thing: write to the console.

This is an example of duplicate logic. The data written to the console is very similar in each case;

We can factor it out to another method,

private static void WriteToConsole(string name, string description, int age) {
    Console.WriteLine(format, name, description, address, age);
}

Updated Code below,

internal class SolutionIteration3 {
    private static string address = Constants.Address;
    private static string format = Constants.StandardFormat;
    public static void DoSomething() {
        WriteToConsole("Abraham", "a good friend", 30);
    }
    public static void DoSomethingAgain() {
        WriteToConsole("Jeethu", "a neighbour", 54);
    }
    public static void DoSomethingMore() {
        WriteToConsole("Pramod", "my brother", 4);
    }
    public static void DoSomethingExtraordinary() {
        WriteToConsole("Parvathy", "my sister's best friend", 4);
    }
    private static void WriteToConsole(string name, string description, int age) {
        Console.WriteLine(format, name, description, address, age);
    }
}

Magic Numbers

It’s not only magic strings that can cause trouble but magic numbers as well.

Imagine that you have the following class in your application,

internal class Associate {
    public string Name {
        get;
        set;
    }
    public int Age {
        get;
        set;
    }
    public string Department {
        get;
        set;
    }
}

We will mock the database as below,

private static IEnumerable < Associate > GetAssociates() {
    return new List < Associate > () {
        new Associate() {
                Age = 30, Department = "IT", Name = "Prasad"
            },
            new Associate() {
                Age = 34, Department = "Marketing", Name = "Praveen"
            },
            new Associate() {
                Age = 28, Department = "Security", Name = "Pramod"
            },
            new Associate() {
                Age = 40, Department = "Management", Name = "Steve"
            }
    };
}

Notice the usage of the index 1 in the following method,

private static void DoMagicInteger() {
    var associates = GetAssociates().ToList();
    if (associates.Count > 0) {
        Console.WriteLine(string.Concat("Age: ", associates[1].Age, ", department: ", associates[1].Department, ", name: ", associates[1].Name));
    }
}

We only want to output the properties of the second associate in the list, i.e. the one with index 1.

• One issue is a conceptual one: why are we only interested in that particular associate? What’s so special about him/her? This is not clear for anyone investigating the code. 
• The second issue is that if we want to change the value of the index then we’ll need to do it in three places. If this particular index is important elsewhere as well then we’ll have to visit those places too and update the index.

We can solve both issues using the same simple techniques as in the previous example. Set a new constant in Constants.cs

internal class Constants {
    public static readonly string Address = "Isline, US";
    public static readonly string StandardFormat = "{0} is {1}, lives in {2}, age {3}";
    public static readonly int IndexOfMyFavouriteAssociate = 1;
}

Then introduce a new class level variable

private static int indexOfMyFavouriteAssociate= Constants.IndexOfMyFavouriteAssociate;

The updated DoMagicInteger() method looks as follows,

private static void DoMagicInteger() {
    var associates = GetAssociates().ToList();
    if (associates.Count > 0) {
        var associate = associates[indexOfMyFavouriteAssociate];
        Console.WriteLine(string.Concat("Age: ", associate.Age, ", department: ", associate.Department, ", name: ", associate.Name));
    }
}

We will be discussing the below two DRY series in the upcoming tutorials.

• Repeated logic
• RepeatedExecution Pattern

Next in this series: DRY Part 2