[NWayne]

Hello Guys,

I've got some kind of problem with my stock, but i cannot understand what and where is problem?
I am glad if you help me <3

stock int GetRandomPlayerFromTT()
{
    int clients[MaxClients + 1];
    int clientCount;
    for (int i = 1; i <= MaxClients; i++)
    	if (IsClientInGame(i) && (GetClientTeam(i) == CS_TEAM_T))
      	  clients[clientCount++] = i;
    return (clientCount == 0) ? -1 : clients[GetRandomInt(0, clientCount - 1)];
}

error 161: brackets after variable name indicate a fixed-size array, but a dynamic size was given - did you mean to use 'new int[size]' syntax?

[backwards]

You have to replace "MaxClients" with "MAXPLAYERS". MaxClients is a variable that isn't defined until a server is booted up. MAXPLAYERS is predefined as 65 or 64. This way it has a static value that isn't changing. Since the compiler compiles code before its loaded it can't have a dynamic value define the array size.

[NWayne]

Thank you the topic is solved.

[xerox8521]

It is actually possible to declare variables with a dynamic size as of 1.7

https://wiki.alliedmods.net/Introduc...#Declaration_2

[Fyren]

Originally Posted by xerox8521 It is actually possible to declare variables with a dynamic size as of 1.7 https://wiki.alliedmods.net/Introduc...#Declaration_2

You could create dynamic arrays before 1.7. The old syntax for doing so was replaced in 1.7 and made an error (the one in the original post).

[ThatOneGuy]

I'd suggest using an ADT array ("arraylist" in new syntax) for this. Push the client user ID's to an array, get the size of the array, then get a random int from 0 to (size - 1). Convert the user IDs back to client index.

[eyal282]

Originally Posted by ThatOneGuy I'd suggest using an ADT array ("arraylist" in new syntax) for this. Push the client user ID's to an array, get the size of the array, then get a random int from 0 to (size - 1). Convert the user IDs back to client index.

Terrible idea to sell out up to 31 cells and buy lag.

[xerox8521]

Originally Posted by Fyren You could create dynamic arrays before 1.7. The old syntax for doing so was replaced in 1.7 and made an error (the one in the original post).

Didn't know about that. But we are not talking about CreateArray right ?

[Fyren]

Originally Posted by xerox8521 Didn't know about that. But we are not talking about CreateArray right ?

I am not.