array mob's in quest

[weTr1k]

Прошу совета.
Сборка - посл. шара феникса
Сообственно в квесте определен массив(заполненый id мобов) мобов

Код:

int[] mobs = {22321, 22320,
    22324, 22325, 22327, 22329,
    22328, 22330, 22335, 22339,
    22334, 22337, 22340, 32350,
    22336, 22349, 22351, 22352,
    22353, 22451, 22448, 22343,
    22342, 18493};

и переменная главного квестового моба

Код:

private static int MOB = 21320;

По квесту я ограничиваю игрока от убийства мобов содержащихся в массиве, следующим образом:

Код:

public String onKill(L2NpcInstance npc, QuestState st)
    {
        int npcId = npc.getNpcId();
        L2Player player = st.getPlayer();
        if(npcId == MOB)
        {
            st.setCond(2);
            st.giveItems(KRIS, 1); //игрок получает квестовый предмет
            st.getPlayer().sendPacket(new ExShowScreenMessage("Поздравляю, вы убили МОБА", 5000, ScreenMessageAlign.TOP_CENTER, true));
        }
        else
        {
          [COLOR="Red"]for(int i:mobs)
          {
            if(npcId == mobs[i])
            {
               st.exitCurrentQuest(false);
            }
          }[/COLOR]    
        }

само собой выделенное красным ограничивает игрока и отменяет квест, и в этом вопрос, не слишком ли это тяжко или по нубски, таким образом решать такую задачу?

[krisadr]

я не силен в яве но строчка
for(int i:mobs)
"впихивает массив в одну переменную" а потом вы еще ее в массив тот же пихаете ...у вас получилось помоему так mobs[mobs]. и логики я тут не вижу. но может я не прав)

if(npcId == i)
так вроде нада

[weTr1k]

я не силен в яве но строчка
for(int i:mobs)
"впихивает массив в одну переменную" а потом вы еще ее в массив тот же пихаете ...у вас получилось помоему так mobs[mobs]. и логики я тут не вижу. но может я не прав)

if(npcId == i)
так вроде нада

нет, вы не правы

[Gaikotsu]

Код:

public String onKill(L2NpcInstance npc, QuestState st)
    {
        int npcId = npc.getNpcId();
        L2Player player = st.getPlayer();
        if(npcId == MOB)
        {
             st.setCond(2);
            st.giveItems(KRIS, 1); //игрок получает квестовый предмет
            st.getPlayer().sendPacket(new ExShowScreenMessage("Поздравляю, вы убили МОБА", 5000, ScreenMessageAlign.TOP_CENTER, true));
        }
        else
        {
             st.exitCurrentQuest(false);
        }
    }

этого по идее вполне достаточно

[weTr1k]

Код:

public String onKill(L2NpcInstance npc, QuestState st)
    {
        int npcId = npc.getNpcId();
        L2Player player = st.getPlayer();
        if(npcId == MOB)
        {
             st.setCond(2);
            st.giveItems(KRIS, 1); //игрок получает квестовый предмет
            st.getPlayer().sendPacket(new ExShowScreenMessage("Поздравляю, вы убили МОБА", 5000, ScreenMessageAlign.TOP_CENTER, true));
        }
        else
        {
             st.exitCurrentQuest(false);
        }
    }

этого по идее вполне достаточно

Так изначально пробывал, не работает.

[krisadr]

нет, вы не правы

ну ладна , хотя всетаки я прав...

[weTr1k]

хотя всетаки я прав...

Интересно в чем?

[krisadr]

lдопустим если вам нада чтобы при убийстве мобов в массиве mobs персонаж выходил из квеста то хватит достаточно
for(int i:mobs)
if(npc.getNpcId() = i) {
st.exitCurrentQuest(false);
}

Добавлено через 2 минуты
ну такое работает на лыже...
да к стати зарегистрировал мобов в
for(i:mobs)
addKillId(i)

[weTr1k]

я не силен в яве но строчка
for(int i:mobs)
"впихивает массив в одну переменную" а потом вы еще ее в массив тот же пихаете ...у вас получилось помоему так mobs[mobs]. и логики я тут не вижу. но может я не прав)

if(npcId == i)
так вроде нада

чистый бед

lдопустим если вам нада чтобы при убийстве мобов в массиве mobs персонаж выходил из квеста то хватит достаточно
for(int i:mobs)
if(npc.getNpcId() = i) {
st.exitCurrentQuest(false);
}

Добавлено через 2 минуты
ну такое работает на лыже...
да к стати зарегистрировал мобов в
for(i:mobs)
addKillId(i)

имхо топорнее

[krisadr]

ну извените я не обладаю познаниями ярого программиста, самоучка я, но голова на плечах имеется. объясните почему бред...

for (int id : MOBS)
addKillId(id);
когда эта фишка работает на ура...