Ayende @ Rahien

It's a girl

Code of the week

This piece of code make me so happy:

private static IDictionary<string, Func<Stream, ICommand>> commandFactories =
	new Dictionary<string, Func<Stream, ICommand>>
		{
			//storage
			{"add", s => new AddCommand(s)},
			{"append", s => new AppendCommand(s)},
			{"cas", s => new CasCommand(s)},
			{"delete", s => new DeleteCommand(s)},
			{"prepend", s => new PrependCommand(s)},
			{"replace", s => new ReplaceCommand(s)},
			{"set", s => new SetCommand(s)},

			// retrieval
			{"get", s => new GetCommand(s)},
			{"gets", s => new GetsCommand(s)},

			//modifications
			{"incr", s => new IncrCommand(s)},
			{"decr", s => new DecrCommand(s)},

			//misc
			{"flush_all", s => new FlushAllCommand(s)},
			{"quit", s => new QuitCommand(s)},
			{"version", s => new VersionCommand(s)},
		};

This is something that I do quite often, and it is good to have a way to do it easily.

Comments

James L
06/06/2008 12:22 PM by
James L

They look like memcached commands

Alex Simkin
06/06/2008 01:18 PM by
Alex Simkin

You should have request commands from IoC container, not from the dictionary.

Gary A
06/06/2008 01:20 PM by
Gary A

So what does it do, and why did you do it this way?

Harry M
06/06/2008 01:27 PM by
Harry M

Have I just made a DSL?

public static void Main(string[] args)

    {

        var me = new Person("Harry");

        var you = new Person("Bob");

        var options = new Dictionary<string, Action<string>>()

                      {

                        {"I", s => DoSomething(me, s)},

                        {"You", s => DoSomething(you, s)}

                      };

        Parse("I say hello", options);

        Parse("I jump", options);

        Parse("You say goodbye", options);


    }


    private static void Parse(string command, IDictionary<string, Action<string>> options)

    {

        string[] parts = command.Split(new char[] {' '}, 2);

        options[parts[0]](parts.Length > 1 ? parts[1] : null);

    }


    private static void DoSomething(Person who, string what)

    {

        var options = new Dictionary<string, Action<string>>()

                      {

                        {"say", s => who.Say(s)},

                        {"jump", s => who.Jump()}

                      };

        Parse(what, options);

    }

    class Person

    {

        private readonly string name;

        public Person(string name)

        {

            this.name = name;

        }

        public void Say(string message)

        {

            Console.WriteLine("{0} says: '{1}'", name, message);

        }

        public void Jump()

        {

            Console.WriteLine("{0} jumps", name);

        }

    }
Vijay Santhanam
06/06/2008 05:29 PM by
Vijay Santhanam

love the dictionary lookups with the generic delegates/lambdas too.

as much as i'm interested in your code, i'm doubley interested in your design process. e.g., how much analyse/OO modelling you do before you start writing real code?

With tools like R#/svn is the up-front design pretty much dead for garage projects?

would like to read about the steps you take before you get to code. i ask because as a sophisicated practitioner, you take intermediate working steps for granted whilst someone at my level struggles to keep up sometimes.

firefly
06/06/2008 09:29 PM by
firefly

The more I dig into C# 3.0 the more I infatuate with it.

firefly
06/07/2008 04:50 AM by
firefly

Question...

private static IDictionary do you gain anything by using the IDictionary interface verse just using a regular Dictionary class?

Ayende Rahien
06/07/2008 05:15 AM by
Ayende Rahien

I like using the most abstract type possible

Andrey Shchekin
06/08/2008 08:58 AM by
Andrey Shchekin

What about

IDictionary<string, Func<Stream, ICommand>> CreateCommandFactories(IEnumerable types) {

      return (

            from type in types

            select new {

                  Name = type.Name.Replace("Command", "").ToLowerInvariant(),

                  Factory = (Func<Stream, ICommand>)(s => Activator.CreateInstance(type, s))

            }

      ).ToDictionary(

            x => x.Name,

            x => x.Factory

      )

}

Of course better way would be to use Regex for parsing and Expression.Compile for constructor.

Ayende Rahien
06/08/2008 09:42 AM by
Ayende Rahien

That involves reflection. I wanted this to be zero reflection effort

Dmitry Gusarov
06/13/2008 07:28 AM by
Dmitry Gusarov

Ok, but Andrey suggested an excellent duplication removal.

I thought of it... what is more valuable? Avoiding reflection or avoiding duplication... In any case we have a tests ;)

Ayende Rahien
06/13/2008 10:21 AM by
Ayende Rahien

Dmitry,

In this case, what I wanted was no reflection. The price paid for it is worth it, I think.

Comments have been closed on this topic.