[TheDiamant953]

I have a little problem with user id.

I have two plugin, the first use a Server Command:

ServerCommand("sm_test1 %d", client);

And the second plugin received it:

public Action:Command_Test1(args)
{
	if (args != 1)
	{
		return Plugin_Handled;
	}
	
	new String:arg1[10];
	GetCmdArg(1, arg1, sizeof(arg1));
	
	new argument1;
	argument1 = StringToInt(arg1);
		
	new target;
	target = GetClientOfUserId(argument1);
	
	if(target && 0 < target <= MaxClients)
	{
		if(g_eClients[target][bChefCt] == false)
		{
			g_eClients[target][bChefCt] = true;
		}
	}
	return Plugin_Continue;
}

In this case, i wan't to turn "bChefCt" to true by using the command in the first plugin with the userid of the player.

My problem: the command work, but the player in not the good one.

For example, when i use status in the console i have:
15 14 Payer1 ....
16 15 Player2 ...

When the plugin use the command on the player2, it's the player1 who is the target and not player2.

I thing i have done something wrong with GetClientOfUserId or by sending client to the command.

Can you maybe help me, it's would be really apreciate.

Thanks and have a good day.

[Kailo]

Use

ServerCommand("sm_test1 %d", GetClientUserId(client));

Client index isn't userid. Client index means online player index and can be 0-65 (0 is server).
Userid is index means player session.
Example: while timer gone, one player (with client index 1) left the game, and another connected (and get index 1 because it's free). After timer you will get wrong target. But userid for 1st player is n, and n+1 for 2nd.

[TheDiamant953]

Originally Posted by Kailo Use ServerCommand("sm_test1 %d", GetClientUserId(client)); Client index isn't userid. Client index means online player index and can be 0-65 (0 is server). Userid is index means player session. Example: while timer gone, one player (with client index 1) left the game, and another connected (and get index 1 because it's free). After timer you will get wrong target. But userid for 1st player is n, and n+1 for 2nd.

But in the second code quote, i take the GetClientUserId, why it dosn't work ?

And what is the differences betwen GetClientUserId and GetClientOfUserId ?

Thanks

[ddhoward]

Originally Posted by TheDiamant953 And what is the differences betwen GetClientUserId and GetClientOfUserId ?

One is the opposite of the other. One gives you the client index using the user ID, and the other gives you the user ID using the client index.

[Kailo]

Originally Posted by TheDiamant953 But in the second code quote, i take the GetClientUserId, why it dosn't work ? And what is the differences betwen GetClientUserId and GetClientOfUserId ? Thanks

When you run command you pass client index and next try to get client index from userid, but you used client index as userid.
Another way just remove

target = GetClientOfUserId(argument1);

because you already pass client index and don't need convert userid in client index.
But pass userid more safety.

[Powerlord]

If you're writing a command that takes users as an argument, please use FindTarget (for single-target commands) or ProcessTargetString.

[asherkin]

Originally Posted by Powerlord If you're writing a command that takes users as an argument, please use FindTarget (for single-target commands) or ProcessTargetString.

And if you're just creating commands for inter-plugin communication, create natives instead.

[TheDiamant953]

Thanks for your help.

I don't think i will use FindTaget or ProcessTargetString, i don't understand them, i'm not good at all in scripting and in english :p

But now it's work. Thanks.

[thecount]

Originally Posted by TheDiamant953 I don't think i will use FindTaget or ProcessTargetString, i don't understand them, i'm not good at all in scripting and in english :p\

FindTarget is more simple and manageable for new scripters. It works like this:

PHP Code:

FindTarget(client, String:target[], bool:nobots, bool:immunity); 


client = client who issued command.
target = String of target to search for by name. In your case, the argument string.
nobots = True for no bots. False otherwise.
immunity = True for admin immunity. You get the idea.

HELPFUL TOOL: https://sm.alliedmods.net/new-api/