Создание простейшей системы чата команды

Вид для печати

10.08.2014, 17:27
Salvacore

Создание простейшей системы чата команды

Команда

PHP код:

CMD:f(playerid, params[])//DC_CMD { if(!strlen(params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]"); new string[144], name[MAX_PLAYER_NAME] ; GetPlayerName(playerid,name,sizeof(name)); format(string, sizeof(string),"[Чат фракции]%s: %s", name, params); TeamSendMsg(playerid, -1, string); SetPlayerChatBubble(playerid,params,-1,20.0,3500); return 1; }

Функция отправки сообщения команде

PHP код:

TeamSendMsg(playerid,color, string[]) { foreach(Player, i)//for(new i=0;i<MAX_PLAYERS;i++) { /*if(!IsPlayerConnected(playerid)) return 1;*/ if(gTeam[i] == gTeam[playerid]) //Измените на своё { SendClientMessage(i, color, string); } } return 1; }

Автор: Salvador

Специально для: Pro-Pawn.Ru

Копирование данной статьи без разрешения автора запрещено!

Библиотека

SendClientMessage
SetPlayerChatBubble
Foreach
GetPlayerName
DC_CMD
10.08.2014, 18:37
DeimoS

Цитата:

Сообщение от Salvador

Команда

PHP код:

static const fmt_str[] = "[Чат фракции]%s: %s"; new string[sizeof(fmt_str)-2+MAX_PLAYER_NAME-2+128], name[MAX_PLAYER_NAME] ;

Вот к чему это извращение тут? Хотя да, проще написать лишнюю строку, нежели сосчитать 15 символов...
Или я что-то не понимаю в этом мире, или подобный способ оправдывает себя только когда у нас есть огромное сообщение, раскиданное на несколько строг (дабы компилятор его прожевал) и которое может редактироваться в дальнейшем (правила сервера или список команд, например). А если под каждое короткое сообщение подобное делать, кол-во ненужных строк возрастёт раза в 2, что явно не отразится на читаемости... В общем, как-то вы уж перебарщиваете с этим извращением.

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

Цитата:

Сообщение от Salvador

Команда

PHP код:

CMD:f(playerid, params[])//DC_CMD { if(sscanf(params, "s[128]", params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]"); static const fmt_str[] = "[Чат фракции]%s: %s"; new string[sizeof(fmt_str)-2+MAX_PLAYER_NAME-2+128] ; GetPlayerName(playerid,string,MAX_PLAYER_NAME); format(string, sizeof(string),fmt_str, string, params); TeamSendMsg(playerid, -1, string); SetPlayerChatBubble(playerid,params,-1,20.0,3500); return 1; }

P.S. Ещё один камень в огород твоего способа подсчёта символов - это ограничения SA-MP. SendClientMessage не прожуёт сообщение больше 144 символов, а у тебя массив явно за 144 ячейки переваливает :)
10.08.2014, 19:38
Seregamil

Цитата:

static const fmt_str[] = "[Чат фракции]%s: %s";
new
string[sizeof(fmt_str)-2+MAX_PLAYER_NAME-2+128],
name[MAX_PLAYER_NAME]
;

Чего вы так за ячейками гоняетесь? И без того известно, что в чат больше 128 символов не уйдет.
11.08.2014, 13:09
Salvacore

Цитата:

Сообщение от Seregamil

Чего вы так за ячейками гоняетесь? И без того известно, что в чат больше 128 символов не уйдет.

Я делаю так, чтобы не нагружать мод, вдруг у них RLS.
А я тебя не оскорблял.
11.08.2014, 17:57
Nazarik

Цитата:

Сообщение от Salvador

PHP код:

