AlliedModders - Simple Loosing Team Slayer

AlliedModders (https://forums.alliedmods.net/index.php)

- Unapproved Plugins (https://forums.alliedmods.net/forumdisplay.php?f=109)

- - Simple Loosing Team Slayer (https://forums.alliedmods.net/showthread.php?t=134593)

Simple Loosing Team Slayer

Admin may remove this plugin now.

Re: Simple Loosing Team Slayer

Quote:

Originally Posted by sEbbo (Post 1263754)

My version doesn't have any cvar's...

Not true.

Code:

public OnMapStart()

{

        target_bombed = CreateConVar("target_bombed", "CT slayed for failing their objective.", "When the bomb detonate.");

        target_saved = CreateConVar("target_saved", "T slayed for failing their objective.", "When the bomb dont get planted");

        bomb_defused = CreateConVar("bomb_defused", "T slayed for failing their objective.!", "When the bomb is defused");

        all_hostages_rescued = CreateConVar("all_hostages_rescued", "T slayed for failing their objective.", "When all hostages are rescued");

        hostages_not_rescued = CreateConVar("hostages_not_rescued", "CT slayed for failing their objective.", "When hostages are NOT rescued");

}

Re: Simple Loosing Team Slayer

Quote:

Originally Posted by sEbbo (Post 1263754)

...
Credits

sEbbo - For the idea and some coding.
Lindgren - Coding almost everything.

Funny

Re: Simple Loosing Team Slayer

This won't be approved because:

CreateConVar should be used in OnPluginStart, not OnMapStart.
Lines 35-40 serve no purpose whatsoever.
GetMaxClients shouldn't be called inside a loop, you'd only need to check the value once. In fact you should just use MaxClients instead of calling GetMaxClients at all.
The stack of ifs in Delayed_Slay should be else ifs or ideally a switch since "param" can't equal more then one value at a time.
GetMaxClients again on line 84.
Health check should be replaced with IsPlayerAlive.
FakeClientCommand should be replaced with ForcePlayerSuicide.