Fatal error on random case

[MightyHythloth]

Fatal error on random case

Greetings. I'm trying to create an item that when double clicked gives you a check for a random amount and deletes the item itself. I have the other parts of the item working fine, but for some reason I keep getting a fatal error with this part. Btw, using 2.0RC1

	public override void OnDoubleClick( Mobile from ) 
	{ 
	if ( !IsChildOf( from.Backpack ) )
			{
				from.SendLocalizedMessage( 1042001 ); // That must be in your pack for you to use 

it.
			} 
         else 
			switch ( Utility.Random(2))
			{
				default:
				case 0: from.AddToBackpack( new BankCheck( 100000 ) ); break;
				case 1: from.AddToBackpack( new BankCheck( 250000 ) ); break;
				
			}
			this.Delete;
	}

 

[Alex21]

I think it might be

default:

you havn't put a

break;

on it which would mean it was un-able to fall through.

but should works like this

default: break;

Any way be nice if you could post the error if that dosn't fix it.

 

[Lord_Greywolf]

the default is fine that way, i have many scripts that way - it is the same as doing "case 0: default:" etc

try this:

         else
		{ 
			switch ( Utility.Random(2))
			{
				default:
				case 0: from.AddToBackpack( new BankCheck( 100000 ) ); break;
				case 1: from.AddToBackpack( new BankCheck( 250000 ) ); break;
				
			}
			this.Delete;
		}

 

[Alex21]

the default is fine that way, i have many scripts that way - it is the same as doing "case 0: default:" etc

try this:

         else
		{ 
			switch ( Utility.Random(2))
			{
				default:
				case 0: from.AddToBackpack( new BankCheck( 100000 ) ); break;
				case 1: from.AddToBackpack( new BankCheck( 250000 ) ); break;
				
			}
			this.Delete;
		}

well. i didn't know the exact syntax that could be used. But what was the error he had?

 

[MightyHythloth]

Thanks Lord_Greywolf, that worked like a charm.

 

[Lord_Greywolf]

no problem

but one of the things i learned about cases is you can do it like this:

case 0:
case 2:
case 6:
case 10: and what you put here will also be done by case 0, 2, 6 & 10; break
case 1:
case 3:
case 9:
etc
case 15: and will do 1, 3, 9, etc & 15; break
default: case 14: case 16: will do all that do not have it + 14 & 16; break

good to know for future reference

works with any variables that match up also

just remember that default means ANY that are not used anywhere else in it, so becareful on using it

 

[MightyHythloth]

good to know, thanks much.