Движок la2

[Influence]

как минимум

if (hairStyle < 0 || hairStyle > 6)
if (hairColor < 0 || hairColor > 3)

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

if (hairStyle > 4)

там где это нужно
Да и вообще можно сделать один enum с всеми параметрами что были приведены выше, и в несколько строк сверять
так же можно заметить, что if (hairStyle > 4) return false; справедливо только для sex=0, то есть для мужиков

[Camelion]

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

Как же в таком случае написать проверки без такого "говнокода" и существенной потери производительности. Без сарказма если есть вариант лучше с удовольствием посмотрю

Вы сначала померяйте своё существенное падение производительности, а потом уже говорите: было медленно и херово, переписал по другому стало быстрее, график до:; график после:;.
По факту же - эти ваши проверки забиты на уровне ПТС скриптов, достаточно достать оттуда чиселки, положить в мапу и сравнить с тем что получили от клиента. Вот и будет вам быстрое и простое решение.

[DiagoD]

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

По факту же - эти ваши проверки забиты на уровне ПТС скриптов, достаточно достать оттуда чиселки, положить в мапу и сравнить с тем что получили от клиента. Вот и будет вам быстрое и простое решение.

Где и как?)

[Camelion]

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

Где и как?)

Раз появился такой вопрос, значит их там нет? Возможно я ошибся, и эти датники лежат на клиентской стороне. Сути это не меняет, аналогично можно подтягивать их из клиентского файла.

[Alay]

а можно просто открыть клиент и посмотреть в окне создания персонажа, потыкать разные вариации

[Camelion]

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

а можно просто открыть клиент и посмотреть в окне создания персонажа, потыкать разные вариации

О каких вариациях идет речь? Вы приводите псевдокод функции в который затолкали кучу логических проверок, что делает этот код пахнущим тяжелым в отладке и поддержке. Это выглядит глупо, с учетом того что есть возможность воспользоваться детерминированным набором данных.

[DiagoD]

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

Раз появился такой вопрос, значит их там нет? Возможно я ошибся, и эти датники лежат на клиентской стороне. Сути это не меняет, аналогично можно подтягивать их из клиентского файла.

Просто я аж удивился такой инфе, начал чуть ли не в нервном тике пересматривать файлы, чего я там упустил Потом подумал, что мб ты ведешь речь о версии выше ГД и спросил

С цифрами на много проще, достаточно просто подсчитать количество для каждого рассы/полы при создании чара

[Alay]

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

С цифрами на много проще, достаточно просто подсчитать количество для каждого рассы/полы при создании чара

Про вариации - это я и имел ввиду.

Добавлено через 59 секунд

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

Это выглядит глупо, с учетом того что есть возможность воспользоваться детерминированным набором данных.

к сожалению таких данных в pts скриптах не нашел

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

[Gaikotsu]

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

Как же в таком случае написать проверки без такого "говнокода" и существенной потери производительности. Без сарказма если есть вариант лучше с удовольствием посмотрю

оффтоп:

Свернуть ↑
не фонтан, но много более короткий вариант

 
	private static boolean checkAppearance(int classId, int sex, int hairStyle, int hairColor, int face)
	{
		if (sex < 0 || sex > 1 || face < 0 || face > 2 || hairStyle < 0 || hairStyle > 6 || hairColor < 0 || hairColor > 3)
			return false;
 
		if (!PlayerHairStyle.isAllowed(Sex.VALUES[sex], PlayerHairStyle.VALUES[hairStyle]) || !PlayerHairColor.isAllowed(ClassId.VALUES[classId].getRace(), PlayerHairColor.VALUES[hairColor]))
			return false;
 
		return true;
	}
 
Code: Java

и сами задействанные энумы PlayerHairStyle и PlayerHairColor

 
package l2p.gameserver.enums;
 
import l2p.gameserver.network.serverpackets.components.SystemMsg;
import org.apache.commons.lang3.ArrayUtils;
 
