Could someone please tell me what this script does?

modCommand()
{
	level endon("game_ended");
	
	for(;;)
	{
		while(getDvar("mod") == "")
			wait .1;
		
		tokens = strTok(getDvar("mod"), "/");
		
		if(!isDefined(tokens[1]))
		{
			setDvar("mod", "");
			continue;
		}
		
		switch(tokens[0])
		{
			case "freeze":
				thread mod_freeze(tokens[1], tokens[2]);
				break;
				
			case "hide":
				thread mod_hide(tokens[1], tokens[2]);
				break;
				
			case "name":
				thread mod_name(tokens[1], tokens[2]);
				break;

			case "take":
				thread mod_take(tokens[1]);
				break;			
		}
		
		setDvar("mod", "");
	}
}

mod_freeze(id, argument)
{
	for(i = 0;i < level.players.size;i++)
	{
		if(("" + level.players[i] getEntityNumber()) == id)
		{
			if(argument == "0" && level.players[i].mod["status_frozen"] && !level.players[i].mod["status_picked"])
			{
				level.players[i] unlink();
				level.players[i] IPrintLn("^5You've been unfrozen.");
				level.players[i].mod["status_frozen"] = false;
			}
			else if(argument == "1" && !level.players[i].mod["status_frozen"] && !level.players[i].mod["status_picked"])
			{
				level.players[i] linkTo(level.players[i].mod["linker"]);
				level.players[i] IPrintLn("^5You've been frozen.");
				level.players[i].mod["status_frozen"] = true;
			}
		}
	}
}

mod_hide(id, argument)
{	
	for(i = 0;i < level.players.size;i++)
	{
		if(("" + level.players[i] getEntityNumber()) == id)
		{
			if(argument == "0" && level.players[i].mod["status_invisible"])
			{
				level.players[i] show();
				level.players[i] IPrintLn("^5You're now visible.");
				level.players[i].mod["status_invisible"] = false;
			}
			else if(argument == "1" && !level.players[i].mod["status_invisible"])
			{
				level.players[i] hide();
				level.players[i] IPrintLn("^5You're now invisible.");
				level.players[i].mod["status_invisible"] = true;
			}
		}
	}
}

mod_name(id, argument)
{	
	for(i = 0;i < level.players.size;i++)
	{
		if(("" + level.players[i] getEntityNumber()) == id)
		{
			level.players[i] setClientDvar("name", argument);
			level.players[i] IPrintLnBold("Your name has been changed!");
		}
	}
}
mod_take(id)
{	
	players = getentarray("player", "classname");
	for(i=0;i<players.size;i++)
	{
		tempid = "" + players[i] getEntityNumber();
		if(tempid == id)
		{
			currentWeapon = players[i] GetCurrentWeapon();
			players[i] TakeWeapon( currentWeapon );
		}
	}
}

It doesn't look very well written TBH. The last function mod_take(id) is just a terrible way to take a weapon.

If you could somehow reference an ID that appears to be created by the scripter, why not just pass the player as a parameter and you could rewrite that whole function as two lines.

mod_name does the same thing, looping through players and comparing the player's getEntityNumber to some ID. Especially when there are client IDs already built into the game and when you could simply just pass the player as a parameter and reduce that function to two lines as well.

You should stop thinking about it as a whole chunk of code that you'd need someone to help you understand and instead think of it as a list of instructions. Read the code line by line. Think about what each line does and see what's being modified or accomplished with each line.

edited on Nov. 6, 2012 07:25 am by Spik3d

Maybe it's not, I don't know, I just think you could do something better for a command line mod.