hey im working on a mod and im trying to make this script find out what weapon i picked, i have this in my .menu scriptMenuResponse "thompson_mp";

then in my menus.gsc i have

if(menu == game["menu_automatics1"] || menu == game["menu_semiautomatics1"] || menu == game["menu_lightmachineguns1"] || menu == game["menu_rifles1"] || menu == game["menu_shotguns1"])
			self thread maps\mp\gametypes\_weaponsystem::setWeaponMenu(response);

then in my setWeaponMenu(response) i have

setWeaponMenu(response)
{
	for(i=0; i <level.weaponnames.size; i++)
	{
		if(response == level.weaponnames[i])
		{
			self iprintlnbold("working");
		}
		else
		{
			self iprintlnbold("notworking");
		}
	}
}

but its playing the test2notworking... why is it saying my response isnt any of the weapon names? any ideas?

The first part of your code is correct, its the second part that is causing the problem. There is no point transferring an argument to another function if you're not going to use it. In this case, you transferred "response" to the second function, but then you completely abandon it and start using level.weaponnames.

Try this instread:

setWeaponMenu(response)
{
	switch( response )
	{
		case "thompson_mp":
			self iprintlnBold( "working" );
			break;
	}
}

Add a case for each of your weapons, then do with it as you will.

edited on Aug. 2, 2010 03:34 am by DemonSeed

thank you it works, the reason i tryed the other method is so i didnt have to go threw ALL those cases to make my life easyer but i guess ill use this way thank you.