Правильна ли созданна система Анти БанниХоп

**Пельмень** · 09.09.2014, 19:53

Сообщение от DeimoS

А если и с таймером, то точно не на pVar

PHP код:


#define PRESSED(%0) (((newkeys & (%0)) == (%0)) && ((oldkeys & (%0)) != (%0)))
new anti_bh_time[MAX_PLAYERS char];


if(PRESSED( KEY_JUMP ) && newkeys & KEY_SPRINT) 
{ 
    if(!IsPlayerInAnyVehicle(playerid)  && anti_bh_time{playerid} == 0)
    { 
        ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 ); 
        anti_bh_time{playerid} = 1;
        SetTimerEx("AntiBH", 500, 0, "d", playerid); 
     } 
    return 1; 
} 


forward AntiBH(playerid);  
public AntiBH(playerid) 
{
    ApplyAnimation( playerid, "PED", "getup_front", 4.1, 0, 0, 0, 0, 0 ); 
    anti_bh_time{playerid} = 0;  
    return 1;
}

Но так вы всё равно не исправите баг с тем, когда люди будут использовать ваш фикс при падении с высоты и выживать

- - - Добавлено - - -

А с отловом индексов код будет выглядеть как-то так

PHP код:


switch(GetPlayerAnimationIndex(playerid))
{
    case 1195, 1196, 1197:
    {
        ClearAnimations(playerid, 0);
        ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
        SetTimerEx("AntiBH", 500, 0, "d", playerid);
        return 1;
    }
}

Свитчить функцию? ата-та так делать :)

**Salvacore** · 09.09.2014, 20:49

Сообщение от Пельмень

Свитчить функцию? ата-та так делать :)

А почему нет?

**Пельмень** · 09.09.2014, 21:15

Сообщение от Salvador

А почему нет?

А зачем вызывать функцию каждый раз? Это лишняя затрата времени, если записать данные в переменную, перебор будет значительно быстрее!

**MR_BEN** · 09.09.2014, 21:22

Сообщение от Пельмень

А зачем вызывать функцию каждый раз? Это лишняя затрата времени, если записать данные в переменную, перебор будет значительно быстрее!

При каждом вызове switch будет использоваться функция GetPlayerAnimationIndex?

**L0ndl3m** · 09.09.2014, 21:37

Сообщение от Пельмень

А зачем вызывать функцию каждый раз? Это лишняя затрата времени, если записать данные в переменную, перебор будет значительно быстрее!

Если честно, не знал, что несколько раз вызывается функция. Обычно было предположение, что компилятор сам создаёт "переменную" при начале проверки случаев, а далее её проверяет.

Сообщение от MR_BEN

При каждом вызове switch будет использоваться функция GetPlayerAnimationIndex?

Сейчас проверил тесты скорости, действительно...

**DeimoS** · 09.09.2014, 21:37

Сообщение от Пельмень

А зачем вызывать функцию каждый раз? Это лишняя затрата времени, если записать данные в переменную, перебор будет значительно быстрее!

Таки не вводите людей в заблуждение. Switch вызывает функцию 1 раз и уже работает с результатом. Даже если мы введём 100 членов для сравнения.
Вот вам банальный код как пример:

PHP код:


stock GetPlayerAnimationIndexEx(playerid)
{
    print("\nФункция GetPlayerAnimationIndex вызвана\n");
    return GetPlayerAnimationIndex(playerid);
}
#define GetPlayerAnimationIndex GetPlayerAnimationIndexEx

public OnPlayerConnect(playerid)
{
    switch(GetPlayerAnimationIndex(playerid))
    {
         case 1195, 1196, 1197:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 12, 123, 357:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 11, 5, 65:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
    }
    switch(GetPlayerAnimationIndex(playerid))
    {
         case 1195, 1196, 1197:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 12, 123, 357:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 11, 5, 65:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
    }
    return 1;
}

В консоль выдаст ровно 2 сообщения. Когда такой код:

PHP код:


stock GetPlayerAnimationIndexEx(playerid)
{
    print("\nФункция GetPlayerAnimationIndex вызвана\n");
    return GetPlayerAnimationIndex(playerid);
}
#define GetPlayerAnimationIndex GetPlayerAnimationIndexEx

public OnPlayerConnect(playerid)
{
    if(GetPlayerAnimationIndex(playerid) == 1195 || GetPlayerAnimationIndex(playerid) == 1196 || GetPlayerAnimationIndex(playerid) == 1197)
    {
        ClearAnimations(playerid, 0);
        ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
        SetTimerEx("AntiBH", 500, 0, "d", playerid);
        return 1;
    }
    return 1;
}

Естественно выдаст 3. Хотя может вы говорите не о том...

**Пельмень** · 09.09.2014, 21:41

Сообщение от MR_BEN

При каждом вызове switch будет использоваться функция GetPlayerAnimationIndex?

