Система вип объявлений

[3]

Всем привет, сегодня я хочу вам представить систему вип объявлений

Для начала, заходим в наш мод, жмем CTRL+F и ищем new. Туда мы вписываем следующее:

new vadtime[MAX_PLAYERS];

Это будет лимит отправки сообщений по времени.

 

Далее переходим к добавлению команды. Ищем строку cmd:.

После команды, которая там стоит добавляем команду подачи объявления:

CMD:vad(playerid, params[])
{
    if(PI[playerid][pVIP] < 7)  //pVIP - ваша переменная на випку, 7 - уровень випки}
        return SCM(playerid, COLOR_RED, NO_DOSTUP_TEXT); //Ваша проверка}

    new text[128];
    if(vadtime[playerid] > gettime())
        return SCM(playerid, COLOR_RED, !"Объявление можно подавать один раз в 1 минуту."); //SCM - SendClientMessage
    if(sscanf(params, "s[128]", text))
        return SCM(playerid, COLOR_RED, !"Используй: /vad [объявление]"); //SCM - SendClientMessage
    if(strlen(text) > 128 || strlen(text) < 10) //Лимит символов
        return SCM(playerid, COLOR_RED, !"Объявление должно содержать от 10 до 128 символов!"); // SCM - SendClientMessage
    if(CheckIsADomen(playerid, text)) return 1;
    {
        SCMAllf(COLOR_BLUE, "[VIP-AD] {FF0000}%s. {FFFFFF}Автор: %s", text, PN(playerid)); //SCMAllf - SendClientMessageToAllf
    }

    vadtime[playerid] = gettime()+60; //Лимит по времени

    return 1;
}

Ну вот и все, если будет много актива то выложу другие системы

[169]

  В 10/28/2021 в 09:08, Noleran сказал:

if(CheckIsADomen(playerid, text))

Читать далее...  

Хотя бы качественнее вырезайте) Где checkisadomen?

 

[3]

Когда вырезал уже так было, но все работает отлично

 

[0]

+

[0]

У меня один вопрос, для чего выделять столько памяти под обычный лимит сообщений?
 

new vadtime[MAX_PLAYERS];

И никого это не смущает?

[560]

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

[1 034]

  В 10/31/2021 в 14:32, babyproger сказал:

столько памяти

Читать далее...  

4 b * 1000 = 4 000 b ~ 4 kB, разве это очень много? 

Как мне кажется, при разработке скриптов на PAWN, лучше обращать внимание на читаемость, понятность, поддерживаемость системы, нежели на чрезмерную оптимизацию, которая в принципе ничего не дает. 

  В 10/28/2021 в 09:08, Noleran сказал:

SCMAllf

Читать далее...  

Как по мне, лучше не использовать таких сокращений, как будто лимит на буквы ввели. Полное название более читаемое

SendClientMessageToAllFormat(...)

  В 10/28/2021 в 09:08, Noleran сказал:

PN

Читать далее...  

Здесь аналогично. 

 

Я бы предложил такой вариант.

  Команда (открыть спойлер)

new VipAdPlayerCoolDown[MAX_PLAYERS];

CMD:vad(playerid, params[])
{
  if(GetPlayerVipLevel(playerid) < 7)  //pVIP - ваша переменная на випку, 7 - уровень випки
  {
    return SendClientMessage(playerid, COLOR_RED, !"У вас нет VIP"); //Ваша проверка}
  }

  if(HasPlayerVipAdCoolDown(playerid))
  {
    return SendClientMessage(playerid, COLOR_RED, !"Объявление можно подавать один раз в 1 минуту.");
  }

  new text[128];

  if(sscanf(params, "s[128]", text))
  {
    return SendClientMessage(playerid, COLOR_RED, !"Используй: /vad [объявление]");
  }

  if(strlen(text) > 128 || strlen(text) < 10) 
  {
    return SendClientMessage(playerid, COLOR_RED, !"Объявление должно содержать от 10 до 128 символов!"); 
  }

  new formattedText[200], playerName[MAX_PLAYER_NAME];
  GetPlayerName(playerid, playerName, sizeof playerName);
  format(formattedText, sizeof formattedText, "[VIP-AD] {FF0000}%s. {FFFFFF}Автор: %s", text, playerName);
  SendClientMessageToAll(COLOR_BLUE, formattedText); 

  SetPlayerVipAdCoolDown(playerid, 60); // кулдаун 60 сек.

  return 1;
}

GetPlayerVipLevel(playerid)
{
  return PI[playerid][pVip]; // глобальная перменная, в которой лежит VIP уровень для игрока playerid.
}

HasPlayerVipAdCoolDown(playerid)
{
  return VipAdPlayerCoolDown[playerid] > gettime();
}

SetPlayerVipAdCoolDown(playerid, seconds)
{
  VipAdPlayerCoolDown[playerid] = gettime() + seconds;
}

 

Тут используется доступ к глобальным переменным через функции, мне кажется, такой подход лучше чем менять глобальное состояние во всем моде. Так все изменения "стекаются" в одно место, и их можно хоть как-то контролировать.