Definitely! The menu structure that this is leading to is begging for a good IoC container and a builder or factory. I didn’t want to go into those details for the meat of this post, though. I think Chad Myers is going to talk about exactly that – IoC and builder / factory – for the PTOM Design Patterns.

*nudge nudge, chad* :)

In the real world would you use something like a MenuBuilder (Builder pattern) to construct your app menu? I am thinking that you could provide the MenuBuilder class references to stuff via DI (like the pricing service you were eluding to).

public class PricedMenuItem
{

private ICommand _menuCommand;
private Price _productPrice;

public Text { get; set; }

public PricedMenuItem(string menuText, IParameterizedCommand menuCommand, Price productPrice)
{
_menuCommand = menuCommand;
_productPrice = productPrice;
}

public void Click()
{
_menuCommand.Execute(_productPrice);
}

} ]]> http://lostechies.com/derickbailey/2008/11/20/ptom-command-and-conquer-your-ui-coupling-problems/#comment-62 derick.bailey Thu, 20 Nov 2008 20:46:15 +0000 /blogs/derickbailey/archive/2008/11/19/ptom-command-and-conquer-your-ui-coupling-problems.aspx#comment-62 @Ike DOH! You've found the glaring hole in my example. :) Given the structure that I outlined, I would probably create a specialized MenuItem for products with a price - something like this: //... somewhere in the application UI Price coffeePrice = myPriceInfo.GetPriceFor("Regular Coffee"); MenuItem regularCoffeeMenuItem = new MenuItem("Regular Coffee", new RegularCoffeeCommand(), coffeePrice); @Ike

DOH! You’ve found the glaring hole in my example. :)

Given the structure that I outlined, I would probably create a specialized MenuItem for products with a price – something like this:

//… somewhere in the application UI
Price coffeePrice = myPriceInfo.GetPriceFor(“Regular Coffee”);
MenuItem regularCoffeeMenuItem = new MenuItem(“Regular Coffee”, new RegularCoffeeCommand(), coffeePrice);

Can you show an example of the menuitem with a parameterized ICommand.
Right now, I don’t see how you can provide the data to the execute method (cleanly), since you need to have some info on the type. Or do you then also use generic MenuItems

Thanks,
Best regards,
Ike