Прыжки с парашюта

[ProG]

Сегодня сделал такую системку: Прыжки с парашюта!
Начнём:
Ко всем new:

new Text3D:parawut;
new samolet[2];
new parajut;
new Sozdan[MAX_PLAYERS]
new Zapusk[MAX_PLAYERS];
new Delete[MAX_PLAYERS]; 

Создаём команду в public OnPlayerCommandText(playerid, cmdtext[]):

    if(strcmp(cmdtext, "/samolet", true) == 0)
    {
        new string[256], s[32], z[32], d[32];
          if(Sozdan[playerid] == 0) { s = "{FF0000}Не выполнено{ffffff}"; } else { s = "{00CD00}Выполнено{ffffff}"; }
          if(Zapusk[playerid] == 0) { z = "{FF0000}Не выполнено{ffffff}"; } else { z = "{00CD00}Выполнено{ffffff}"; }
          if(Delete[playerid] == 0) { d = "{FF0000}Не выполнено{ffffff}"; } else { d = "{00CD00}Выполнено{ffffff}"; }
        format(string, sizeof(string), "Создать самолёт [%s]\nЗапустить самолёт [%s]\nУдалить самолёт [%s]",s,z,d);
        ShowPlayerDialog(playerid,1,DIALOG_STYLE_LIST,"Меню",string,"Выбрать","Отмена");
        return 1;
    } 

Далее создаём функции для диалога в public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[]):

    if(dialogid == 1)
     {
         if(response)
        {
             if(listitem == 0)
            {
                 if(Sozdan[playerid] == 0)
                 {
                      parawut = Create3DTextLabel( "Парашют! :)", 0x7FFF00FF, 2008.8929,-2532.5046,14.0, 40.0, 0 );
                     samolet[0] = CreateObject(14553, 1980.20, -2506.91, 25.82,   0.00, 0.00, 220.00);
                    samolet[1] = CreateObject(14548, 1983.20, -2510.50, 25.50,   0.00, 0.00, 220.00);
                    for(new i=0;i<MAX_PLAYERS;i++)
                    {
                        parajut = 1;
                         SetPlayerCheckpoint(i, 2008.8929,-2532.5046,13.5469, 1.0);
                        SendClientMessage(i, 0xff0000ff,"{ff0000}Внимание: {ffffff}Всех желающих попрыгать с парашюта ждём в {ADD8E6}Аэропорту Los Santos");
                        Sozdan[playerid] = 1;
                        Delete[playerid] = 0;
                    }
                }
                    else
                {
                    SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт уже создан!");
                }
            }
             if(listitem == 1)
             {
                 if(Sozdan[playerid] == 0) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт не создан!");
                if(Zapusk[playerid] == 0)
                {
                     MoveObject(samolet[0], -10.251000404358, -264.27801147461, 2000.32, 40);
                    MoveObject(samolet[1], -7.251000404358, -267.86801147461, 2000.0, 40);
                    Zapusk[playerid] = 1;
                    Delete3DTextLabel(Text3D:parawut);
                    for(new i=0;i<MAX_PLAYERS;i++)
                    {
                        DisablePlayerCheckpoint(i);
                    }
                    return 1;
                }
                    else
                {
                    SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт уже в полёте!");
                }
             }
             if(listitem == 2)
             {
                 if(Sozdan[playerid] == 0) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт не создан!");
                 if(Delete[playerid] == 0)
                {
                      Delete3DTextLabel(Text3D:id);
                      DestroyObject(samolet[0]);
                    DestroyObject(samolet[1]);
                    for(new i=0;i<MAX_PLAYERS;i++)
                    {
                        DisablePlayerCheckpoint(i);
                    }
                    Delete[playerid] = 1;
                    Zapusk[playerid] = 0;
                    Sozdan[playerid] = 0;
                }
                    else
                {
                    SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт уже удалён!");
                }
            }
                else
            {
              }
          }
    } 

И в public OnPlayerEnterCheckpoint(playerid):

 if(parajut == 1)
 {
  GivePlayerWeapon(playerid,46,1);
  return 1;
 } 

---

В принципе всё, опять же: над оптимизацией не работал, главное работает, кому надо тот сделает. Может где-то есть лишняя переменная, но вроде всё на месте
Видео: http://www.youtube.com/watch?v=mrmMhgVu9ME

---

Автор: ProG

[l1ghter]

Да, код подкачивает...

[ProG]

Да, код подкачивает...

