Multi-Jump (mj) 1.6.1

SergeyShorokhov · 5 Мар 2019

WILL_BE, ты точно с пониманием пытаешься обсудить "ревью кода" от sbelov020 ? Кажется, что нет.

5 Мар 2019

WILL_BE написал(а):
Эти слова-воспоминания от опытных людей, не более

И ты их не правильно понял.

WILL_BE · 5 Мар 2019

wopox1337, а что там не понятного? Было дело я поинтересовался про такую оптимизацию, на что получил соответствующий ответ. Я не утверждаю, что меня верно направили, просто говорю то, чему меня "обучили". Учту на будущее, благодарю :ok:

wellasgood · 5 Мар 2019

Еще раз повторюсь, код был написан давно, с той логикой, которую вкладывал на тот момент написания..

Возможно в следующих версиях будет уже по другому.

wellasgood · 7 Мар 2019

WILL_BE,

Вы писали:

WILL_BE написал(а):

wellasgood, и по поводу куска:

Код:

    if(get_user_flags(id) & ADMINCHECK){
        CheckProtectADMIN[id] = 500      // Установка кол-ва прыжков для Админов
    }

    if(get_user_flags(id) & VIPCHECK){
        CheckProtectVIP[id] = 250      // Установка кол-ва прыжков для Вип
    }

    if(get_user_flags(id) & USERCHECK){
        CheckProtectUSER[id] = 100      // Установка кол-ва прыжков для Юзеров
    }

Что если, игрок имеет оба флага?

А, Вы правы, возможно будут косяки, в следующем обновлении поправлю.

Необходимо сделать выход, если флаг игрока будет равняться флага в коде, а не идти дальше искать флаги. Ну и динамичности (гибкости) добавить.

Tranquillity · 11 Апр 2019

wellasgood,

Скрытое содержимое для пользователей: wellasgood

wellasgood · 11 Апр 2019

Ресурс Multi-Jump (mj) был обновлен:

Обновление до версии (1.5)

Нажмите на эту ссылку, чтобы узнать подробности обновления

wellasgood · 11 Апр 2019

Tranquillity, по ошибке закинул, там другой актуальный. А пересоздавать сразу версию не стал..

11 Апр 2019

Tranquillity, пересоздал обновление с учетом Ваших замечаний!

Tranquillity · 11 Апр 2019

wellasgood,

Скрытое содержимое для пользователей: wellasgood

wellasgood · 11 Апр 2019

Tranquillity, спасибо, учту в следующих (обновлениях) работах..

Limbooc · 11 Апр 2019

Код:

if(get_user_flags(id) & ADMINCHECK){
        CheckProtectADMIN[id] = 100      // Установка кол-ва прыжков для Админов
    }

    if(get_user_flags(id) & VIPCHECK){
        CheckProtectVIP[id] = 50      // Установка кол-ва прыжков для Вип
    }

    if(get_user_flags(id) & USERCHECK){
        CheckProtectUSER[id] = 20      // Установка кол-ва прыжков для Юзеров
    }

===>

Код:

