Round && CMD Time 1.0

[KiKiEEKi]

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

Round && CMD Time - Показывает время

Показывает время в начале раунда или при написание команды в чате.
Показывает в HUD, чат или все сразу. Можно настраивать что будет показывать.

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

 

[MagoG]

Может я плохо, что либо заметил, но где RU format?

Если правильно понимаю,

strftime

, даёт по стандарту английский формат времени (Mon, Saturday) и так далее...Могу ошибаться?

 

[RusJJ]

@magog, так в скринах оно и видно)

 

[rejchev]

@KiKiEEKi, у вас откровенный говнокот.
В него входит:

• дублирования блоков с кодом
• утечка памяти
• смешанный синтаксис
• неоправданная перегрузка(понятие, в полной его мере, не уместно в контексте SP, т.к. полиморфизмом тут и не пахло) и тип возвращаемых значений некоторых калбек - функций

 

[RusJJ]

дублирования блоков с кодом

Всего-то два раза: команда и таймер, ничего критичного нет (да и в целом, разницы действительно нет, обычное удобство чтения кода).

утечка памяти

KV просто не закрылся. Память будет кушать в пределах погрешности относительно самого сервера. Хотя по хорошему после прочтения закрыть бы его, прямо в OnPluginStart

смешанный синтаксис

Если уж делать по новому, то действительно надо было доделывать. Нет возвращаемых значений, в аргументов нет типов, читать страшно.

 

[Grey83]

Офигенная идея запускать таймер в начале КАЖДОГО раунда, а убивать только в конце карты

В hKeyValues.GetString() указываешь в параметрах строки g_cText и g_cFormatTime (имеющие размер в 64 символа), а в качестве размера указываешь размер строки szPath (256 символов). Умно

Непонятно для чего прописывать настройки в файл конфига, когда удобнее юзать квары.
Ну да ладно, но для каждого канала цвета делать свой ключ - это вне моего разумения. Есть же функция KvGetColor(Handle kv, const char[] key, int& r, int& g, int& b, int& a)
Положение и время тоже можно хранить вместе и получать с помощью KvGetVector(Handle kv, const char[] key, float vec[3], const float defvalue[3]) (два значения для положения текста, а третье - для времени).
Ну или время отдельно.

Конфиг станет выглядеть примерно вот так:

"RoundCMDTime"
{
    "x_y_time"    "0.85 0.05 3.0"        //Позиция по X, Y и время отображения
    "color"        "255 255 255 255"    //Цвет HUD'a: красный, зеленый, синий, прозрачность
    "hudchat"    "0"            //0-отображать там и там \ 1-отображать в HUD \ 2-отображать в чат
    "text"        "Сегодня"    //Ваш текст
    "formating"    "%c"        //Формат отображения время, даты и т.п
                            //Смотреть все форматы тут >> http://cplusplus.com/reference/ctime/strftime/
}

Сообщения автоматически склеены: 12 Июн 2021

В общем как-то так:

 

[rejchev]

Всего-то два раза: команда и таймер, ничего критичного нет (да и в целом, разницы действительно нет, обычное удобство чтения кода).

Понятие "Удобство", в последнее время, используется, как оправдание всему, что по факту является неуместным.
Неуместным и неоправданным, в данной ситуации, является дублирование кода, которое противоречит этому понятию.

Оправданием подобной ошибки может послужить малый объем кода, который не вызывает основных проблем, при работе с ним.
Яркий пример того, что бывает, когда этим пользуются под эгидой "Удобство": AutoBalance Universal Edition

 

[KiKiEEKi]

Спасибо за отзывы, буду исправлять