Система технических работ

[Узбек Хороший 1]

Всем добрый вечер ребят! 

Я начинающий, сегодня я вам сливаю команду 2(шт) на технические работы

Ловите 

Спойлер

||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
cmd:tex(playerid, params[])
{
    if(PI[playerid][pAdmin] < ADM_GL) return 1; //Проверка на администратора, для использавания команды
 //===============================================================================================
 for(new i = 0; i <= MAX_PLAYERS; i++)
    {
     if(IsPlayerConnected(i))
     {
      if(PI[pAdmin] > SUPPORT) //Если игрок администратор, то высвечиваеися табличка с паролем и т.д
      {
       ShowPlayerDialog(i,0,DIALOG_STYLE_MSGBOX,"Внимание!!!","{CB1A1A}Администраторов не отключается с сервера!\n{CB1A1A} Активированы тех.работы для сервера!\n{CB1A1A}Пароль к серверу: ВАШ ПАРОЛЬ\n{CB1A1A}Причина будет в нашей офф. группе VK: ВАША ГРУППА","Понял","");
      }
        if(PI[pAdmin] == ADM_NONE) //Если игрок не администратор, то высвечевается табличка с информацией, и его отключает с сервера
        {
          ShowPlayerDialog(i,0,DIALOG_STYLE_MSGBOX,"Внимание!!!","{CB1A1A} Внимание массовое отключение с сервера!\n{CB1A1A}Причина будет в нашей офф. группе VK: ВАША ГРУППА","Понял","");
          Kick(i);
     }
     }
  }

 //===============================================================================================
 SendRconCommand("password Ваш пароль");
 SendRconCommand("hostname TimePlay RolePlay || У нас Тех.Работы, не грустите!");
 //===============================================================================================
 SendClientMessageToAll(0xF04245FF, "Руководитель сервера активиривал Тех.Работы!");
 return 1;
}

cmd:texoff(playerid, params[])
{
    if(PI[playerid][pAdmin] < ADM_GL) return 1;
 //===============================================================================================
 SendRconCommand("password 0"); //Снос пароля
 SendRconCommand("hostname TimePlay RolePlay || Заходи к нам, мы вас ждем!");
 //===============================================================================================
    callcmd::gmx(i, ""); //Вызов команды для рестарта сервера
 //===============================================================================================
 SendClientMessageToAll(0xF04245FF, "Руководитель сервера деактивиривал Тех.Работы! Спасибо всем кто помагал!");
 return 1;
}

 

 

Отредактировано 17 апреля пользователем Cawfee

[user408337 Хороший 13]

Отсутствие табуляции, синтаксические и орфографические ошибки, излишние зависимости.

И это 30 строк кода

enum //admin levels struct
{
    ADM_NONE = 0,
    SUPPORT,  //1
    ADM_GL    //2
}

cmd:tex(playerid, params[])
{
    if(PI[playerid][pAdmin] < ADM_GL) return 1; //Проверка на администратора, для использавания команды

    SendRconCommand("password Ваш пароль");
    SendRconCommand("hostname TimePlay RolePlay || У нас Тех.Работы, не грустите!");
    SendClientMessageToAll(0xF04245FF, !"Руководитель сервера активиривал Тех.Работы!");

    for(new i = 0, size = GetPlayerPoolSize(); i <= size; ++i)
    {
        if(!IsPlayerConnected(i)) continue;
        if(PI[i][pAdmin] == ADM_NONE) //Если игрок не администратор, то высвечевается табличка с информацией, и его отключает с сервера
        {
            ShowPlayerDialog(i, 32767, DIALOG_STYLE_MSGBOX, "Внимание!!!",
                "\
                    {CB1A1A} Внимание массовое отключение с сервера!\n\
                    {CB1A1A}Причина будет в нашей офф. группе VK: ВАША ГРУППА\
                ", "Понял", "");
            Kick(i);
        }
        else
        {
            ShowPlayerDialog(i, 32767, DIALOG_STYLE_MSGBOX, "Внимание!!!",
                "\
                    {CB1A1A}Администраторов не отключается с сервера!\n\
                    {CB1A1A}Активированы тех.работы для сервера!\n\
                    {CB1A1A}Пароль к серверу: ВАШ ПАРОЛЬ\n\
                    {CB1A1A}Причина будет в нашей офф. группе VK: ВАША ГРУППА\
                ", "Понял", "");
        }
    }
    return 1;
}

cmd:texoff(playerid, params[])
{
    if(PI[playerid][pAdmin] < ADM_GL) return 1;

    SendRconCommand("password 0"); //Снос пароля
    SendRconCommand("hostname TimePlay RolePlay || Заходи к нам, мы вас ждем!");

    SendClientMessageToAll(0xF04245FF, !"Руководитель сервера деактивиривал Тех.Работы! Спасибо всем кто помагал!");
    return 1;
}

 

[Узбек Хороший 1]

@user408337 тоесть это уже правельныйвариант?