Thread: Case statement puzzle

Can someone tell me what's wrong with this block of skeleton code so far??

Code:

        protected override void WndProc(ref Message m) 
        {
            // Listen for operating system messages.
            switch (m.Msg)
            {
                case DBT_DEVICEARRIVAL:                 
                        MessageBox.Show("A device has been docked");
                        break;                    

                case DBT_DEVICEREMOVECOMPLETE:         
                        MessageBox.Show("A Device has been removed");
                        break;
                
                default:<<=========== This part gets underlined in blue with the message: Control cannot fall through from one case label to another.
                        MessageBox.Show("Nothing detected"); 
                        break;                  

            }
            base.WndProc(ref m);
        }//end wndProc

Far as I can see, there's nothing wrong here. I may be overlooking something as I am tired at the moment

That exception is thrown if the label ("default" in this instance) does not terminate, however you have a break so I find it most bizarre that this is happening.

For what it's worth, you don't strictly need to use the default label. You could terminate the function if one of your two labels are detected, or otherwise allow your default label code to occur outside of the switch. Example:

Code:

        protected override void WndProc(ref Message m) 
        {
            switch (m.Msg)
            {
                case DBT_DEVICEARRIVAL:
                    MessageBox.Show("A device has been docked");
                    return;

                case DBT_DEVICEREMOVECOMPLETE:
                    MessageBox.Show("A Device has been removed");
                    return;
            }

            MessageBox.Show("Nothing detected");
            base.WndProc(ref m);
        }

Thanks.
I actually added the default (Contextually a logical error as it causes an infinite loop) to see if I could solve a previous error as it ws giving me the same error message with my last case statement. In any case I just rewrote my code the exact same way and it worked!!
Yes even I'm puzzled. o_0?

Originally Posted by WDT

Thanks.
I actually added the default (Contextually a logical error as it causes an infinite loop) to see if I could solve a previous error as it ws giving me the same error message with my last case statement. In any case I just rewrote my code the exact same way and it worked!!
Yes even I'm puzzled. o_0?

I think some part of "exact" wasn't as exact as you wanted it to be - I've been there and done that MANY times.

--
Mats