public enum PlayerHairStyle
{
	NONE(-1, null),
	A(0, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_A_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	B(1, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_B_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	C(2, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_C_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	D(3, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_D_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	E(4, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_E_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	F(5, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_F_IS_BEING_USED, Sex.FEMALE),
	G(6, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_G_IS_BEING_USED, Sex.FEMALE);
 
	public static final PlayerHairStyle[] VALUES = values();
 
	private final int _id;
	private final SystemMsg _systemMsg;
	private final Sex[] _sex;
 
	private PlayerHairStyle(int id, SystemMsg systemMsg, Sex... sex)
	{
		_id = id;
		_sex = sex;
		_systemMsg = systemMsg;
	}
 
	public int getId()
	{
		return _id;
	}
 
	public SystemMsg getSystemMsg()
	{
		return _systemMsg;
	}
 
	public static boolean isAllowed(Sex sex, PlayerHairStyle hairStyle)
	{
		return ArrayUtils.contains(hairStyle._sex, sex);
	}
}
 
package l2p.gameserver.enums;
 
import l2p.gameserver.network.serverpackets.components.SystemMsg;
import org.apache.commons.lang3.ArrayUtils;
 
public enum PlayerHairColor
{
	NONE(-1, null),
	A(0, SystemMsg.HAIR_DYE_POTION__TYPE_A_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF, Race.KAMAEL, Race.ERTHEIA),
	B(1, SystemMsg.HAIR_DYE_POTION__TYPE_B_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF, Race.KAMAEL, Race.ERTHEIA),
	C(2, SystemMsg.HAIR_DYE_POTION__TYPE_C_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF, Race.KAMAEL, Race.ERTHEIA),
	D(3, SystemMsg.HAIR_DYE_POTION__TYPE_D_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF);
 
	public static final PlayerHairColor[] VALUES = values();
 
	private final int _id;
	private final SystemMsg _systemMsg;
	private final Race[] _race;
 
	private PlayerHairColor(int id, SystemMsg systemMsg, Race... race)
	{
		_id = id;
		_systemMsg = systemMsg;
		_race = race;
	}
 
	public int getId()
	{
		return _id;
	}
 
	public SystemMsg getSystemMsg()
	{
		return _systemMsg;
	}
 
	public static boolean isAllowed(Race race, PlayerHairColor hairColor)
	{
		return ArrayUtils.contains(hairColor._race, race);
	}
}
 
Code: Java

З.Ы. эти энумы у меня изначально не для этих проверок, а задействованы в скиллах для поушнов смены причесок и т.п., потому и куча параметров и всего такого. но так как в целом эти же энумы подходят и для проверок при создании перса, то к чему плодить сущности

Свернуть ↑Развернуть ↓

[Influence]

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

оффтоп:

Свернуть ↑
не фонтан, но много более короткий вариант

 
	private static boolean checkAppearance(int classId, int sex, int hairStyle, int hairColor, int face)
	{
		if (sex < 0 || sex > 1 || face < 0 || face > 2 || hairStyle < 0 || hairStyle > 6 || hairColor < 0 || hairColor > 3)
			return false;
 
		if (!PlayerHairStyle.isAllowed(Sex.VALUES[sex], PlayerHairStyle.VALUES[hairStyle]) || !PlayerHairColor.isAllowed(ClassId.VALUES[classId].getRace(), PlayerHairColor.VALUES[hairColor]))
			return false;
 
		return true;
	}
 
Code: Java

и сами задействанные энумы PlayerHairStyle и PlayerHairColor

 
package l2p.gameserver.enums;
 
import l2p.gameserver.network.serverpackets.components.SystemMsg;
import org.apache.commons.lang3.ArrayUtils;
 
public enum PlayerHairStyle
{
	NONE(-1, null),
	A(0, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_A_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	B(1, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_B_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	C(2, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_C_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	D(3, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_D_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	E(4, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_E_IS_BEING_USED, Sex.MALE, Sex.FEMALE),
	F(5, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_F_IS_BEING_USED, Sex.FEMALE),
	G(6, SystemMsg.THE_HAIR_STYLE_CHANGE_POTION__TYPE_G_IS_BEING_USED, Sex.FEMALE);
 
	public static final PlayerHairStyle[] VALUES = values();
 
	private final int _id;
	private final SystemMsg _systemMsg;
	private final Sex[] _sex;
 
	private PlayerHairStyle(int id, SystemMsg systemMsg, Sex... sex)
	{
		_id = id;
		_sex = sex;
		_systemMsg = systemMsg;
	}
 
	public int getId()
	{
		return _id;
	}
 
	public SystemMsg getSystemMsg()
	{
		return _systemMsg;
	}
 
	public static boolean isAllowed(Sex sex, PlayerHairStyle hairStyle)
	{
		return ArrayUtils.contains(hairStyle._sex, sex);
	}
}
 
package l2p.gameserver.enums;
 
import l2p.gameserver.network.serverpackets.components.SystemMsg;
import org.apache.commons.lang3.ArrayUtils;
 
public enum PlayerHairColor
{
	NONE(-1, null),
	A(0, SystemMsg.HAIR_DYE_POTION__TYPE_A_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF, Race.KAMAEL, Race.ERTHEIA),
	B(1, SystemMsg.HAIR_DYE_POTION__TYPE_B_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF, Race.KAMAEL, Race.ERTHEIA),
	C(2, SystemMsg.HAIR_DYE_POTION__TYPE_C_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF, Race.KAMAEL, Race.ERTHEIA),
	D(3, SystemMsg.HAIR_DYE_POTION__TYPE_D_IS_BEING_APPLIED, Race.HUMAN, Race.ELF, Race.DARK_ELF, Race.ORC, Race.DWARF);
 
	public static final PlayerHairColor[] VALUES = values();
 
	private final int _id;
	private final SystemMsg _systemMsg;
	private final Race[] _race;
 
	private PlayerHairColor(int id, SystemMsg systemMsg, Race... race)
	{
		_id = id;
		_systemMsg = systemMsg;
		_race = race;
	}
 
	public int getId()
	{
		return _id;
	}
 
	public SystemMsg getSystemMsg()
	{
		return _systemMsg;
	}
 
	public static boolean isAllowed(Race race, PlayerHairColor hairColor)
	{
		return ArrayUtils.contains(hairColor._race, race);
	}
}
 
Code: Java

З.Ы. эти энумы у меня изначально не для этих проверок, а задействованы в скиллах для поушнов смены причесок и т.п., потому и куча параметров и всего такого. но так как в целом эти же энумы подходят и для проверок при создании перса, то к чему плодить сущности

Свернуть ↑Развернуть ↓

вот именно про такое решение я и говорил выше