Hero Aura

[Influence]

А что, возможно, что _activeChar.getFactionId() будет равен 0 или 3? Если нет - то зачем второе условие? Просто else.
PS: Приучайте себя все условные операторы окружать блоком { } (как в моем примере), на много читабельнее, и это хороший тон.

Думаю он изначально будет равен нулю, так как фракцию, как видно по дифу(из другой темы), будут выбирать.

п.с. досмотрел что это только для геройки

[Phantomas]

да работает :
if(_activeChar.isHero()){
if(_activeChar.getFactionId() == 1){
writeC(0x01);
} else {
writeC(0x02);
}
} else {
writeC(0x00);
}

а какая разница в этих 2 ух мои кодах?
только читать легче или ище что то?
У меня свой файл L2FactionInstance.java в ядре если это к теме я написал)

[Johnson]

Да, читать легче. Если будешь потом с кем-то работать на пару - он будет благодарен за это.