[EmJay88]

Hello everyone,

i'm new in programming and i think i use the GetClientName function the wrong way. The ID is different but the name is always the same. What can i do to print all Names? I need this for a more complex plugin and i have already filtered out that it is due to this problem. Thanks a lot!

PHP Code:

public OnPluginStart()
{
    HookEvent("round_start", EventRoundStart);
}

public EventRoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{

for(new i = 1; i <= MaxClients; i++)
    {
        new String:name[MAX_NAME_LENGTH];        
        PrintToChatAll("Name: %N", GetClientName(i, name, sizeof(name)));
        PrintToChatAll("Index: %d", i);
    }
} 


[Natsheh]

It's incorrect forum but here you go. I'm also new to sourcepawn.

    for(new i = 1, String:name[MAX_NAME_LENGTH]  ; i <= MaxClients; i++)
    {       
       if(  GetClientName(i, name, sizeof(name)-1)  ) {
              PrintToChatAll("ID #%i Name: %s", i, name);
         }
    }

[Bacardi]

GetClientName return value is bool, 1 or 0

bool GetClientName(int client, char[] name, int maxlen)

https://sm.alliedmods.net/new-api/clients/GetClientName


Here is three example, works same way

PHP Code:

    char buffer[MAX_NAME_LENGTH]; // string where store text

    GetClientName(client, buffer, sizeof(buffer));
    PrintToChatAll("Name - %s", buffer);

    // https://wiki.alliedmods.net/Format_Class_Functions_(SourceMod_Scripting)#Format_Specifiers
    Format(buffer, sizeof(buffer), "%N", client);
    PrintToChatAll("Name - %s", buffer);

    PrintToChatAll("Name - %N", client); 


And here if loop, you should check first, is client in game before continue code.

PHP Code:

    for(int client = 1; client <= MaxClients; client++)
    {
        // skip client indexes, which is not in-game
        if(!IsClientInGame(client)) continue;
        
        
        PrintToChatAll("Index %i - Name %N", client, client);
    } 


[MAGNAT2645]

You can just pass client index for %N:

for (int i = 1; i <= MaxClients; i++) {
	if ( IsClientConnected( i ) )
		PrintToChatAll( "Index: %i | Name: %N", i, i );
}

[FAQU]

If you're going to use the format specifier %N, then you should give it a client index.
If you want to use GetClientName, then you should use the specifier %s.

Example:

PHP Code:

for(int i = 1; i <= MaxClients; i++)
    if(IsClientConnected(i))
    {
        PrintToChatAll("Name: %N", i);
        PrintToChatAll("Index: %d", i);
    } 


https://wiki.alliedmods.net/Format_C...eMod_Scripting)