[kasikadam]

I want to make a plugin, when a player types !join, plugin will store players userid in an array and randomly select 2 userids. I wrote this piece of code but when i try to compile i get this error :
error 035: argument type mismatch (argument 2) at PushArrayString line

public Action Join(client, args)
{
	CPrintToChat(client, "Okay..");
	new String:struserid = GetClientUserId(client);
	PushArrayString(arr, struserid);
	return Plugin_Continue;
}

[CrazyHackGUT]

Use PushArrayCell() instead PushArrayString(). And don't try convert int to string.

[8guawong]

public Action Join(client, args)
{
	CPrintToChat(client, "Okay..");
	new struserid = GetClientUserId(client);
	PushArrayCell(arr, struserid);
	return Plugin_Continue;
}

[kasikadam]

Thank you it works now but players can join multiple times how can i prevent this?

[8guawong]

Originally Posted by kasikadam Thank you it works now but players can join multiple times how can i prevent this?

you loop through the array and check if userid is in already in it or not
if not then PushArrayCell

[kasikadam]

Originally Posted by 8guawong you loop through the array and check if userid is in already in it or not if not then PushArrayCell

something like that ?

	new abc;
	for (abc = 1; abc <= GetArraySize(arr); ++abc){
		if FindValueInArray(arr, strone) == -1{
			PushArrayCell(arr, strone);
		}
	}

[CrazyHackGUT]

No. Just use FindValueInArray().

Something like this:

public Action Join(client, args)
{
	int iUserID = GetClientUserId(client);
	if (FindValueInArray(arr, iUserID) == -1)
	{
		CPrintToChat(client, "Okay..");
		PushArrayCell(arr, iUserID);
	} else {
		CPrintToChat(client, "Not okay. You already joined.");
	}
	return Plugin_Handled;
}

[8guawong]

Originally Posted by CrazyHackGUT No. Just use FindValueInArray(). Something like this: public Action Join(client, args) { int iUserID = GetClientUserId(client); if (FindValueInArray(arr, iUserID) == -1) { CPrintToChat(client, "Okay.."); PushArrayCell(arr, iUserID); } else { CPrintToChat(client, "Not okay. You already joined."); } return Plugin_Handled; }

didn't know about FindValueInArray()

[kasikadam]

Originally Posted by CrazyHackGUT No. Just use FindValueInArray(). Something like this: public Action Join(client, args) { int iUserID = GetClientUserId(client); if (FindValueInArray(arr, iUserID) == -1) { CPrintToChat(client, "Okay.."); PushArrayCell(arr, iUserID); } else { CPrintToChat(client, "Not okay. You already joined."); } return Plugin_Handled; }

makes more sense thank you !

[eyal282]

Originally Posted by kasikadam makes more sense thank you !

Don't forget:

public OnClientDisconnect(client)
{
new slot = FindValueInArray(Array, GetClientUserId(client));

if(slot != -1)
RemoveFromArray(Array, slot);
}

Note that I may mistaken the names of the functions.