MODSonline - unrecognized variable sidearm

Welcome Guest to MODSonline

Main-Menu

Tutorials

Members-Online

piznac	[Downl]
IchKG	[Forum]
cobbspur	[Forum]
shivix	[Tutor]
starkk	[Today]

5 Members and 18 Guests

Chat MODSonline

0 People Now Chatting

MODSOnline TeamSpeak

download TeamSpeak

In-The-News

Monday, Dec. 1st
A message from foyleman
Sunday, Nov. 30th
MODSonair 156 Live
Friday, Nov. 28th
MODSonair Episode 155
Wednesday, Nov. 26th
MODSonair Forum Buttons
Monday, Nov. 24th
Holiday Gift Guide 2008 - PC

Latest Poll

Newsletter

Your Membership

User Name:
Password:
	Register.

In The Forums

Arched window?
CoD:WW Level Des.. Posts: (9) Views: (87) by shivix

nazi_zombie problem with map : (
CoD:WW Level Des.. Posts: (4) Views: (51) by Papik

Appreciation
MODSonline.. Posts: (5) Views: (42) by Rasta

if your map switches to airfield read this !
CoD:WW Level Des.. Posts: (2) Views: (20) by darkcod5

Forums

MODSonline.com Forums
Welcome to the MODSonline.com forums. Looking for Frequently Asked Questions? Check out our FAQs section or search it out using the SEARCH link below. If you are new here, you may want to check out our rules and this great user's guide to the forums and the website. For more mapping and modding information, see our Wiki: MODSonWiki.com
Jump To:

Forum:	All Forums : Call of Duty 4
Category:	CoD 4 Scripting Scripting and coding with Call of Duty 4.
Moderators:	foyleman, rEdrUmMDK, Foxhound, Rasta, StrYdeR, The_Caretaker, batistablr, playername, Welshy

Author

Topic: unrecognized variable sidearm

DaveR

General Member
Since: Dec 5, 2006
Posts: 308
Last: Nov 23, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Wednesday, Sep. 17, 2008 01:05 pm
Hello, I'm trying to remove the ammo from both teams by scripting. I've edited the war.gsc document to do this; I added this to the bottom: Code: removeammo() { while(1) { players = getentarray("player", "classname"); for(i = 0; i < players.size; i++) { self removeAmmo( sidearm ); } wait 0.5; // do this function twice per second } } and I added this under main(): Code: thread removeammo(); When I test this it says 'unrecognized variable 'sidearm''. Does someone know what is going wrong and how I fix it? Thx.

Sevenz

General Member
Since: Apr 24, 2006
Posts: 1133
Last: Nov 28, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Wednesday, Sep. 17, 2008 01:52 pm
Code: removeammo() { while(1) { players = getentarray("player", "classname"); for(i = 0; i < players.size; i++) { self removeAmmo( sidearm ); // recursive call ??? } wait 0.5; // do this function twice per second } } Code: remove_ammo_all() { player = getentarray("player", "classname"); for(i = 0; i < players.size; i++) { player remove_ammo(); } } remove_ammo() { // self ... (setweaponammoclip etc. to set all ammo to 0) } edited on Sep. 17, 2008 01:53 pm by Sevenz

DaveR

General Member
Since: Dec 5, 2006
Posts: 308
Last: Nov 23, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Wednesday, Sep. 17, 2008 03:26 pm
thx I've made it like this: Code: remove_ammo_all() { player = getentarray("player", "classname"); for(i = 0; i < players.size; i++) { player remove_ammo(); } } remove_ammo() { // self SetWeaponAmmoClip( grenadeTypePrimary, grenadeCount ); } Then it says 'unilitialized variable 'players''. :/

Sevenz

General Member
Since: Apr 24, 2006
Posts: 1133
Last: Nov 28, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Wednesday, Sep. 17, 2008 03:42 pm
do you understand your own code? i think you don't... Quote: for(i = 0; i < players.size; i++) --> player.size Quote: // self SetWeaponAmmoClip( grenadeTypePrimary, grenadeCount ); --> why did you comment it out? and did you define both variables you use in the call?

Tally

General Member
Since: Apr 21, 2005
Posts: 199
Last: Dec 2, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Wednesday, Sep. 17, 2008 11:52 pm
Everything in a script has to be defined, unless it is preceeded with level.whatever (permanently defined until a round reset), or game["whatever"] (permanently defined even after a round reset). Example: level.players is defined in _globallogi.gsc and doesnt need defining again, hence you dont need to assign to player ent array: Code: players = level.players; for(i=0;i<players.size; i++) or simply: Code: for(i=0;i<level.players.size;i++) However, grenadeTypePrimary and grenadeCount are not level definitions, hence they would return as undefined. So, you need to define them. To remove ammo from a weapon, you need to tell the game what weapon. Otherwise, it will return as undefined. Hence you would need to either construct an array where every weapon in the game is found and checked against what the player actually has. Or, use some stock game functions and list each weapon one by one (tedius and clumsy). The game has some stock functions which can be used to check a players weapon: self hasweapon() self getCurrentWeapon() Try to use those to search for a players weapon and check it against an array of weapons. _weapons.gsc has some useful stuff in it, so start there. edited on Sep. 18, 2008 12:06 am by Tally

DaveR

General Member
Since: Dec 5, 2006
Posts: 308
Last: Nov 23, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Thursday, Sep. 18, 2008 10:21 am
Wow, this is kinda hard to understand for me, I'm new to scripting and I'm still learning all the stuff. I've found this in _weapons.gsc: Code: isPistol( weapon ) { return isdefined( level.side_arm_array[ weapon ] ); } hasScope( weapon ) { if ( isSubStr( weapon, "_acog_" ) ) return true; if ( weapon == "m21_mp" ) return true; if ( weapon == "aw50_mp" ) return true; if ( weapon == "barrett_mp" ) return true; if ( weapon == "dragunov_mp" ) return true; if ( weapon == "m40a3_mp" ) return true; if ( weapon == "remington700_mp" ) return true; return false; } I guess this are some weapon names? I'm trying to follow what you are saying, but probably I won't understand it completely. I put this under main() in war.gsc: Code: thread remove_ammo_all(); And then I put this at the bottom of war.gsc: Code: remove_ammo_all() { for(i=0;i<level.players.size;i++) { player remove_ammo(); } } remove_ammo() { self getCurrentWeapon() self SetWeaponAmmoClip( m21_mp, aw50_mp, barrett_mp ); } And then I add all the other weapons too it? Probably this sounds very stupid, but I'm still learning this.

.KiLL3R.

General Member
Since: Oct 26, 2006
Posts: 1090
Last: Dec 2, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Thursday, Sep. 18, 2008 10:46 am
Heres some quick examples - This code will get the uninitialized variable error Code: giveweap() { self giveWeapon(myweapon); } But this wont Code: giveweapon() { myweapon = "m4_mp"; self giveWeapon(myweapon); } this one wont Code: giveweapon(myweapon) { self giveWeapon(myweapon); } and this one wont, but will will give an error saying myweapon is undefined Code: giveweapon() { myweapon = undefined; self giveWeapon(myweapon); } In the 1st example, you will see that 'myweapon' hasnt been initialized before being used, but the other 3 have it initialized somewhere before its used. myweapon = "m4_mp"; Will initialize 'myweapon' as "m4_mp". giveweapon(myweapon) Can be called up as self thread giveweapon("m4_mp"); If there isnt anything in the ( and ) then 'myweapon' becomes undefined myweapon = undefined; Will initialize 'myweapon' as undefined. Stuff like [entity].myweapon and game["myweapon"] are always undefined unless you defined them somewhere (dont need to be initialized)

DaveR

General Member
Since: Dec 5, 2006
Posts: 308
Last: Nov 23, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Thursday, Sep. 18, 2008 12:37 pm
Thanks for your reply killer, that really helped me. I've edited the script to this: Code: remove_ammo_all() { for(i=0;i<level.players.size;i++) { player remove_ammo(); } } remove_ammo() { weapons = "m4_mp"; self SetWeaponAmmoClip( weapons ); } Now it should remove the ammunition for the m4 I guess, so I test it ingame, but it still gives me the 'uninitialized variable 'players'' error. :/ edited on Sep. 18, 2008 12:38 pm by DaveR

.KiLL3R.

General Member
Since: Oct 26, 2006
Posts: 1090
Last: Dec 2, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Thursday, Sep. 18, 2008 12:49 pm
The variable 'player' hasnt initialized been in Code: remove_ammo_all() { for(i=0;i<level.players.size;i++) { player remove_ammo(); } }

DaveR

General Member
Since: Dec 5, 2006
Posts: 308
Last: Nov 23, 2008
[view latest posts]

Category: CoD 4 Scripting Posted: Thursday, Sep. 18, 2008 01:10 pm
So I have to make something like: Code: player = self; or something like that? edited on Sep. 18, 2008 01:10 pm by DaveR