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
		{
		  for(int i:mobs)
		  {
		    if(npcId == mobs[i])
		    {
			   st.exitCurrentQuest(false);
	        }
		  }	
		}

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

[krisadr]

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

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

[weTr1k]

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

я не силен в яве но строчка
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]

Сообщение от 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);
		}
	}

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

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

[krisadr]

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

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

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

[weTr1k]

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

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

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

[krisadr]

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

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

[weTr1k]

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

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

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

чистый бед

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

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);
когда эта фишка работает на ура...