GameCMS Time Awards

Krenko · 19 Авг 2018

Спасибо заранее.

19 Авг 2018

Уменьшил

Код:

new timeAward[] = {0, 30, 60, 90, 120, 180, 240, 300, 360};

на

Код:

new timeAward[] = {0, 30, 60, 90, 120};

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

19 Авг 2018

fantom · 19 Авг 2018

moneyAward должен иметь такое же количество элементов как и timeAward[ судя по коду. Но если честно то не знаю чем думал zhorzh78 когда писал плагин. Чего стоит такая строчка кода

Код:

for(new i = charsmax(timeAward); i > 0; i--)

Дальше этой смотреть аж страшно

zhorzh78 · 19 Авг 2018

fantom написал(а):
Но если честно то не знаю чем думал

Чтобы узнать, нужно понять задачу алгоритма.

fantom написал(а):
как и timeAward[ судя по коду.

// Все массивы должны иметь равное количество значений. Призы выдаются только 1 раз за каждый промежуток времени!

Судя по тому, что читается все через строчку, узнать о чем думал автор не представляется реальным.

Krenko · 19 Авг 2018

Уменьшил кол-во призов.

zhorzh78 · 19 Авг 2018

Krenko,

C++:

new timeAward[] = {0, 30, 60, 90, 120, 180, 240, 300, 360};
new moneyAward[] = {0, 0, 0, 0, 0, 0, 0, 0, 0}; //нужно было добавить значения
new Float:shillAward[] = {0.0, 0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8};
//все 3 массива должны иметь одинаковое количество

fl0wer · 19 Авг 2018

fantom написал(а):
Чего стоит такая строчка кода

fantom написал(а):
Код:

for(new i = charsmax(timeAward); i > 0; i--)

Что с ней не так?

Sonyx · 19 Авг 2018

fl0wer написал(а):
Что с ней не так?

https://forums.alliedmods.net/showthread.php?t=135799

fantom · 19 Авг 2018

fl0wer написал(а):
Что с ней не так?

То, что charsmax используется для строк.. И не логично использовать его для массивов, хоть и оно работает. Тогда давайте возвращать FMRES_IGNORED в Ham хуках. Оно так же будет работать.

zhorzh78 написал(а):
Судя по тому, что читается все через строчку, узнать о чем думал автор не представляется реальным.

Судя по тому что логика в плагине вывернута через изнанку: читать код с первого взгляда практически нереально.
Тот же кусок кода можна было сделать намного более читаемым и логичным. Например

Код:

public CheckAwards(id, itime) {
    for (new i = sizeof timeAward, t = itime/60; i > PlTimer[id][timePrize]; i--) {
        if (timeAward[i] > t) {
            continue;
        }

        new bool:isGiven = false;
        new szAwtext[160];
        new len = formatex(szAwtext, charsmax(szAwtext), "^1Ты играешь ^4%d минут^1, получаешь призы:", itime/60);
        if(moneyAward[i]) {
            len += formatex(szAwtext[len], charsmax(szAwtext) - len, " [^4%d $^1]", moneyAward[i]);
            isGiven = true;
        }

#if defined REALMONEY
        if(g_API && shillAward[i] > 0.0 && cmsapi_is_user_member(id) && cmsapi_get_user_money(id) < maxRealMoney) {
            cmsapi_add_user_money(id, shillAward[i]);
            len += formatex(szAwtext[len], charsmax(szAwtext) - len, " [^4%.2f рублей^1]", moneyAward[i]);
            isGiven = true;
        }
#endif

#if defined SET_FLAGS
        if (flagsAward[i][0]) {
            new flags = read_flags(flagsAward[i]);
            if (~get_user_flags(id) & flags) {
                len += formatex(szAwtext[len], charsmax(szAwtext) - len, " ^4флаг доступа ^1^"%s^"", moneyAward[i]);
                if(g_API)
                    cmsapi_set_user_flags(id, flagsAward[i], -1, "ТОП по времени");
                else
                    set_user_flags(id, flags);

                isGiven = true;
            }
        }
#endif

        if(isGiven) {
            client_print_color(id, 0, szAwtext);
            PlTimer[id][timePrize] = i;
#if defined DEBUG
            log_to_file(logFile, "CheckAwards.. Client- %s | %s", AuthIDs[id], szAwtext);
#endif
            break;
        }
    }
}

Код куда более логичен и оптимизирован. PlTimer[id][timePrize] правда нужно чтобы всегда был >= 0. Но это легко гарантировать если построить логику.

Дополнительно флагы желательно закэшировать вместо тупого условия

Код:

if(flagsAward[i][0] && (~(get_user_flags(id)) & (flags = read_flags(flagsAward[i]))))

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

А для любителей мегаоптимизация в ущерб читаемости кода и наоборот стоит начать хотя бы с этой статьи.

Часто этим грешит такой тип программистов как “Олимпиадник”. Знание эффективных алгоритмов или внутреннего устройства процессора заставляет их думать, что код должен быть максимально эффективен.

На практике на оптимизацию тратится слишком много времени, код становится совершенно неподдерживаемым, нерасширяемым и неустойчивым к ошибкам.

zhorzh78 · 19 Авг 2018

Так много написано в мой адрес..

C++:

for (new i = sizeof timeAward, t = itime/60; i > PlTimer[id][timePrize]; i--) {

Наверное, есть какая-то острая необходимость выходить за пределы массива, но я не стал этого делать.
Да, каюсь, charsmax использовать не логично. Логично - sizeof(timeAward) -1. Но, мне лень, поэтому я использую "хитрость"

C++:

#define charsmax(%1) (sizeof(%1)-1)

C++:

if (timeAward[i] > t){
           continue;
}

В этом тоже скрытый смысл, наверное. Вероятнее, знак > выглядит красивее, чем <= и меньше пробелов табуляции последующего блока, удовлетворяющего условию.
И переменную t тоже нужно создавать. Негоже проводить деление двух чисел itime/60 аж 2 раза.

C++:

isGiven = true;

Это да.. Логика создания отдельной переменной и ее выставления в true в каждом блоке - не вызывает никаких сомнений.

C++:

if(flagsAward[i][0] && (~(get_user_flags(id)) & (flags = read_flags(flagsAward[i]))))

Действительно, тупое условие. Оно ведь проверяет на отсутствие флага с одновременным кешированием, а не кешируется с последующей проверкой.

Спасибо за бесполезно потраченное на меня время.
За сим, откланяюсь.

19 Авг 2018

zhorzh78 написал(а):
И переменную t тоже нужно создавать. Негоже проводить деление двух чисел itime/60 аж 2 раза.

Пожалуй, соглашусь. Поздней ночью не всегда поймешь, что она в цикле.

19 Авг 2018

fantom написал(а):
PlTimer[id][timePrize] правда нужно чтобы всегда был >= 0

Правда.. ну так а почему не проверяете?

Krenko · 19 Авг 2018

zhorzh78, спасибо!

Minni · 19 Авг 2018

zhorzh78, дальнейшее обсуждение проводите в другой теме. Эта закрыта по причине того, что изначальная проблема решена.