if(get_user_flags(id) & USERCHECK)
{
    CheckProtect[id] = 20      // Установка кол-ва прыжков для Юзеров
}
else
if(get_user_flags(id) & VIPCHECK)
{
    CheckProtect[id] = 50      // Установка кол-ва прыжков для Вип
}
else
if(get_user_flags(id) & ADMINCHECK
{
    CheckProtect[id] = 100      //  Установка кол-ва прыжков для Админов
}

а дальше

Код:

if(get_user_flags(id) & ADMINCHECK)
    {

        if(CheckProtectADMIN[id] == 0){
            client_print_color(id, print_team_default, "%L", id, "MJ_STOP_ADMIN")
            set_task(30.0,"func_info")
            return PLUGIN_HANDLED;
        }

        if(OLDzq != NEWzq){
            client_print_color(id, print_team_default, "%L", id, "MJ_ALREADY_ACTIVE")
            return PLUGIN_HANDLED;
        }else if(OLDzq == NEWzq){
            client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_1", zName)
            client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_2", CheckProtectADMIN[id])
            activate[id] = 1
            qq++;
            NEWzq = qq
            set_task(30.0,"func_info")
            return PLUGIN_HANDLED;
        }
        return PLUGIN_HANDLED;
    }

    if(get_user_flags(id) & VIPCHECK)
    {
        if(CheckProtectVIP[id] == 0){
            client_print_color(id, print_team_default, "%L", id, "MJ_STOP_VIP_1")
            client_print_color(id, print_team_default, "%L", id, "MJ_STOP_VIP_2")
            set_task(30.0,"func_info")
            return PLUGIN_HANDLED;
        }

        if(OLDzq != NEWzq){
            client_print_color(id, print_team_default, "%L", id, "MJ_ALREADY_ACTIVE")
            return PLUGIN_HANDLED;
        }else if(OLDzq == NEWzq){
            client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_1", zName)
            client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_2", CheckProtectVIP[id])
            activate[id] = 1
            qq++;
            NEWzq = qq
            set_task(30.0,"func_info")
            return PLUGIN_HANDLED;
        }
        return PLUGIN_HANDLED;
    }

    if(get_user_flags(id) & USERCHECK)
    {

        if(CheckProtectUSER[id] == 0){
            client_print_color(id, print_team_default, "%L", id, "MJ_STOP_USER_1")
            client_print_color(id, print_team_default, "%L", id, "MJ_STOP_USER_2")
            set_task(30.0,"func_info")
            return PLUGIN_HANDLED;
        }

        if(OLDzq != NEWzq){
            client_print_color(id, print_team_default, "%L", id, "MJ_ALREADY_ACTIVE")
            return PLUGIN_HANDLED;
        }else if(OLDzq == NEWzq){
            client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_1", zName)
            client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_2", CheckProtectUSER[id])
            activate[id] = 1
            qq++;
            NEWzq = qq
            set_task(30.0,"func_info")
            return PLUGIN_HANDLED;
        }
        return PLUGIN_HANDLED;
    }

====>>

Код:

if(CheckProtect[id] == 0)
    {
        client_print_color(id, print_team_default, "%L", id, "MJ_STOP_VIP_1")
        client_print_color(id, print_team_default, "%L", id, "MJ_STOP_VIP_2")
        set_task(30.0,"func_info")
        return PLUGIN_HANDLED;
    }

    if(OLDzq != NEWzq)
    {
        client_print_color(id, print_team_default, "%L", id, "MJ_ALREADY_ACTIVE")
        return PLUGIN_HANDLED;
    }else if(OLDzq == NEWzq){
        client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_1", zName)
        client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_2", CheckProtect[id])
        activate[id] = 1
        qq++;
        NEWzq = qq
        set_task(30.0,"func_info")
        return PLUGIN_HANDLED;
    }

У тебя три функции одинаковы за исключение одной ячейки, все это можно сделай одной
и на последок

Код:

public client_disconnect(id)
{
    jumpnum[id] = 0
    dojump[id] = false
    activate[id] = 0

    if(get_user_flags(id) & ADMINCHECK)
    {
        CheckProtectADMIN[id] = 100
    }

    if(get_user_flags(id) & VIPCHECK)
    {
        CheckProtectVIP[id] = 50
    }

    if(get_user_flags(id) & USERCHECK)
    {
        CheckProtectUSER[id] = 20
    }
}

Собственно зачем выдавать mj для отключившийся?
===>

Код:

public client_disconnect(id)
{
    jumpnum[id] = 0
    dojump[id] = false
    activate[id] = 0
    CheckProtect[id] = 0
}

wellasgood · 11 Апр 2019

Limbooc, с половиной не согласен, вы не поняли моей сути в некоторых моментах, остальное переделаю в глобальной обнове, спасибо за критику.

11 Апр 2019

Limbooc написал(а):

Код:

if(get_user_flags(id) & ADMINCHECK){
        CheckProtectADMIN[id] = 100      // Установка кол-ва прыжков для Админов
    }

    if(get_user_flags(id) & VIPCHECK){
        CheckProtectVIP[id] = 50      // Установка кол-ва прыжков для Вип
    }

    if(get_user_flags(id) & USERCHECK){
        CheckProtectUSER[id] = 20      // Установка кол-ва прыжков для Юзеров
    }

===>

Код:

if(get_user_flags(id) & USERCHECK)
{
    CheckProtect[id] = 20      // Установка кол-ва прыжков для Юзеров
}
else
if(get_user_flags(id) & VIPCHECK)
{
    CheckProtect[id] = 50      // Установка кол-ва прыжков для Вип
}
else
if(get_user_flags(id) & ADMINCHECK
{
    CheckProtect[id] = 100      //  Установка кол-ва прыжков для Админов
}

Смотрите, здесь просто заводятся счетчики! Тут нету выполнительных действий. Тут неважно. Т.е если флаг админа завдится счетчик на 100, если еще и вип о заведется и на 50 например, т.е 2 счетчика для игрока будут заведены, но дальше при проверки в других функциях будет фигурировать лишь один массив с данными и выход их функции будет осуществлен. Короче когда я писал - зложил тот смысл который был. Возможно он ошибочен)) Но на тот момент показался наиболее почему-то логичным.

