Login

OFFREAL · 17.03.2011, 00:38

Здравствуйте!
- - -
Появилась идея оптимизировать циклы вида:

PHP Code:



for(new i=0;i<MAX_PLAYERS;i++)

{ ... }

Сначала (после #inclide <a_samp>) писал:

PHP Code:



#if defined MAX_PLAYERS

#undef MAX_PLAYERS

#define MAX_PLAYERS 100

#else

#define MAX_PLAYERS 100

#endif

Это конечно давало некоторый прирост, но я пошел еще дальше!
- - -
К сожалению, в pawn нельзя создавать динамические массивы, поэтому я придумал использовать что-то вроде псевдодинамического массива для хранения ид подключенных игроков. Создал глобальный масив:

PHP Code:



new PLIDs[MAX_PLAYERS];

Далее я ввел переменную содержащую количество игроков на сервере:

PHP Code:



new MaxPlayers = 0;

Очищаю весь массив PLIDs в OnGameModInit:

PHP Code:



for(new i=0;i<sizeof(PLIDs);i++){PLIDs[i] = -1;}

В паблик OnPlayerConnect добавил необходимые действия:

PHP Code:



PLIDs[MaxPlayers] = playerid;

MaxPlayers++;

Игрок подключается, и его ИД помещается в первый свободный слот массива PLIDs, ну а переменная MaxPlayers становится на единицу больше.
- - -
Паблик OnPlayerDisconnect - тут все немного сложнее. Игрок выходит, а значит слот в PLIDs котором записан его ИД надо очистить. Но мне не нужен пустой слот в середине массива, поэтому я присваиваю ему значение самого последнего занятого слота массиваPLIDs, а вот последний уже очищаю:

PHP Code:



for(new i=0;i<MaxPlayers;i++)

{

    if(PLIDs[i] == playerid)

    {

        MaxPlayers--;

        PLIDs[i] = PLIDs[MaxPlayers];

        PLIDs[MaxPlayers] = -1;

        break;

    }

}

- - -
Использую как-то так:
Пусть был массив:

PHP Code:



for(new i=0;i<MAX_PLAYERS;i++)

{

    if(!IsPlayerConnected(i)){continue;}

    SendClientMessage(i, -1, "Message");

}

После оптимизации:

PHP Code:



for(new i=0;i<MaxPlayers;i++)

{

    SendClientMessage(PLIDs[i], -1, "Message");

}

- - -
Плюсы данного метода:
- отсутствуют лишние такты циклов
- не требуется проверять подключен ли игрок

Минусы:
- меняется порядок ИД игроков при выполнении циклов

- - -
Очень интересно ваше мнение по поводу этого!

CrunkBankS · 17.03.2011, 05:30

Лучше юзать foreach

OKStyle · ( *Last edited by OKStyle; 17/03/2011 at 06:29 AM.* )

Поподробнее про эти динамические циклы рассказал бы кто-нибудь. Кстати, похожий код http://forum.sa-mp.com/showpost.php?...postcount=1512

Johnny_Xayc · 17.03.2011, 09:43

Вот этого вполне достаточно!

pawn Code:

#undef MAX_PLAYERS
#define MAX_PLAYERS 50 // кол-во нужных вам игроков!
for(new i = 0; i < MAX_PLAYERS; i++)
{
      if(!IsPlayerConnected(i))continue;
}

Stepashka · 17.03.2011, 10:22

Quote:

Originally Posted by CrunkBankS

Лучше юзать foreach

лучше купить мозг и не пороть чущь!
Для того что только что описал автор, достаточно 1 массива и 1 доп функции.
Ничего нового для себя в этой теме не нашел.

Romanius · 17.03.2011, 12:57

степаха опять прогоняешь. foreach построен по примерно такому же принципу, но более "оптимизирован".

OFFREAL · 17.03.2011, 13:05

Quote:

Originally Posted by Johnny_Xayc

Вот этого вполне достаточно!

pawn Code:

#undef MAX_PLAYERS
#define MAX_PLAYERS 50 // кол-во нужных вам игроков!
for(new i = 0; i < MAX_PLAYERS; i++)
{
      if(!IsPlayerConnected(i))continue;
}

Я писал, что раньше пользовался этим методом, но у него явные минусы - каждый раз выполняется проверка IsPlayerConnected и используется много лишних тактов.
- - -

Quote:

Originally Posted by Stepashka

лучше купить мозг и не пороть чущь!
Для того что только что описал автор, достаточно 1 массива и 1 доп функции.
Ничего нового для себя в этой теме не нашел.

У меня и есть один массив

Можешь привести пример доп функции?

OFFREAL · 17.03.2011, 13:10

Romanius, то-есть как-то так:

PHP Code:



foreach (new i in PLIDs)
{ 
    SendClientMessage(i, -1, "Message"); 
}

OFFREAL · 17.03.2011, 13:16

xD
в pawn же нет foreach ...

CrunkBankS · 17.03.2011, 13:54

https://sampforum.blast.hk/showthread.php?tid=92679

Stepashka · 17.03.2011, 13:56

Quote:

Originally Posted by Romanius

степаха опять прогоняешь. foreach построен по примерно такому же принципу, но более "оптимизирован".

в том-то и дело что он нифига не оптимизирован, там наоборот много лишнего.

Johnny_Xayc · 17.03.2011, 14:49

Quote:

Originally Posted by Stepashka

в том-то и дело что он нифига не оптимизирован, там наоборот много лишнего.

Согласен.

Romanius · 17.03.2011, 14:51

если ты смотрел его внимательно, то ты бы увидел сортировку, поэтому он работает немного дольше. НО сортировка отключаится, и скорость в выполнения уменьшается

Stepashka · 17.03.2011, 15:46

Quote:

Originally Posted by Romanius

если ты смотрел его внимательно, то ты бы увидел сортировку, поэтому он работает немного дольше. НО сортировка отключаится, и скорость в выполнения уменьшается

если бы ты смотрел его внимательно ты бы его не исользовал.

Makar93 · 17.03.2011, 16:10

Да бред, передефайнил MAX_PLAYERS и хватит... С учетом, что максимальное кол-во игроков - 500, даже можно и не оптимизировать.

Wyu · 17.03.2011, 19:31

Quote:

Originally Posted by Stepashka

в том-то и дело что он нифига не оптимизирован, там наоборот много лишнего.

Не оптимизирован? Это где это?

pawn Code:

// foreach.inc
#define foreach(%1,%2) \
        for (new %2 = YSI_gS%1; _:%2 != -1; %2 = YSI_gA%1[%2])

И чего тут лишнее?

Stepashka · 17.03.2011, 19:38

Quote:

Originally Posted by Wyu

Не оптимизирован? Это где это?

pawn Code:

// foreach.inc
#define foreach(%1,%2) \
        for (new %2 = YSI_gS%1; _:%2 != -1; %2 = YSI_gA%1[%2])

И чего тут лишнее?

судишь конфету по обертке

Wyu · 17.03.2011, 19:52

Если под "лишнее" имелся ввиду доп. функционал, то он вовсе не вносит никакого оверхеда, да и никто не заставляет им пользоваться

Quote:

Originally Posted by Stepashka

судишь конфету по обертке

Надеюсь пояснения будут?

MX_Master · 22.03.2011, 11:09

мои 5 копеек, мобыть кому-то понравится

PHP Code:



stock playerMaxID = 0;


public OnPlayerConnect ( playerid ) 

{

    if ( playerid > playerMaxID ) playerMaxID = playerid;

}


public OnPlayerDisconnect ( playerid, reason )

{

    if ( playerid <= playerMaxID )

    {

        for ( new i = playerid - 1; i >= 0; --i )

        {

            if ( IsPlayerConnected(i) )

            {

                playerMaxID = i;

                break;

            }

        }

    }

}


// - как использовать? 

// - вот пример


for ( new i; i <= playerMaxID; ++i )

{

    if ( IsPlayerConnected(i) )

    {

        // какое-то действие над игроком с ИДом "i"

    }

}

не проверял на ошибки, по старой памяти, должно быть правильно. 1 переменка, но какая полезная !! ((:

Miller · 09.04.2011, 16:04

MX_Master, мне... :D

Login
Username:
Password:	Lost Password?
	Remember me