TEST_helper [Удалено]

Статус
В этой теме нельзя размещать новые ответы.

kirich

KirichHub.ru / myDiscord: kirich_sexy
Сообщения
143
Реакции
351
kirich добавил(а) новый ресурс:

TEST_helper - Уведомления для сбора тестировщиков
 
Последнее редактирование:

Nekro

Терра инкогнита
Сообщения
4,169
Реакции
2,500
kirich добавил(а) новый ресурс:

TEST_helper - Уведомления для сбора тестировщиков
1. Зачем там это?
#include <sourcemod>
#include <sdktools>

2. В OnClientPostAdminCheck не логично ли было изначально проверять работает ли плагин, а уже потом валидность игрока, а не наоброт?
3. В таймере Timer_ShowMenu аналогично второго пункта
4. Убийство таймера обычно не
C-подобный:
KillTimer(g_hShowTimer[client]);
g_hShowTimer[client] = null;

а
C-подобный:
delete g_hShowTimer[client];
5. Меню каждую рекламу? Дык плохим словом всю родню будут вспоминать же..
В калбеке меню вместо StrEqual(info, "info") лучше смотреть индекс пункта.
 

Grey83

не пишу плагины с весны 2022
Сообщения
8,805
Реакции
5,254
Убийство таймера обычно не
Вообще это равнозначно, а в старых билдах SM1.8+ было даже предпочтительнее, т.к. delete работало не всегда правильно, емнип.

И самому код глянуть, что ли?
Сообщения автоматически склеены:

menu.AddItem("back", "Назад");OMG!

Чувак, ты почитай вики SM про создание меню, что ли.
Ты делаешь меню, но код написан как будто ты пытаешься создать панель, а не меню.

И если создаёшь статичное меню, которое будет часто использоваться, то лучше его один раз создать его при запуске плагина, а потом его вызывать для игрока, а не создавать каждый раз при вызове и потом удалять.
Сообщения автоматически склеены:

И ещё: menu.SetTitle() поддерживает форматирование.

Вообще я бы там многое переписал.
 
Последнее редактирование:

AzureLane

Участник
Сообщения
51
Реакции
32
Все написано отлично, исходя из того, что если я поменяю на то как ты просишь, ничего не изменится и останется прежним.

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

Grey83

не пишу плагины с весны 2022
Сообщения
8,805
Реакции
5,254
Вот как-то так:
SM1.13:
//// sm_helper_menu.sp
//
// Code size:         7108 bytes
// Data size:         4020 bytes
// Stack/heap size:      16536 bytes
// Total requirements:   27664 bytes
//
// Compilation Time: 0,09 sec
// ----------------------------------------

//// TEST_helper.sp
//
// Code size:         8812 bytes
// Data size:         4396 bytes
// Stack/heap size:      16744 bytes
// Total requirements:   29952 bytes
//
// Compilation Time: 0,12 sec
// ----------------------------------------
 

Вложения

  • sm_helper_menu.sp
    4.4 КБ · Просмотры: 21
Статус
В этой теме нельзя размещать новые ответы.
Сверху Снизу