11 Апр 2019

Вот в этой части кода:

if(get_user_flags(id) & ADMINCHECK)
{

if(CheckProtectADMIN[id] == 0){
client_print_color(id, print_team_default, "%L", id, "MJ_STOP_ADMIN")
set_task(30.0,"func_info")
return PLUGIN_HANDLED;
}

if(OLDzq != NEWzq){
client_print_color(id, print_team_default, "%L", id, "MJ_ALREADY_ACTIVE")
return PLUGIN_HANDLED;
}else if(OLDzq == NEWzq){
client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_1", zName)
client_print_color(id, print_team_default, "%L", id, "MJ_ACTIVE_2", CheckProtectADMIN[id])
activate[id] = 1
qq++;
NEWzq = qq
set_task(30.0,"func_info")
return PLUGIN_HANDLED;
}
return PLUGIN_HANDLED;
}

, после проверки на админа и выполнения, будет произведен выход (return PLUGIN_HANDLED;), по этому оно будет функционировать.

d3m37r4 · 12 Апр 2019

Флаги мб один раз лучше получить, а не для каждого условия заного?)

Limbooc · 12 Апр 2019

wellasgood, так ведь если игрок не админ и игрок не вип то код проверки выполняется в любом случая
другими словами простой игрок выполняет две лишних проверок:

Код:

if(AdminCheck = ложь)
иначе
if(VipCheck = ложь)
иначе
if(UserCheck = истина)
продолжаем

wellasgood · 12 Апр 2019

Limbooc, ну, с точки зрения оптимизации, Вы правы.

12 Апр 2019

d3m37r4, если честно, писал давненько сложные моменты эти, и залезать в свои старые дебри, какой я там смысл закладывал, сложно вспоминать. Тут неоднозначно.. Можно сказать что правильно, а что нет.

wellasgood · 24 Июн 2019

wellasgood обновил(а) ресурс Multi-Jump (mj) новой записью:

Обновление до версии (1.6)

Нажмите на эту ссылку, чтобы узнать подробности обновления

fl0wer · 25 Июн 2019

Что-то у вас версия 1.6, а код как был говном, так и остался.

wellasgood · 25 Июн 2019

fl0wer написал(а):
Что-то у вас версия 1.6, а код как был говном, так и остался.

ну эт, маленькое обновление

wellasgood · 31 Дек 2019

Ресурс Multi-Jump (mj) был обновлен:

Обновление до версии (1.6.1)

Нажмите на эту ссылку, чтобы узнать подробности обновления

fl0wer · 1 Янв 2020

Переход на 3-х значную версию

А что, в двухзначной цифр не хватило?)0))

wellasgood · 1 Янв 2020

fl0wer, просто так правильней из сообразительности.

Multi-Jump (mj) 1.6.1

SergeyShorokhov

WILL_BE

wellasgood

wellasgood

Tranquillity

wellasgood

wellasgood

Tranquillity

wellasgood

Limbooc

wellasgood

d3m37r4

111111

Limbooc

wellasgood

wellasgood

fl0wer

wellasgood

wellasgood

fl0wer

wellasgood

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