L2Skill.java проверка на таргет

[Noize]

КОД1

[SRC="java"] public final L2Object[] getTargetList(L2Character activeChar, boolean onlyFirst)
{
// Init to null the target of the skill
L2Character target = null;

// Get the L2Objcet targeted by the user of the skill at this moment
L2Object objTarget = activeChar.getTarget();
// If the L2Object targeted is a L2Character, it becomes the L2Character target
if (objTarget instanceof L2Character)
{
target = (L2Character) objTarget;
}

return getTargetList(activeChar, onlyFirst, target);
}[/SRC]

Конечный код

[SRC="java"] public final L2Object[] getTargetList(L2Character activeChar, boolean onlyFirst)
{

// Init to null the target of the skill
L2Character target = null;

// Get the L2Objcet targeted by the user of the skill at this moment
L2Object objTarget = activeChar.getTarget();
// If the L2Object targeted is a L2Character, it becomes the L2Character target
if (objTarget instanceof L2Character)
{
target = (L2Character) objTarget;
}

if (target instanceof L2PcInstance)
{
if (((L2PcInstance) activeChar).isGood() && ((L2PcInstance) target).isGood())
{
return _emptyTargetList;
}
if (((L2PcInstance) activeChar).isGood() && ((L2PcInstance) target).isGood())
{
return _emptyTargetList;
}
}

return getTargetList(activeChar, onlyFirst, target);
}[/SRC]

Хотел реализовать проверку на скилы, чтобы нельзя было бить свою фракцию. Попробовал по разному, но не как не получается, выше написал код, что не правильно? Я думаю из-за того что таргет проверяется из L2Object, а фракции есть только у игроков, как исправить?
Исходники l2jserver.

КОД ОШИБКИ

[SRC="java"]мар 22, 2013 1:14:25 PM com.l2jserver.gameserver.model.actor.L2Character callSkill
WARNING: L2PcInstance: callSkill() failed.
java.lang.ClassCastException: [Lcom.l2jserver.gameserver.model.L2Object; cannot be cast to [Lcom.l2jserver.gameserver.model.actor.L2Character;
at handlers.skillhandlers.Continuous.useSkill(Continuous.java:78)
at com.l2jserver.gameserver.model.actor.L2Character.callSkill(L2Character.java:6886)
at com.l2jserver.gameserver.model.actor.L2Character.onMagicHitTimer(L2Character.java:6583)
at com.l2jserver.gameserver.model.actor.L2Character$MagicUseTask.run(L2Character.java:3037)
at com.l2jserver.gameserver.ThreadPoolManager$RunnableWrapper.run(ThreadPoolManager.java:86)
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(Unknown Source)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)[/SRC]

[Ashe]

А что мешает проверять в методах doCast/doAttack ?

[Noize]

[SRC="java"] public void doCast(L2Skill skill, L2Character target, L2Object[] targets)
{
if (!checkDoCastConditions(skill))
{
setIsCastingNow(false);
return;
}

if (target instanceof L2PcInstance)
{
if (skill.isOffensive() && ((L2PcInstance)this).isGood() && ((L2PcInstance)target).isGood())
{
sendPacket(ActionFailed.STATIC_PACKET);
return;
}
if (skill.isOffensive() && ((L2PcInstance)this).isEvil() && ((L2PcInstance)target).isEvil())
{
sendPacket(ActionFailed.STATIC_PACKET);
return;
}
}
// Override casting type
if (skill.isSimultaneousCast())
{
doSimultaneousCast(skill, target, targets);
return;
}

stopEffectsOnAction();

// Recharge AutoSoulShot
// this method should not used with L2Playable

beginCast(skill, false, target, targets);
}[/SRC]
Скилы по прежнему кастуются. Что не верно в моем коде?

[Ashe]

Dubstep, в приведённом коде всё норм. Покажите реализацию isGood/isEvil.

[Noize]

[SRC="java"] public boolean isGood()
{
return _isGood;
}

public boolean isEvil()
{
return _isEvil;
}[/SRC]

В этом все нормально, сделал цвет для этих фракций и авто-атакой бить нельзя. А вот с магическими и физическими умениями проблема.
Странно, в L2Skill сделал проверку на масс. скилы. Свою фракцию они не задевают.