I make this script. I'm trying to make the player id random but do not know which way...

My script random "id" player does not work.

Funkcja()
{
	array = [];
	array[array.size] = self.clientid = 0;
	array[array.size] = self.clientid = 1;
	
	players = getentarray("player", "classname");
	for(i = 0; i < players.size; i++)
	{
		player = players[i];
		
		if(player.clientid = 0)
		{
			player iprintlnbold("Give Item");
		}
		
		break;
	}
}

Someone help me odnalezeniu this way?

Well that break at the end of your for loop should be removed. Breaks break out of for and while loops and jump to the end of the for loop. So with your script, you're only going through that loop for player[0].

I'm not sure what you're trying to do with the array though, the array isn't used anywhere after you initialize it and give it two elements.

array = [];
array[array.size] = self.clientid = 0;
array[array.size] = self.clientid = 1;

I don't know whether you can or cannot do this with CoD's scripting language, but you have two declaration statements on the same lines (array[array.size] = self.clientid = 0;). If it is supported though, who is self in this scope and why are you setting the same self's clientid to 0, then immediately afterwards to 1?

You also shouldn't be trying to print like this. That function doesn't have support to be called on a player in CoD2 AFAIK.

player iprintlnbold("Give Item");

This is for CoD4, it's probably supported in CoD2, but you should use ClientPrint(player, message) instead.

You also don't need to do this, you can just use player[i] instead of player for the rest of the loop and it will behave the same.

player = players[i];

But lastly, I don't see anything generating random numbers for players. I don't even see a random number generator being used in your function there. I'm not entirely sure what you're trying to do though, I wouldn't rely on random numbers to distinguish players between each other if you'd be using that id to print something to their screen.

If you want to print "Give Item" to their screen after they give or are given an item, you should print it in your script soon after the actual action is being done so you can reference the player who was given or gave an item and print it to their screen without any random numbers or looping involved.

iprintlnbold is supported in CoD2. An addon to the list of things wrong with this code is also

if(player.clientid = 0)

which should be

if(player.clientid == 0)

since you have to compare two values in an if statement.

Damian17 writes...

You cannot change a client's ID. It is given to a player according to the order in which they connect to a server. Even if they leave, their ID is not then given to someone else joining the server.

Give up the idea of changing a clien't ID, and find some other way to find the player you are looking for.

you could just simply make a custom var to take place instead of using "clientid" when they connect. so when they connect you could do.

level.idNums = 0; // put this in your main func so it dosnt keep reseting its self to 0

players.customID = level.idNums; // put when they connect
level.idNums++; // put when they connect

then you can monitor when they disconnect and put level.idNums--; so everything will stay neat and in order.

level.idNums = 0; - zero means that the player no have on the server, right?

Damian17 writes...

im sorry i dont know what your trying to say? level.idNums is just a custom var i came up with that holds an int of 0. so your first person to connect will hold that memory. then next will be 1 ect ect. its just like clientid but you are aloud to change the players level.idNums.