[VIP] Reload Progress Bar 1.0.0

HolyHender · 22 Сен 2020

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

[VIP] Reload Progress Bar - Индикатор выполнения перезарядки для вип-игроков.

При перезарядке в центре экрана у вип-игроков появляется прогресс-бар.

Оригинальный плагин
Но у @Grey83 есть ошибка в коде, в callback таймера индекс клиента передаёт. Поэтому за основу используется форк от @Wend4r

Видео демонстрация:

...

Узнать больше об этом ресурсе...

Grey83 · 22 Сен 2020

HolyHender написал(а):
Но у @Grey83 есть ошибка в коде, в callback таймера индекс клиента передаёт.

C-подобный:

if((iClient = GetClientOfUserId(iClient)) && IsClientInGame(iClient) /* When disconnecting it can have an index */)

Масло маслянное.
Если игрок с указанным UserId не на сервере, то GetClientOfUserId() вернёт 0, т.е. проверка IsClientInGame() не нужна.

А закомментированная строка с GetRandomInt(0, 15) нужна была только для тестов. =)
И ещё при математических операциях с int и float на выходе получается float, даже если int не преобразовать в float.
Так что float(iProgressTime) - это лишняя перестраховка.
Вот это:

C-подобный:

    if(g_hTimer[iClient] != INVALID_HANDLE)
    {
        KillTimer(g_hTimer[iClient]);
        g_hTimer[iClient] = INVALID_HANDLE;
    }

в новом синтаксисе выглядит вот так:

C-подобный:

    if(g_hTimer[iClient] != null)
    {
        KillTimer(g_hTimer[iClient]);
        g_hTimer[iClient] = null;
    }
 
или просто

    if(g_hTimer[iClient]) delete g_hTimer[iClient];

И у тебя при каждой перезагрузке создаётся хук SDKHook_WeaponSwitch, который при срабатывании удаляется. А смысл? Если игрок не переключится на другое оружие он так и будет работать.
Ты бы и при срабатывании таймера его отключал (у тебя будет туева хуча включенных хуков, если игрок несколько раз перезарядит оружие, но ни разу не сменит его при этом).

xstage · 22 Сен 2020

Grey83 написал(а):
C-подобный:

if((iClient = GetClientOfUserId(iClient)) && IsClientInGame(iClient) /* When disconnecting it can have an index */)

Масло маслянное.
Если игрок с указанным UserId не на сервере, то GetClientOfUserId() вернёт 0, т.е. проверка IsClientInGame() не нужна.

А закомментированная строка с GetRandomInt(0, 15) нужна была только для тестов. =)
И ещё при математических операциях с int и float на выходе получается float, даже если int не преобразовать в float.
Так что float(iProgressTime) - это лишняя перестраховка.
Вот это:

C-подобный:

if(g_hTimer[iClient] != INVALID_HANDLE) { KillTimer(g_hTimer[iClient]); g_hTimer[iClient] = INVALID_HANDLE; }

в новом синтаксисе выглядит вот так:

C-подобный:

if(g_hTimer[iClient] != null) { KillTimer(g_hTimer[iClient]); g_hTimer[iClient] = null; } или просто if(g_hTimer[iClient]) delete g_hTimer[iClient];

И у тебя при каждой перезагрузке создаётся хук SDKHook_WeaponSwitch, который при срабатывании удаляется. А смысл? Если игрок не переключится на другое оружие он так и будет работать.
Ты бы и при срабатывании таймера его отключал (у тебя будет туева хуча включенных хуков, если игрок несколько раз перезарядит оружие, но ни разу не сменит его при этом).

Какая разница INVALID_HANDLE или null, они обозначают одно и тоже.
И delete вроде не приравнивает к null

HolyHender · 22 Сен 2020

Grey83 написал(а):
C-подобный:

if((iClient = GetClientOfUserId(iClient)) && IsClientInGame(iClient) /* When disconnecting it can have an index */)

Масло маслянное.
Если игрок с указанным UserId не на сервере, то GetClientOfUserId() вернёт 0, т.е. проверка IsClientInGame() не нужна.

