[WillCz]

Hello,
I'm trying to write a plugin that mutes every player on player_death event, I managed to mute players and write this state into an array. However I would like to clean (empty) the array on round_start... Is there a solution?

Muting is performed this way:

PHP Code:

PerformMute(target)
{
  new userId = GetClientUserId(target);
    muted[mutedCount++] = userId;
    SetClientListeningFlags(target, VOICE_MUTED);
} 


I need simply clean the array so it would contain no keys and then reset the counter.

[Seta00]

for (new i = 0; i < sizeof muted; ++i) muted[i] = -1;

?

[Monkeys]

Originally Posted by Seta00 Code: for (new i = 0; i < sizeof muted; ++i) muted[i] = -1; ?

*

PHP Code:

for(new i = 0; i < sizeof(muted); i++)
muted = 0; 


If he wants to set it as new. Because new = nulled.

[WillCz]

Okay, I've got following and still won't work: (I'm getting only one message of player unmute instead of multiple which should be in the array)

PHP Code:

PerformMute(target)
{
  new userId = GetClientUserId(target);
    muted[mutedCount++] = userId;
    PrintToChatAll("DEBUG: muting player %d", userId);
}

PerformUnMute(target)
{
    new userId = GetClientUserId(target);
    PrintToChatAll("DEBUG: unmuting player %d", userId);
}

public Action:Event_PlayerDeath(Handle:event, const String:name[], bool:dontBroadcast)
{
  new client = GetClientOfUserId(GetEventInt(event, "userid"));
  if(!(GetUserFlagBits(client) & ADMFLAG_KICK))
  {
    PerformMute(client);
  }
  else
  {
    PrintToChatAll("DEBUG: %d is admin", GetEventInt(event, "userid"));
  }
  return Plugin_Continue;
}

public Action:Event_RoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{
  PrintToChatAll("Muted count: %d", mutedCount);
  for(new i = 0; i < mutedCount; i++)
  {
    PerformUnMute(GetClientOfUserId(muted[i]));
  }
  mutedCount = 0;
  
  PrintToChatAll("All players can now speak.");
  PrintToChatAll("Muted count: %d", mutedCount);
  return Plugin_Continue;
} 


My problem is that all code in roundstart won't be executed (i.e. i'm getting muted count: 8, unmuting player: xxx and nothing more - 1 unmute instead of 8 and additional display of 2 another messages)

[WillCz]

I'm getting these messages in error log, what I'm doing wrong?

L 09/09/2010 - 21:23:49: [SM] Native "GetClientUserId" reported: Client index 0 is invalid
L 09/09/2010 - 21:24:14: [SM] Plugin encountered error 15: Array index is out of bounds

[Monkeys]

How is your muted array declared?

[WillCz]

new muted[MAXPLAYERS+1];
new mutedCount = 0;

What I've seen on the server, the counter even does not reset, on one map the muted count goes to hundreds, (i.e. muted count: 918, no-one unmuted)

[Seta00]

You should remove the mute on disconnected players as well, otherwise if various muted players disconnect you'll keep getting array index out of bounds errors.

[WillCz]

Is there any way how to imitate the foreach cycle in SourcePawn? In PHP, I would write something like this:

PHP Code:

$muted = array();

foreach($muted as $key => $player)
{
  PerformUnMute($key);
}

$muted = array(); // Resetting array

// ------ client disconnect -------
function onClientDisconnect($client)
{
  global $muted;
  $key = getUserIdOfClient($client)
  unset($muted[$key]);
}

// ------ death -------
function Event_PlayerDeath($client)
{
  global $muted;
  $userid = GetClientUserId($client);
  $muted[$userid] = TRUE;
  PerformMute($client);
} 


Also, it is possible to use sizeof function on arrays?

[Seta00]

Originally Posted by WillCz Is there any way how to imitate the foreach cycle in SourcePawn? In PHP, I would write something like this: PHP Code: $muted = array(); foreach($muted as $key => $player) {   PerformUnMute($key); } $muted = array(); // Resetting array

No, use a normal for and index the array properly:

PHP Code:

for (new i = 0; i < sizeof array; ++i) {
    PerformMute(array[i]);
} 


Also, it is possible to use sizeof function on arrays?

Yes, see code above.