if(sscanf(params, "s[128]", params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]");

Бред...

Лучше уж:

PHP код:

if(isnull(params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]"); // ну или if(strlen(params) < 1)

макрос:

PHP код:

#define isnull(%1) \ (((%1[0]) == 0) || ((%1[0]) == '\1') && ((%1[1]) == 0))
11.08.2014, 19:02
DeimoS

Цитата:

Сообщение от Salvador

Я делаю так, чтобы не нагружать мод, вдруг у них RLS.
А я тебя не оскорблял.

Ты напечатал более 50 символов ради того, чтоб не считать 15... И после этого ты что-то на людей с P-I говоришь... :facepalm:
Нравится насиловать себя и свой код - пожалуйста. Только не продвигайте это в массы :negative:
11.08.2014, 19:36
Seregamil

Цитата:

Сообщение от Nazarik

Бред...

Лучше уж:

PHP код:

if(isnull(params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]"); // ну или if(strlen(params) < 1)

макрос:

PHP код:

#define isnull(%1) \ (((%1[0]) == 0) || ((%1[0]) == '\1') && ((%1[1]) == 0))

PHP код:

if( !strlen( params ) ) return false;

Не?
11.08.2014, 20:28
Nazarik

Цитата:

Сообщение от Seregamil

PHP код:

if( !strlen( params ) ) return false;

Не?

Можна и так, но вроде в этом случае выведет сообщения "SERVER: Unknown Command".
12.08.2014, 06:24
Seregamil

Цитата:

Сообщение от Nazarik

Можна и так, но вроде в этом случае выведет сообщения "SERVER: Unknown Command".

Я как пример привёл же...
10.09.2014, 21:22
Salvacore

Update.
19.03.2016, 14:38
Иван Бубнов

PHP код:

new string[128 - (15 + (-2 + MAX_PLAYER_NAME)) + 1];

Получается так?
19.03.2016, 14:42
$continue$

Вы про что?

Цитата:

Сообщение от Иван Бубнов

PHP код:

new string[128 - (15 + (-2 + MAX_PLAYER_NAME)) + 1];

Получается так?
19.03.2016, 15:28
iWors

Я к примеру бы иным способом сделал :)

PHP код:

CMD:f(playerid, params[]) { if(!strlen(params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]"); new name[MAX_PLAYER_NAME] ; GetPlayerName(playerid, name, sizeof(name)); static const fmt_str[] = "[Чат фракции] %s: %s"; new string[sizeof(fmt_str) - 2 + MAX_PLAYER_NAME]; format(string, sizeof(string), fmt_str, name, params); TeamSendMsg(playerid, -1, string); SetPlayerChatBubble(playerid, params, -1, 20.0, 3500); return 1; }
19.03.2016, 18:22
DeimoS

Цитата:

Сообщение от iWors

Я к примеру бы иным способом сделал :)

PHP код:

CMD:f(playerid, params[]) { if(!strlen(params)) return SendClientMessage(playerid, -1, "[ВВЕДИ]: (/f) [Текст]"); new name[MAX_PLAYER_NAME] ; GetPlayerName(playerid, name, sizeof(name)); static const fmt_str[] = "[Чат фракции] %s: %s"; new string[sizeof(fmt_str) - 2 + MAX_PLAYER_NAME]; format(string, sizeof(string), fmt_str, name, params); TeamSendMsg(playerid, -1, string); SetPlayerChatBubble(playerid, params, -1, 20.0, 3500); return 1; }

А в чём, собственно, разница, кроме перестановки кода с одного места в другое?
08.09.2016, 15:03
TheMallard

а ты уверен что выводится сообщение игрока?
08.09.2016, 21:30
vovandolg

Цитата:

Сообщение от TheMallard

а ты уверен что выводится сообщение игрока?

Выводится там всё) размеры может быть слегка не правильно посчитаны
13.09.2016, 20:36
MrJu[N]ior

PHP код:

for(new i=0;i<MAX_PLAYERS;i++) { if(!IsPlayerConnected(playerid)) return 1;

Это один из допустимых тобой вариантов, вот в проверке на коннект почему-то вместо i стоит playerid. Норм))
13.09.2016, 23:56
m1n1vv

А если?

PHP код:

new fmt_str[37] = !"[Чат фракции] "; strcat(fmt_str, name); strcat(fmt_str, !": "); strcat(fmt_str, params);

MrJu[N]ior, проблема не только в неверной переменной, там и функция "останавливается", если ID не найден.

PHP код:

if (!IsPlayerConnected(i)/* || i == INVALID_PLAYER_ID*/) continue;

И если делать через format, то можно же так:

PHP код:

format(params, 144, fmt_str, name, params); SendClientMessage(playerid, -1, params);

Только сначала надо будет выводить SetPlayerChatBubble, а потом format с TeamSendMsg.
И чисто для красоты кода, я назвал бы функцию SendClientMessageToTeam.