Движок la2

[Influence]

как минимум

Код:

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

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

Код:

if (hairStyle > 4)

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

[Camelion]

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

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

[DiagoD]

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

Где и как?)

[Camelion]

Где и как?)

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

[Alay]

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

[Camelion]

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

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

[DiagoD]

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

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

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

[Alay]

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

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

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

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

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

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

[Gaikotsu]

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

оффтоп

не фонтан, но много более короткий вариант
[SRC="java"]
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;
}
[/SRC]
и сами задействанные энумы PlayerHairStyle и PlayerHairColor
[SRC="java"]
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);
}
}
[/SRC]
З.Ы. эти энумы у меня изначально не для этих проверок, а задействованы в скиллах для поушнов смены причесок и т.п., потому и куча параметров и всего такого. но так как в целом эти же энумы подходят и для проверок при создании перса, то к чему плодить сущности

[Influence]

оффтоп

не фонтан, но много более короткий вариант
[SRC="java"]
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;
}
[/SRC]
и сами задействанные энумы PlayerHairStyle и PlayerHairColor
[SRC="java"]
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);
}
}
[/SRC]
З.Ы. эти энумы у меня изначально не для этих проверок, а задействованы в скиллах для поушнов смены причесок и т.п., потому и куча параметров и всего такого. но так как в целом эти же энумы подходят и для проверок при создании перса, то к чему плодить сущности

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