Нужна. Игрок в момент OnClientDisconnect не в игре, но имеет индекс.

Grey83 написал(а):
И у тебя при каждой перезагрузке создаётся хук SDKHook_WeaponSwitch, который при срабатывании удаляется. А смысл? Если игрок не переключится на другое оружие он так и будет работать.
Ты бы и при срабатывании таймера его отключал (у тебя будет туева хуча включенных хуков, если игрок несколько раз перезарядит оружие, но ни разу не сменит его при этом).

Чтобы функция часто не вызывалась, а только когда нужно.

Grey83 написал(а):
И ещё при математических операциях с int и float на выходе получается float, даже если int не преобразовать в float.
Так что float(iProgressTime) - это лишняя перестраховка.

sourcemod/plugins/include/float.inc at master · alliedmodders/sourcemod

SourceMod - Source Engine Scripting and Administration - alliedmodders/sourcemod

github.com

Grey83 · 22 Сен 2020

xstage написал(а):
Какая разница INVALID_HANDLEили null, они обозначают одно и тоже.

INVALID_HANDLE - это дефайн, который подставляет 0.

alliedmodders/sourcemod

SourceMod - Source Engine Scripting and Administration - alliedmodders/sourcemod

github.com

xstage написал(а):
И delete вроде не приравнивает к null

Этот оператор приравнивает. Я когда-то специально писал тестовый плагин, чтобы проверить это.

Сообщения автоматически склеены: 22 Сен 2020

HolyHender написал(а):
Чтобы функция часто не вызывалась, а только когда нужно.

Grey83 написал(а):
Ты бы и при срабатывании таймера его отключал (у тебя будет туева хуча включенных хуков, если игрок несколько раз перезарядит оружие, но ни разу не сменит его при этом).

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

HolyHender написал(а):
Нужна. Игрок в момент OnClientDisconnect не в игре, но имеет индекс.

Поэтому я в OnClientDisconnect() у себя в коде убиваю таймер.

Wend4r · 22 Сен 2020

Grey83 написал(а):
C-подобный:

if((iClient = GetClientOfUserId(iClient)) && IsClientInGame(iClient) /* When disconnecting it can have an index */)

Масло маслянное.
Если игрок с указанным UserId не на сервере, то GetClientOfUserId() вернёт 0, т.е. проверка IsClientInGame() не нужна.

Внимательнее пожалуйста: /* When disconnecting it can have an index */

Grey83 написал(а):
А закомментированная строка с GetRandomInt(0, 15) нужна была только для тестов. =)

Нужно было только для тех, кто захочет проверить другие типы.

Grey83 написал(а):
И ещё при математических операциях с int и float на выходе получается float, даже если int не преобразовать в float.

sourcemod/plugins/include/float.inc at master · alliedmodders/sourcemod

SourceMod - Source Engine Scripting and Administration - alliedmodders/sourcemod

github.com

Grey83 написал(а):
if(g_hTimer[iClient]) delete g_hTimer[iClient];

В этом случае, должна быть ошибка на подобии:

C-подобный:

L 09/20/2020 - 20:23:54: [SM] Plugin "vip/VIP_ThrowingKnives.smx" encountered error 23: Native detected error
L 09/20/2020 - 20:23:54: [SM] Invalid timer handle 23110323 (error 3) during timer end, displayed function is timer callback, not the stack trace
L 09/20/2020 - 20:23:54: [SM] Unable to call function ".8792.OnThrowingKnifeTimer" due to above error(s).

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

Это уже дело каждого, я этот код писал (форк с GItHub(а)), с упором на нормальное восприятие кода. По моему мнению, по этому сложно судить.

Grey83 · 22 Сен 2020

Wend4r написал(а):
Внимательнее пожалуйста: /* When disconnecting it can have an index */

Выше уже пояснял.

Wend4r написал(а):
В этом случае, должна быть ошибка на подобии:

Поэтому я у себя в таймере и прописал hTimer[client] = null;

xstage · 22 Сен 2020