Именно! Доступ к переменной, в которой хранится значение, которое вернула функция, будет осуществляться в разы быстрее, также, можно записывать в переменную данные из одномерных, двумерных и т.д, массивов, для получения результата. т.к к одномерным, двумерным и т.д массивам доступ осуществляется медненнее, чем к обыкновенным переменным.

**DeimoS** · 09.09.2014, 21:48

Сообщение от Londlem

Сейчас проверил тесты скорости, действительно...

Не знаю как вы проверяли... Вот вам мой тест. При записи в переменную время увеличилось (оно и понятно, ибо 2 новых действия добавляем ко всему)

PHP код:


public OnPlayerConnect(playerid)
{
    new time = GetTickCount();
    for(new i; i != 1000000; i++)
    {
        switch(GetPlayerAnimationIndex(playerid))
        {
             case 1195, 1196, 1197:
            {
                ClearAnimations(playerid, 0);
                ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
                SetTimerEx("AntiBH", 500, 0, "d", playerid);
                return 1;
            }
            case 12, 123, 357:
            {
                ClearAnimations(playerid, 0);
                ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
                SetTimerEx("AntiBH", 500, 0, "d", playerid);
                return 1;
            }
            case 11, 5, 65:
            {
                ClearAnimations(playerid, 0);
                ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
                SetTimerEx("AntiBH", 500, 0, "d", playerid);
                return 1;
            }
        }
    }
    printf("Цикл 1 сработал за %d мс", GetTickCount()-time);
    
    time = GetTickCount();
    for(new i; i != 1000000; i++)
    {
        new index = GetPlayerAnimationIndex(playerid);
        switch(index)
        {
             case 1195, 1196, 1197:
            {
                ClearAnimations(playerid, 0);
                ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
                SetTimerEx("AntiBH", 500, 0, "d", playerid);
                return 1;
            }
            case 12, 123, 357:
            {
                ClearAnimations(playerid, 0);
                ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
                SetTimerEx("AntiBH", 500, 0, "d", playerid);
                return 1;
            }
            case 11, 5, 65:
            {
                ClearAnimations(playerid, 0);
                ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
                SetTimerEx("AntiBH", 500, 0, "d", playerid);
                return 1;
            }
        }
    }
    printf("Цикл 2 сработал за %d мс\n", GetTickCount()-time);
    return 1;
}

**Пельмень** · 09.09.2014, 21:53

Сообщение от DeimoS

Таки не вводите людей в заблуждение. Switch вызывает функцию 1 раз и уже работает с результатом. Даже если мы введём 100 членов для сравнения.
Вот вам банальный код как пример:

PHP код:


stock GetPlayerAnimationIndexEx(playerid)
{
    print("\nФункция GetPlayerAnimationIndex вызвана\n");
    return GetPlayerAnimationIndex(playerid);
}
#define GetPlayerAnimationIndex GetPlayerAnimationIndexEx

public OnPlayerConnect(playerid)
{
    switch(GetPlayerAnimationIndex(playerid))
    {
         case 1195, 1196, 1197:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 12, 123, 357:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 11, 5, 65:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
    }
    switch(GetPlayerAnimationIndex(playerid))
    {
         case 1195, 1196, 1197:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 12, 123, 357:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
        case 11, 5, 65:
        {
            ClearAnimations(playerid, 0);
            ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
            SetTimerEx("AntiBH", 500, 0, "d", playerid);
            return 1;
        }
    }
    return 1;
}

В консоль выдаст ровно 2 сообщения. Когда такой код:

PHP код:


stock GetPlayerAnimationIndexEx(playerid)
{
    print("\nФункция GetPlayerAnimationIndex вызвана\n");
    return GetPlayerAnimationIndex(playerid);
}
#define GetPlayerAnimationIndex GetPlayerAnimationIndexEx

public OnPlayerConnect(playerid)
{
    if(GetPlayerAnimationIndex(playerid) == 1195 || GetPlayerAnimationIndex(playerid) == 1196 || GetPlayerAnimationIndex(playerid) == 1197)
    {
        ClearAnimations(playerid, 0);
        ApplyAnimation( playerid, "PED", "KO_skid_back", 4.1, 0, 0, 0, 0, 0 );
        SetTimerEx("AntiBH", 500, 0, "d", playerid);
        return 1;
    }
    return 1;
}

Естественно выдаст 3. Хотя может вы говорите не о том...

Это аналог if, else if, else if (function() == 1) else if (function() == 2) else if (function() == 3) - функция будет вызвана 3 раза, также и в свитче, если я не ошибаюсь, конечно же.

**Salvacore** · 09.09.2014, 21:59

Сообщение от Пельмень

А зачем вызывать функцию каждый раз? Это лишняя затрата времени, если записать данные в переменную, перебор будет значительно быстрее!

Пасиб.

Тема: Правильна ли созданна система Анти БанниХоп

Опции темы

Поиск по теме

Информация о теме

Пользователи, просматривающие эту тему

Ваши права