Multi-Jump (mj) 1.6.1

[wellasgood]

wellasgood добавил(а) новый ресурс:

Multi-Jump (mj) - Двойной прыжок на сервере

Нажмите на эту ссылку, чтобы узнать больше

 

[wellasgood]

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

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

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

 

[wellasgood]

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

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

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

 

[wellasgood]

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

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

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

 

[wellasgood]

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

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

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

 

[wellasgood]

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

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

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

 

[wellasgood]

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

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

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

 

[Tranquillity]

Перенесено в общий на суд скриптеров и публики

 

[Nordic Warrior]

new jumpnum[33] = 0

= 0 не нужно. переменные и массивы по умолчанию создаются с нулевым значением.
то же самое касается new bool:dojump[33] = false

Массивы, использующиеся для записи id игроков лучше записывать c использованием констант, для повышения читабельности кода:

-new activate[33]
+new activate[MAX_PLAYERS + 1]

Аналогично используем константы:

-new zName[33]
+new zName[MAX_NAME_LENGHT]

-client_print_color(id, 0, "^3[^4Сервер^3] ^1Ув. Админ, ^3Вы ^1исчерпали использование ^4multi-прыжков^1!")
+client_print_color(id, print_team_default, "^3[^4Сервер^3] ^1Ув. Админ, ^3Вы ^1исчерпали использование ^4multi-прыжков^1!")

Почему-то в get_mapname() использовал charsmax(), а в get_user_name() нет.

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

    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      // Установка кол-ва прыжков для Юзеров
    }
}

не дёргаем каждый раз get_user_flags(), оптимизируем:

public client_putinserver(id)
{
+    new flags = get_user_flags(id)
    jumpnum[id] = 0
    dojump[id] = false
    activate[id] = 0

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

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

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

 

[wellasgood]

Хмм, я конечно погляжу, возможно что-то войдет в следующую версию.. Но пока что.. Вот так, там плагин сам не мой. А другово Автора. Я лишь переделываю. И его данные я некторые не трогал.

5 Мар 2019

А также:

У всех свой стиль написания. Если я сделал как сделал, значит на тот момент мне показалось это логически правильным.

 

[Tranquillity]

У всех свой стиль написания. Если я сделал как сделал, значит на тот момент мне показалось это логически правильным.

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

Не бывает правильных стилей кода. Бывают утвержденные стили в команде или общепринятые стили.

 

[wellasgood]

Tranquillity, Вы правы.

 

[WILL_BE]

sbelov020, ну по поводу "оптимизации" new flags = get_user_flags(id), я бы так не говорил. Ибо в логичном решении, на мой взгляд, он и так - и так будет дергать get_user_flags(id)

 

[wellasgood]

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      // Установка кол-ва прыжков для Юзеров
    }

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

 

[Nordic Warrior]

WILL_BE, в каком плане? Получаем один раз флаги юзера в переменную, дальше просто 3 раза сравниваем их с нужным флагом. Или 3 раза дергать get_user_flags() ?

 

[WILL_BE]

sbelov020, это уже обсуждалось на старом-добром goldsrc, что никакой нужды в такой "оптимизации" нет

5 Мар 2019

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

 

[Nordic Warrior]

WILL_BE, https://wiki.alliedmods.net/Optimizing_Plugins_(AMX_Mod_X_Scripting)

 

[WILL_BE]

sbelov020, я же уже объяснил,

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

Ибо в те времена, самому была интересна такого рода оптимизация

 

[Nunf]

sbelov020, я же уже объяснил,

Ибо в те времена, самому была интересна такого рода оптимизация

Тоесть лучше 3 раза вызвать функцию? Наркомания