Качество кода

[Vaqtincha]

Log_entity

А че log_to_file уже не актуальна ?

pFile = fopen(szLogsDir, "at")

и зачем переоткрывать файл каждый раз когда вызывается pfn_keyvalue ?

 

[Subb98]

А че log_to_file уже не актуальна ?

Сделаешь через log_to_file запись в лог по дням? То есть, 1 сутки = 1 файл?

и зачем переоткрывать файл каждый раз когда вызывается pfn_keyvalue ?

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

7 Авг 2017

get_basedir(g_Dir, charsmax(g_Dir)); add(g_Dir, charsmax(g_Dir), "/logs/entities.log");

Лучше получить

get_localinfo("amxx_logs", g_Dir, charsmax(g_Dir))

т.к. в core.ini пути могут быть переопределены.

 

[fantom]

Сделаешь через log_to_file запись в лог по дням? То есть, 1 сутки = 1 файл?

Можна чекать раз в минуту или даже больше. Но лучше даже не чекать. Смениться карта - будет новый файл с новой датой

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

fflush(file)

 

[Subb98]

Можна чекать раз в минуту или даже больше. Но лучше даже не чекать. Смениться карта - будет новый файл с новой датой

Не нужно ничего чекать, это совершенно лишние действия. Новая карта - новый файл - это неверная логика для задачи 1 сутки - 1 файл. На сервере карта за сутки может смениться много раз и будет куча файлов.

7 Авг 2017

fflush(file)

О, новые функции.

 

[Vaqtincha]

Да вообще причем тут карта ? Смотрите описание натива

Спойлер

/* Sends log message to specified file. */
native log_to_file(const file[],const message[],any:...);

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

Кто сказал что надо держать открытым ? Открываем файл в precache закрываем файл в init. Изи

 

[Subb98]

Кто сказал что надо держать открытым ? Открываем файл в precache закрываем файл в init. Изи

Извиняюсь, меня занесло. В контексте конкретно этой задачи, действительно, можно обойтись стандартными функциями. Я начал вещать про log_to_file в целом. И вот именно в целом он не актуален, ИМХО.

 

[Vaqtincha]

Subb98, хотя мне кажется log_to_file медленее (он будет работат как и сток Log_entity)

 

[fantom]

Продолжение
http://dev-cs.ru/threads/222/#post-7480

8 Авг 2017

е нужно ничего чекать, это совершенно лишние действия. Новая карта - новый файл - это неверная логика для задачи 1 сутки - 1 файл. На сервере карта за сутки может смениться много раз и будет куча файлов.

Это был пример. Ниже я написал что при старте карты пробовать создать файл с датой. Если существует, то открыть для записи в конец

8 Авг 2017

О, новые функции

Старая уже. В 1.8.2 была

8 Авг 2017

native log_to_file(const file[],const message[],any:...);

Тут на каждую запись тдет открытие файлового дескриптора

8 Авг 2017

А так задача суто локальная не больше. Там дебаг. И пофигу на производительность. Хоть 1000 раз открыть файл для одной строкы