Ага ;(

[Folleah]

Нормально

[Snoowker]

if(Sozdan[playerid] == 0) { s = "{FF0000}Не выполнено{ffffff}"; } else { s = "{00CD00}Выполнено{ffffff}"; }
if(Zapusk[playerid] == 0) { z = "{FF0000}Не выполнено{ffffff}"; } else { z = "{00CD00}Выполнено{ffffff}"; }
if(Delete[playerid] == 0) { d = "{FF0000}Не выполнено{ffffff}"; } else { d = "{00CD00}Выполнено{ffffff}"; }

Сблеванул.

Нельзя заюзать new str[][] = {{"Не выполнено"}, {"Выполнено"}}; и обращаться к ним через str[значение]? Дальше этого кода смотреть перестал, Джексона начинает напоминать.

[lovely]

if(Sozdan[playerid] == 0) { s = "{FF0000}Не выполнено{ffffff}"; } else { s = "{00CD00}Выполнено{ffffff}"; }
if(Zapusk[playerid] == 0) { z = "{FF0000}Не выполнено{ffffff}"; } else { z = "{00CD00}Выполнено{ffffff}"; }
if(Delete[playerid] == 0) { d = "{FF0000}Не выполнено{ffffff}"; } else { d = "{00CD00}Выполнено{ffffff}"; }

Сблеванул.

Нельзя заюзать new str[][] = {{"Не выполнено"}, {"Выполнено"}}; и обращаться к ним через str[значение]? Дальше этого кода смотреть перестал, Джексона начинает напоминать.

Он же написал:

В принципе всё, опять же: над оптимизацией не работал, главное работает, кому надо тот сделает.

На*** придираться?

[H1tman]

Немного оптимизации не повредит

new Text3D:Parachute, Plane[2];
#define PLANE_DIALOG 12345 

if(!strcmp(cmdtext, "/samolet", true))
{
    static dialog[150];
    format(dialog, sizeof(dialog), "Создать самолёт [%s]\nЗапустить самолёт [%s]\nУдалить самолёт [%s]",
    (!GetPVarInt(playerid, "PlaneCreated")) ? ("{FF0000}Не выполнено{ffffff}") : ("{00CD00}Выполнено{ffffff}"),
    (!GetPVarInt(playerid, "PlaneMoved")) ? ("{FF0000}Не выполнено{ffffff}") : ("{00CD00}Выполнено{ffffff}"),
    (!GetPVarInt(playerid, "PlaneRemove")) ? ("{FF0000}Не выполнено{ffffff}") : ("{00CD00}Выполнено{ffffff}"));
    return ShowPlayerDialog(playerid,PLANE_DIALOG,DIALOG_STYLE_LIST,"{FF9900}Меню",string,"Выбрать","Отмена"), true;
} 

public OnDialogResponse(playerid)

if(dialogid == PLANE_DIALOG)
{
    if(!response) return true;
    switch(listitem)
    {
        case 0:
        {
            if(GetPVarInt(playerid, "PlaneCreated")) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт уже создан!");
            Parachute = Create3DTextLabel( "Ïàðàøþò! :)", 0x7FFF00FF, 2008.8929,-2532.5046,14.0, 40.0, 0);
            Plane[0] = CreateObject(14553, 1980.20, -2506.91, 25.82, 0.00, 0.00, 220.00), Plane[1] = CreateObject(14548, 1983.20, -2510.50, 25.50, 0.00, 0.00, 220.00);
            SetPVarInt(playerid, "PlaneCreated", 1);
            for(new i = GetMaxPlayers() - 1; i != -1; i--)
            {
                if(!IsPlayerConnected(i)) continue;
                SetPlayerCheckpoint(i, 2008.8929,-2532.5046,13.5469, 1.0), SetPVarInt(i, "GotParachute", 1);
                SendClientMessage(i, 0xff0000ff,"{ff0000}Информация: {ffffff}Желающих полетать на парашюте прибыть в {ADD8E6}Аэропорт г.Лос-Сантос.");
            }
        }
        case 1:
        {
            if(!GetPVarInt(playerid, "PlaneCreated")) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт не создан!");
            if(GetPVarInt(playerid, "PlaneMoved")) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт уже запущен!");
            SetPVarInt(playerid, "PlaneMoved", 1);
            MoveObject(Plane[0], -10.251000404358, -264.27801147461, 2000.32, 40);
            MoveObject(Plane[1], -7.251000404358, -267.86801147461, 2000.0, 40);
            Delete3DTextLabel(Parachute);
            for(new i = GetMaxPlayers() - 1; i != -1; i--) if(IsPlayerConnected(i)) DisablePlayerCheckpoint(i), DeletePVar(i, "GotParachute");
        }
        case 2:
        {
            if(!GetPVarInt(playerid, "PlaneCreated")) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт не создан!");
            if(GetPVarInt(playerid, "PlaneRemove")) return SendClientMessage(playerid, 0xff0000ff, "Ошибка: {ffffff}Самолёт удалён!");
            SetPVarInt(playerid, "PlaneRemove", 1);
            DeletePVar(playerid, "PlaneCreated");
            DeletePVar(playerid, "PlaneMoved");
            Delete3DTextLabel(Parachute), DestroyObject(Plane[0]), DestroyObject(Plane[1]);
            for(new i = GetMaxPlayers() - 1; i != -1; i--) if(IsPlayerConnected(i)) DisablePlayerCheckpoint(i), DeletePVar(i, "GotParachute");
        }
    }
} 

public OnPlayerEnterCheckpoint(playerid)

if(GetPVarInt(playerid, "GotParachute"))
{
    GivePlayerWeapon(playerid, 46, 1);
} 

[l1ghter]

Ох, гангстер, снувкер же написал, как сделать лучше.

new _str[2][28] = {  "{FF0000}Не выполнено{ffffff}", "{00CD00}Выполнено{ffffff}" } ; 

и обращение:

_str[ GetPVarInt(playerid, "PlaneCreated") ] 

[H1tman]

Ох, гангстер, снувкер же написал, как сделать лучше.

new _str[2][28] = {  "{FF0000}Не выполнено{ffffff}", "{00CD00}Выполнено{ffffff}" } ; 

и обращение:

_str[ GetPVarInt(playerid, "PlaneCreated") ] 

Какая разница, какую проверку ставить? Тем более в моей не используется лишних переменных.

[Ленивый]

Какая разница, какую проверку ставить? Тем более в моей не используется лишних переменных.

это не переменная это массив! ><