Оптимизировать код

Можно ли как нибудь оптимизировать код

PHP код:

---

if(IsPlayerInAnyVehicle(playerid))
    {
         if(newkeys & 65536)
          {
            if(GetPVarInt(playerid,"gInvs")) return true;
              ShowInv(playerid);
            SetPVarInt(playerid, "gInv", 1);
        }
        return 1;
    }
    else
    {
        if(newkeys == KEY_CTRL_BACK)
        {
            if(GetPVarInt(playerid,"gInvs")) return true;
             ShowInv(playerid);
              SetPVarInt(playerid, "gInv", 1);
        }
    } 


---

Лучше на уроки по русскому языку в школу ходи.

Цитата:

---

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

Лучше на уроки по русскому языку в школу ходи.

---

Можно ли как нибудь оптимизировать код ?

Цитата:

---

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

Можно ли как нибудь оптимизировать код

PHP код:

---

if(IsPlayerInAnyVehicle(playerid))
    {
         if(newkeys & 65536)
          {
            if(GetPVarInt(playerid,"gInvs")) return true;
              ShowInv(playerid);
            SetPVarInt(playerid, "gInv", 1);
        }
        return 1;
    }
    else
    {
        if(newkeys == KEY_CTRL_BACK)
        {
            if(GetPVarInt(playerid,"gInvs")) return true;
             ShowInv(playerid);
              SetPVarInt(playerid, "gInv", 1);
        }
    } 


---

---

PHP код:

---

new right_key = IsPlayerInAnyVehicle(playerid) ? KEY_YES : KEY_CTRL_BACK;

if (right_key & KEY_YES)
{
    if (GetPVarInt(playerid, "gInvs"))
        return true;

    ShowInv(playerid);
    SetPVarInt(playerid, "gInv", 1);
} 


---

Цитата:

---

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

PHP код:

---

if (right_key & KEY_YES) 


---

---

Наверное, вы имели в виду "if (newkeys & right_key)" ?

Как бы то ни было, это не совсем правильная оптимизация. Дело в том, что в исходном варианте в одном случае нажатие клавиши проверяется с помощью "&", а в другом с помощью "==", т.е. не допускается нажатие других клавиш.
Поэтому более корректным будет следующий вариант:

PHP код:

---

    if(IsPlayerInAnyVehicle(playerid) && (newkeys & 65536) || (newkeys == KEY_CTRL_BACK))
    {
        if(GetPVarInt(playerid,"gInvs"))
            return true;
        ShowInv(playerid);
        SetPVarInt(playerid, "gInv", 1);
    } 


---

UPD: Беру свои слова обратно. В одной из веток if был return 1, а в другой нет.
Топикстартеру: оставляйте свой код, как есть. Или решайте, что делать с return 1: убрать его в одной из веток if или добавить в конец другой.