DOH!!!

[Pete OHanlon]

Yes.

*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier

[BobJanova]

I think we've all done something similar. But this is a compiler warning, certainly in C# (actually I think I remember it being so all the way back to the Zortech ANSI C compiler I used to use aged 10), so you should have caught it.

[Pete OHanlon]

And this is why I love working with MVVM and WPF - that wouldn't have been user interface code (and it didn't actually need to be there in this case either).

*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier

[BobJanova]

Surely whatever framework you're using you're going to have a line like

if(DialogResult.Yes == MessageBox.Show( ... )){
modelView.DoDeleteRows(selectedRows);
}

... in the UI layer? User confirmation in the UI is part of the requirement here. If you screw up that line, which is what happened here, your model-view and model code can be as perfect and as tested as you like and you can still make this mistake.

[Pete OHanlon]

Nope. I'd use Dependency Injection to control the message box, so it would look something like this:

IMessageBoxService mbox = ServiceContainer.Resolve();
bool? result = mbox.Show("....")
if (result.HasValue && result)
{
DeleteRows();
}

There's no need to have a selecteRows property because this would have been handled with the binding.

*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier

[J4amieC]

You;re right I read it wrong. I thought the OP was opening a custom Form.

[ZurdoDev]

The real question is how did that get out of development? Clearly no one actually tested it, right?

There are only 10 types of people in the world, those who understand binary and those who don't.

[Bernhard Hiller]

That's a pretty adequate programming style when you want to test your maintenance programmers. Hardly anyone will spot it.

[KP Lee]

I have to admit, it took me a short while to see it. I was verifying parenthesis while thinking the compiler would have blown up if you forgot one. Then I had to go "DOH!" too.

[Adriaan Davel]

Happened to me in c++ as a student, nearly gave up programming because of it :)

____________________________________________________________ Be brave little warrior, be VERY brave