FizzBuzz in Csharp

Question

For my first CodeReview, I would like to present my FizzBuzz solution in C#. My design has all of the rule logic about when to display what in separate classes through an interface. This design should be able to handle the common follow-up changes (ie. mod 7 instead of mod 3, bazz, etc.) for this problem by simply adding another rule to the game. Below is the problem statement.

And as I am not a professional developer, I am especially interested in additional feedback on style, formatting, and adherence to SOLID principles, as well as limitations of this design.

Problem: Write a short program that prints each number from 1 to 100 on a new line. For each multiple of 3, print "Fizz" instead of the number. For each multiple of 5, print "Buzz" instead of the number. For numbers which are multiples of both 3 and 5, print "FizzBuzz" instead of the number.

Solution:

class Program
{
    static void Main(string[] args)
    {
        var rules = new List<IRule>
        {
            new FizzMod3Rule(),
            new BuzzMod5Rule(),
            new BazzMod7Rule()
        };

        new FizzBuzzGame(rules).Play();

        Console.ReadKey();
    }
}

public class FizzBuzzGame
{
    private List<IRule> _rules;

    public FizzBuzzGame(List<IRule> rules)
    {
        _rules = rules ?? new List<IRule>();
    }

    public void Play(int numberOfRounds = 100)
    {
        for (var i = 1; i <= numberOfRounds; i++)
        {
            var output = string.Empty;
            foreach (var rule in _rules) { output += rule.Run(i); }
            if (output == string.Empty) { output += new EmptyRule().Run(i); }
            Console.WriteLine(output);
        }
    }
}

public interface IRule
{
    string Run(double number);
}

public class FizzMod3Rule : IRule
{
    public string Run(double number)
    {
        return (number % 3) == 0 ? "Fizz" : string.Empty;
    }
}

public class BuzzMod5Rule : IRule
{
    public string Run(double number)
    {
        return (number % 5) == 0 ? "Buzz" : string.Empty;
    }
}

public class BazzMod7Rule : IRule
{
    public string Run(double number)
    {
        return (number % 7) == 0 ? "Bazz" : string.Empty;
    }
}

public class EmptyRule : IRule
{
    public string Run(double number)
    {
        return number.ToString();
    }
}

Output:

1
2
Fizz
4
Buzz
Fizz
Bazz
8
Fizz
Buzz
11
Fizz
13
Bazz
FizzBuzz
16
17
Fizz
19
Buzz
FizzBazz
...

Answer 1

The intention of classes in object oriented programming is not to replace if sentences, but to model code that differs in behavior. Your 4 classes can be merged into 2, since the first 3 of them use structurally identical code. The code could then be:

var rules = new List<IRule>
{
    new ModWordRule(3, "Fizz"),
    new ModWordRule(5, "Buzz"),
    new ModWordRule(7, "Bazz")
};

---

On second sight, your code completely misses the point of the task. The task says:

Write a short program.

Your program has 74 lines, and I'm not even counting the using declarations that you omitted at the top of the file. You should not omit these when you post your code for code review.

I just wrote the same program in 20 lines, without all the "object oriented" boilerplate. It is easier readable, and equally easy to extend to more rules.

I don't see any point in adding so many interfaces and classes if the problem can be solved in a few lines of code, especially if the simple code is easily testable.

The argument of "don't modify the existing classes when adding features" is moot here since adding a "7 Bazz" rule obviously has to modify the output, and that output has to be assembled by some class. In the FizzBuzz example the code is still simple enough to go into a single function instead of spreading it over 4 classes.

The task does not mention the ReadKey anywhere, therefore you should omit it.

The task says to output "7" for the number 7, your program outputs "Bazz". This latter word is mentioned nowhere in the task.

When dealing with collections of things, it is a common assumption that these collections are never null. Therefore there's no need to check for that condition in the FizzBuzzGame constructor. Just let the program crash in such a case. It's a serious programming mistake, and it must be fixed. Your current code only hides the bug.

As I said above, I prefer code that is to the point, while still testable. Such as the following:

using System;

namespace FizzBuzz231713
{
    class Program
    {
        static object FizzBuzz(int n)
        {
            var fizz = n % 3 == 0 ? "Fizz" : "";
            var buzz = n % 5 == 0 ? "Buzz" : "";
            var word = fizz + buzz;
            return word != "" ? word : (object)n;
        }

        static void Main()
        {
            for (int i = 0; i < 100; i++)
                Console.WriteLine(FizzBuzz(i + 1));
        }
    }
}