Try the following code. The only problem I see wit...

2007-06-20T21:13:00.000-06:00

Try the following code. The only problem I see with you're code is that you are implicitly creating your strategy classes in your code. Tell, Don't Ask is my motto. Make those type codes work for a living.

Additional Classes is a cached repository for the strategies classes. and then a locater for the repository. I even included a composite strategy, ALL, cause sometimes we have too much time on our hands.

using System.Collections;
using NUnit.Framework;

namespace Ghetto
{
[TestFixture]
public class TestGhetto
{
private Person gutzer;

[SetUp]
public void SetUp()
{
gutzer = new Person("Gutzer");
}

[TearDown]
public void TearDown()
{
gutzer = null;
}

[Test]
public void ShouldDJ()
{
gutzer.WhatImDoing(Doing.DJ);
gutzer.DoingIt();
Assert.IsTrue(gutzer.Spent);
}

[Test]
public void ShouldBMX()
{
gutzer.WhatImDoing(Doing.BMX);
gutzer.DoingIt();
Assert.IsTrue(gutzer.Spent);
}

[Test]
public void ShouldGetDrunk()
{
gutzer.WhatImDoing(Doing.DRUNK);
gutzer.DoingIt();
Assert.IsTrue(gutzer.Spent);
}

[Test]
public void ShouldDoItAll()
{
gutzer.WhatImDoing(Doing.ALL);
gutzer.DoingIt();
Assert.IsTrue(gutzer.Spent);
}
}

public class Doing
{
public static Doing DJ = new Doing();
public static Doing BMX = new Doing();
public static Doing DRUNK = new Doing();
public static Doing ALL = new Doing();
}

public class Person
{
private readonly string mName;
private bool spent;
private Doing isDoing;

public Person(string name)
{
mName = name;
}

public string Name
{
get { return mName; }
}

public bool Spent
{
get { return spent; }
}

public void DoingIt()
{
DoingThings things = DoingThingsLocater.GetDoingThings();
spent = things.DoIt(isDoing);
}

public void WhatImDoing(Doing doing)
{
isDoing = doing;
}
}

public class DoingThingsLocater
{
public static DoingThings GetDoingThings()
{
return new DoingThings();
}
}

public class DoingThings
{
private readonly Hashtable things = new Hashtable();

public DoingThings()
{
things[Doing.BMX] = new BMXStrategy();
things[Doing.DJ] = new DJStrategy();
things[Doing.DRUNK] = new GetDrunkStrategy();
things[Doing.ALL] = new AllStrategy(new DJStrategy(), new BMXStrategy(), new GetDrunkStrategy());
}

public bool DoIt(Doing doing)
{
DoingStrategy doingIt = things[doing] as DoingStrategy;
if (doingIt != null)
return doingIt.GettingItDone();
return false;
}
}

public abstract class DoingStrategy
{
public abstract bool GettingItDone();
}

public class DJStrategy : DoingStrategy
{
public override bool GettingItDone()
{
return true;
}
}

public class BMXStrategy : DoingStrategy
{
public override bool GettingItDone()
{
return true;
}
}

public class GetDrunkStrategy : DoingStrategy
{
public override bool GettingItDone()
{
return true;
}
}

public class AllStrategy : DoingStrategy
{
private readonly IList mStrategyCompositeList = new ArrayList();

public AllStrategy(DoingStrategy dj, DoingStrategy bmx, DoingStrategy drunk)
{
mStrategyCompositeList.Add(dj);
mStrategyCompositeList.Add(bmx);
mStrategyCompositeList.Add(drunk);
}

public override bool GettingItDone()
{
bool result = true;

foreach (DoingStrategy strategy in mStrategyCompositeList)
{
result &= strategy.GettingItDone();
}

return result;
}
}
}

I have so much enum centric code. I just tried ref...

2006-11-09T21:47:00.000-07:00

I have so much enum centric code. I just tried refactoring my code as you have shown here and it works so much better. Thank you!

Comments on e.zed: Switch Statement = Ghetto... Enum = Ghetto

Try the following code. The only problem I see wit...

I have so much enum centric code. I just tried ref...