[FreezeTag Mod]

FrOZeN F1rE · **Plugin Info:**

FreezeTag Mod
Mod Originally By Javavi

Description
It Is A Very Simple And Fun Mod.
The CT's Have 3 Minutes(You Can Change This)
The Terrorists have to run from CT's in order to stay alive.
The CT's - When CT's Touch The T's , The T's Freeze!
The T's - T's Can Unfreeze Their Teamates By Touching Them..!
There Are Basically 2 Shops in this mod..

For CT's and Other For T's

Client Commands:

PHP Code:


			
/status

/shop

Edits:

PHP Code:


			
Added Shop.

Fixed Minor Bugs.

Added No Fall Damage ~ Credits v3x

Shop Items Help :- Eagle07 :D

Credits:

PHP Code:


			
Javavi :D // The Mod Idea And The Original Code

Eagle07 :) // Shop and Other Bug Helps :wink:

Note:

PHP Code:


			
This mod is originally made by Javavi . 

I just edited it as some players suggested.

Mod is Just For Fun!

Any Suggestions and any Bug Reports may be appriciated :D 

Thanks Everyone :)

Online Compiler Is Not Working..

Compile It Yourself

YamiKaitou · 10-07-2014 , 12:13 Re: [FreezeTag Mod]

Remove the AMXX file from the archive

FrOZeN F1rE · 10-07-2014 , 12:13 Re: [FreezeTag Mod]

Ohkay... Done

~My First Post

**HamletEagle** · 10-07-2014 , 12:25 Re: [FreezeTag Mod]

Some improvements:

Don't use fakemeta_util.
V3x no fall damage plugin is old and badly coded, now you have Ham_TakeDamage. Check fo DMG_FALL and block the damage.
Don't use ResetHud for spawn, use Ham_Spawn.
In Spawn_2 you are creating a permanent task, at 0.5 seconds, this is a bad way. Use set_task with flag b.
dhud doesn't have channels, if your hud is updated too fast, it will get bugged.
get_user_godmode(id), set_user_godmode(id, 1) for what are you using this ? It doesn't make sense.
In fw_Touch why some variables are static and some with new ? This seems to be a copy-paste.
for ( new id = 1; id <= g_maxplayers; id++ ) don't create a for loop like this. Use get_players
I'm almost sure that the code from msg_statusicon is not yours, it seems to be connor's code.
Indent your code.
In an if- else structure: in the if case, you don't need to return plugin_handled, it will not go to else, if the condition is true.
A lot of code is simply copy-paste, what do you really coded ?

There are much more things that can be done better or in another way, but I don't have time to check more.

FrOZeN F1rE · 10-07-2014 , 12:32 Re: [FreezeTag Mod]

Quote:

Originally Posted by HamletEagle

Some improvements:

Don't use fakemeta_util.
V3x no fall damage plugin is old and badly coded, now you have Ham_TakeDamage. Check fo DMG_FALL and block the damage.
Don't use ResetHud for spawn, use Ham_Spawn.
In Spawn_2 you are creating a permanent task, at 0.5 seconds, this is a bad way. Use set_task with flag b.
dhud doesn't have channels, if your hud is updated too fast, it will get bugged.
get_user_godmode(id), set_user_godmode(id, 1) for what are you using this ? It doesn't make sense.
In fw_Touch why some variables are static and some with new ? This seems to be a copy-paste.
for ( new id = 1; id <= g_maxplayers; id++ ) don't create a for loop like this. Use get_players
I'm almost sure that the code from msg_statusicon is not yours, it seems to be connor's code.
Indent your code.
In an if- else structure: in the if case, you don't need to return plugin_handled, it will not go to else, if the condition is true.
A lot of code is simply copy-paste, what do you really coded ?

There are much more things that can be done better or in another way, but I don't have time to check more.

Thanks For Opinion:
I'll Look Through it

N yes I have Copy-Pasted Some Codes ...
And I'll remove all the useless codes

Thank You .

5aloOod · 10-10-2014 , 01:45 Re: [FreezeTag Mod]

PHP Code:


			
L 10/10/2014 - 09:42:02: Start of error session.

L 10/10/2014 - 09:42:02: Info (map "hns_fantazy") (file "addons/amxmodx/logs/error_20141010.log")

L 10/10/2014 - 09:42:02: Function is not present (function "advertise_ii") (plugin "freezetag-new.amxx")

L 10/10/2014 - 09:42:02: [AMXX] Displaying debug trace (plugin "freezetag-new.amxx", version "2")

L 10/10/2014 - 09:42:02: [AMXX] Run time error 10: native error (native "set_task")

L 10/10/2014 - 09:42:02: [AMXX]    [0] freezetag-new.sma::advertise_i (line 316)

bsparty · 10-11-2014 , 07:06 Re: [FreezeTag Mod]

Nice Mod
Will Surely try !

DonKisgot · 10-12-2014 , 15:27 Re: [FreezeTag Mod]

Some Questions ;

- How teams are changing ? (By random ? Per round?)

My Suggestions ;

- Mix Teams Every Round.

**HamletEagle** · 10-13-2014 , 09:14 Re: [FreezeTag Mod]

More in deep:

Don't write string size manually, use charsmax.
Use pcvars.
You are registering a cvar with an empty string, then you get it into a string and check if it's 0. It's obvious that it will be like that, you should double check what you are doing.
Use formatex, not format.
When removing the ents, you may want to check if that ent is valid with pev_valid.
If you block chooseteam cmd, block jointeam too.
You have the b flag for looping a task, use it.
Why you are removing ents in two places ? Just remove them with the same loop.
Use get_players, you have flags to filter alive/dead players and much more.
I would advise you to use offset names, 235 is m_fMapZones if I'm not wrong.
Why you are switching when using only 1 case ?
In death_event you don't need to create a var only for using it one time, write read_data directly.
Why you are using FM's player pretink and engine one too ? Stick to one of them.
In your case, you don't need to use the menu info param, so you will avoid some native calls( menu_item_getinfo )

Please try to do some of what I wrote to you, it may help you improve this plugin

FrOZeN F1rE · 10-14-2014 , 06:41 Re: [FreezeTag Mod]

Quote:

Originally Posted by DonKisgot

Some Questions ;

- How teams are changing ? (By random ? Per round?)

My Suggestions ;

- Mix Teams Every Round.

Bro, The Teams Change When One Team Defeats The Other ..
That Is - When CT's Freeze All The T's, Then The Round Ends and The CT's Become T's and Vice-Versa..
If The CT's Fail to do so, No Team Change