Solved client_disconnected problem?

[PurposeLess]

Hello,
I want to change index after someone left the game but sometimes bug happens.
I will give an example and you'll see the problem.

Happens;

(He dropped)
empty dropped.
empty has left the game.

(Next terrorist did not change. It was still empty even if he left the server.)
empty left the game, next terrorist is empty.

new pNextTerrorist;

public client_disconnected(pPlayer) {
    new iPlayers[32], iNum;
    get_players(iPlayers, iNum);
    //... inum checks

    if(pNextTerrorist == pPlayer) {
        pNextTerrorist = iPlayers[random(iNum)];
        rg_set_user_team(pNextTerrorist, TEAM_TERRORIST);
        client_print_color(0, 0, "%L", LANG_PLAYER, "REDEATHRUN_LEFT_GAME", pPlayer, pNextTerrorist);
    }
}

 

[BlackSignature]

В рамках 'disconnected' игрок ещё считается подключённым т.е. выборка get_players() включает его в массив. Понимаешь? Тебе нужно проверять что рандом выбрал не его же.

 

[Nordic Warrior]

while(pNextTerrorist != pPlayer)
    pNextTerrorist = iPlayers[random(iNum)];

 

[fl0wer]

Nordic Warrior, вечный луп же

30 Янв 2021

Ручками в цикле собрать нужных игроков и рэндмить уже готовых игроков.

 

[PurposeLess]

Ручками в цикле собрать нужных игроков и рэндмить уже готовых игроков.

How to do that? I'm not sure that I understand because translate is kinda bad from ru to en.

 

[fl0wer]

PurposeLess, Collect players in owned array at loop instead of get_players.

 

[PurposeLess]

Does it work?

public client_disconnected(pPlayer) {
    pNextTerrorist = RandomPlayer(pPlayer);
}

RandomPlayer(const NotIncludedPlayer) {
    ArrayClear(g_aConnectedPlayers);

    for(new pPlayer = 1; pPlayer <= MaxClients; pPlayer++) {
        if(!is_user_connected(pPlayer) || pPlayer == NotIncludedPlayer) { // I think I don't need NotIncludedPeople here. First checks too.
            continue;
        }

        ArrayPushCell(g_aConnectedPlayers, pPlayer);
    }

    return ArrayGetCell(g_aConnectedPlayers, random(ArraySize(g_aConnectedPlayers))); // or random_num(1, ...) ?
}

 

[BaHeK]

new pNextTerrorist;

public client_disconnected(pPlayer) {
    new iPlayers[32], iNum;
    get_players(iPlayers, iNum);
    //... inum checks
    if(iNum == 1 || pNextTerrorist != pPlayer) {
        return;
    }
    
    while (pNextTerrorist == pPlayer) {
        pNextTerrorist = iPlayers[random(iNum)];
    }

    rg_set_user_team(pNextTerrorist, TEAM_TERRORIST);
    client_print_color(0, 0, "%L", LANG_PLAYER, "REDEATHRUN_LEFT_GAME", pPlayer, pNextTerrorist);
}

 

[PurposeLess]

BaHeK,

вечный луп же

 

[d3m37r4]

Создайте вы массив, циклом for последовательно сохраните в этот массив индексы, исключая индекс игрока, который отсоединился от сервера и работайте дальше с этим массивом, как сказал fl0wer

 

[PurposeLess]

Создайте вы массив, циклом for последовательно сохраните в этот массив индексы, исключая индекс игрока, который отсоединился от сервера и работайте дальше с этим массивом, как сказал fl0wer

How about that client_disconnected problem? | Dev-CS.ru (dev-cs.ru) ?

 

[d3m37r4]

PurposeLess, why not?

 

[fl0wer]

    new playersNum;
    new playersArray[MAX_PLAYERS];

    for (new i = 1; i <= MaxClients; i++)
    {
        if (i == id)
            continue;

        if (!is_user_connected(i))
            continue;

        playersArray[++playersNum] = i;
    }

    if (playersNum)
    {
        new nextTerrorist = playersArray[random_num(0, playersNum - 1)];
        // actions
    }

 

[PurposeLess]

fl0wer, that's great tho.