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

Кидала, мошенник, вымогатель

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