Правильно ли построил условия

Привет. Хочу убедиться в правильности условия. А если если есть возможность, то и упростить.

Цель:

Цитата:

---

Если kill_detected больше нуля, то иконка игрока красного цвета, а если еще BagWithMoneyPlayer равно 1, то чередуются красного и желтого цвета. В противном случае прозрачная иконка.

---

Работает в секундном таймере.

PHP код:

---

if (kill_detected[playerid] != 0) // if (kill_detected[playerid])
{
    kill_detected[playerid]--;

    if (BagWithMoneyPlayer{playerid} == 1)
    {
        if (kill_detected[playerid] % 2 == 0)
        {
            SetPlayerColor(playerid, COLOR_RED1); // красный
        }
        else
        {
            SetPlayerColor(playerid, COLOR_BIGMONEY); // желтый
        }
    }
    else
    {
        SetPlayerColor(playerid, COLOR_RED1); // красный
    }
}
else
{
    SetPlayerColor(playerid, COLOR_WHITE_A); // прозрачный
} 


---

  Открыть/закрыть

PHP код:

---

if (kill_detected[playerid])
{
    kill_detected[playerid]--;

    if (BagWithMoneyPlayer{playerid})
    {
        if (kill_detected[playerid] % 2 == 0)
        {
            SetPlayerColor(playerid, COLOR_RED1);
        }
        else
        {
            SetPlayerColor(playerid, COLOR_BIGMONEY);
        }
    }
    else
    {
        SetPlayerColor(playerid, COLOR_RED1);
    }
}
else
{
    SetPlayerColor(playerid, COLOR_WHITE_A);
} 


---

PHP код:

---

if(kill_detected[playerid])
{
    kill_detected[playerid]--;

    if(BagWithMoneyPlayer{playerid})
        SetPlayerColor(playerid, 
            (kill_detected[playerid] % 2) ? COLOR_BIGMONEY : COLOR_RED1);
    else
        SetPlayerColor(playerid, COLOR_RED1);
}
else
    SetPlayerColor(playerid, COLOR_WHITE_A); 


---

Могу предложить такой вариант, если конечно до этого твой код нормально функционировал

Все же есть ошибка в построении. Если kill_detected равно 0, а BagWithMoneyPlayer нет. Нет вероятности исключительно для желтой иконки.

PHP код:

---

if (kill_detected[playerid] && BagWithMoneyPlayer{playerid})
{
    kill_detected[playerid]--;
    SetPlayerColor(playerid, (kill_detected[playerid] % 2) ? COLOR_RED1 : COLOR_BIGMONEY);
}
else if (kill_detected[playerid])
{
    kill_detected[playerid]--;
    SetPlayerColor(playerid, COLOR_RED1);
}
else if (BagWithMoneyPlayer{playerid})
{
    SetPlayerColor(playerid, COLOR_BIGMONEY);
}
else
{
    SetPlayerColor(playerid, COLOR_WHITE_A);
} 


---

PHP код:

---

if(-- kill_detected[playerid] != 0)
{
    SetPlayerColor(playerid,
    (
        !BagWithMoneyPlayer{playerid} ? COLOR_RED1 : COLOR_BIGMONEY)
    );
}
else
    SetPlayerColor(playerid,
    (
        !BagWithMoneyPlayer{playerid} ? COLOR_WHITE_A : COLOR_BIGMONEY)
    ); 


---

Цитата:

---

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

PHP код:

---

if(-- kill_detected[playerid] != 0)
{
    SetPlayerColor(playerid,
    (
        !BagWithMoneyPlayer{playerid} ? COLOR_RED1 : COLOR_BIGMONEY)
    );
}
else
    SetPlayerColor(playerid,
    (
        !BagWithMoneyPlayer{playerid} ? COLOR_WHITE_A : COLOR_BIGMONEY)
    ); 


---

---

Спасибо. Но тогда kill_detected[playerid] станет меньше нуля, и упустил кратность. Получается такое:

PHP код:

---

if(kill_detected[playerid])
{
    kill_detected[playerid]--;
    SetPlayerColor(playerid, 
    (
        (!BagWithMoneyPlayer{playerid}) ?
        COLOR_RED1 : ((kill_detected[playerid] % 2) ? COLOR_RED1 : COLOR_BIGMONEY))
    );
}
else
{
    SetPlayerColor(playerid, (!BagWithMoneyPlayer{playerid}) ? COLOR_WHITE_A : COLOR_BIGMONEY);
} 


---

)