Grey83 написал(а):
INVALID_HANDLE - это дефайн, который подставляет 0.

alliedmodders/sourcemod

SourceMod - Source Engine Scripting and Administration - alliedmodders/sourcemod

github.com

Этот оператор приравнивает. Я когда-то специально писал тестовый плагин, чтобы проверить это.

Сообщения автоматически склеены: 22 Сен 2020

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

Ну делают то они одно и тоже

null - A new, general keyword to replace INVALID_HANDLE.

SourcePawn Transitional Syntax - AlliedModders Wiki

wiki.alliedmods.net

Grey83 · 22 Сен 2020

Keyword и define это несколько разные вещи, даже если используются с одной и той же целью и одинаковым результатом.
public, static и continue - это тоже keywordы, например.
И вообще:

Instead of CloseHandle, you can now use the new delete keyword.

SourceMod 1.7.0 Release Notes - AlliedModders Wiki

wiki.alliedmods.net

alesh15 · 22 Сен 2020

Вообще хотел узнать можно ли сделать круг перезарядки поменьше?

Grey83 · 22 Сен 2020

alesh15, вот ответ: [CS:GO] Reload Progress Bar

G.R.I.L.F.I.K.S · 24 Сен 2020

Плагин хороший но не хватает:
1. Поддержки Модуль плагина - [VIP] Fast Reload
2. Когда сам делаешь перезарядку таймер показует когда патроны закончились и начинает автоматическую перезарядку то таймера нет.

babka68 · 29 Окт 2020

Всем привет,а такой вопрос можно ли сделать индикатор перезарядки для css,как показано на видео из cs go?

xstage · 29 Окт 2020

babka68 написал(а):
Всем привет,а такой вопрос можно ли сделать индикатор перезарядки для css,как показано на видео из cs go?

[CS:GO] Reload Progress Bar

xstage добавил(а) новый ресурс: [CS:GO] Reload Progress Bar - Добавляет индикатор выполнения перезарядки При перезарядке в центре экрана появляется прогресс-бар выполнения перезарядки. Код прогресс-бара взят с этого поста Узнать больше об этом ресурсе...

hlmod.ru

babka68 · 29 Окт 2020

xstage написал(а):
[CS:GO] Reload Progress Bar

xstage добавил(а) новый ресурс: [CS:GO] Reload Progress Bar - Добавляет индикатор выполнения перезарядки При перезарядке в центре экрана появляется прогресс-бар выполнения перезарядки. Код прогресс-бара взят с этого поста Узнать больше об этом ресурсе...

hlmod.ru

Обычная перезарядка,как при разминировании бомбы,а мне нужно как в cs go)

Grey83 · 29 Окт 2020

@babka68, максимум текстом секунды и десятые доли секунд
И шкалу псевдографикой

babka68 · 29 Окт 2020

Grey83 написал(а):
@babka68, максимум текстом секунды и десятые доли секунд
И шкалу псевдографикой

Жаль,спасибо за информацию=)

CoMaR · 6 Июн 2022

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

DOK-26-RUS · 31 Май 2023

И пофиксите пожалуйста надпись "обезвреживание бомбы" и "спасаете заложника". Эти надписи начинают появляться после первого применения к обезвреживанию бомбы или спас заложника.

d3v · 31 Май 2023

DOK-26-RUS написал(а):
И пофиксите пожалуйста надпись "обезвреживание бомбы" и "спасаете заложника". Эти надписи начинают появляться после первого применения к обезвреживанию бомбы или спас заложника.

Ресурс не поддерживается автором

[VIP] Reload Progress Bar 1.0.0

Прокуратура на связи

не пишу плагины с весны 2022

🏹

Прокуратура на связи

не пишу плагины с весны 2022

I'm Source Engine masochist

не пишу плагины с весны 2022

🏹

не пишу плагины с весны 2022

Участник

не пишу плагины с весны 2022

Участник

Участник

🏹

Участник

не пишу плагины с весны 2022

Участник

Участник

Участник

Участник

Похожие темы