Движок la2

[Alay]

оффтоп

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

Вообще не ожидал что этим еще кто-то запаривался

[Gaikotsu]

Вообще не ожидал что этим еще кто-то запаривался

Просто я стараюсь руководствоваться принципом "если есть возможность прислать невалидные данные, то игроки этим обязательно воспользуются" и посему стараюсь по возможности проверять и перепроверять все что присылает клиент, ибо по определению "все врут" (с) ^_^

[DiagoD]

Вообще не ожидал что этим еще кто-то запаривался

У каждого свои ожидания, есть кто не ожидал, что еще кто-то не сделал валидацию входящих данных.

[Alay]

Но на шарпе так не получиться красиво провернуть через энум. В шарпе энум это константа. в яве - инстанс. (enum может быль лишь базовым типом - int char и тд. Следовательно придется делать структуру и добавлять ее в массив и потом уже опрашивать... и вся эта красота пропадает.

[flopix]

Кстати на ПТС те же грабли.
Когда ввели GoD хроники на американских серверах создание персов с неверными параметрами работало и перс был невидимым.

[Alay]

Кстати на ПТС те же грабли.
Когда ввели GoD хроники на американских серверах создание персов с неверными параметрами работало и перс был невидимым.

Вот это поворот...

Добавлено через 18 минут
не знаю как вам, а лично мне работать над таким кодом не очень приятно.

Код:

internal struct HStyle
    {
        private readonly int[] _sex;

        public HStyle(int[] sex)
        {
            _sex = sex;
        }

        public bool IsAllowed(Appearance.BaseSex sex)
        {
            return _sex.Any(t => t == (int)sex);
        }
    }
    internal struct HColor
    {
        private readonly int[] _race;

        public HColor(int[] race)
        {
            _race = race;
        }

        public bool IsAllowed(Appearance.BaseRace race)
        {
            return _race.Any(t => t == (int)race);
        }
    }


        public static bool CheckAppearenceV2(BaseRace baseRace, BaseSex baseSex, int baseClassId, int face, int hairStyle,
            int hairColor)
        {
            if (baseSex < 0 || (int)baseSex > 1 ||
                face < 0 || face > 2 ||
                hairStyle < 0 || hairStyle > 6 ||
                hairColor < 0 || hairColor > 3 ||
                baseRace == BaseRace.Ethria && baseSex == BaseSex.Male)
                return false;
            return IsValidHairStyle(baseSex, hairStyle) && IsValidHairColor(baseRace, hairColor);
        }

        #region Private

        private static readonly HStyle[] HairStyles =
        {
            new HStyle(new []{ (int)BaseSex.Male, (int)BaseSex.Female}) ,
            new HStyle(new []{ (int)BaseSex.Male, (int)BaseSex.Female}) ,
            new HStyle(new []{ (int)BaseSex.Male, (int)BaseSex.Female}) ,
            new HStyle(new []{ (int)BaseSex.Male, (int)BaseSex.Female}) ,
            new HStyle(new []{ (int)BaseSex.Male, (int)BaseSex.Female}) ,
            new HStyle(new []{ (int)BaseSex.Female}),
            new HStyle(new []{ (int)BaseSex.Female})
        };
        private static readonly HColor[] HairColors =
        {
            new HColor(new []{ (int)BaseRace.Human, (int)BaseRace.Elf, (int)BaseRace.DarkElf, (int)BaseRace.Ork, (int)BaseRace.Dwarf, (int)BaseRace.Kamael, (int)BaseRace.Ethria}),
            new HColor(new []{ (int)BaseRace.Human, (int)BaseRace.Elf, (int)BaseRace.DarkElf, (int)BaseRace.Ork, (int)BaseRace.Dwarf, (int)BaseRace.Kamael, (int)BaseRace.Ethria}),
            new HColor(new []{ (int)BaseRace.Human, (int)BaseRace.Elf, (int)BaseRace.DarkElf, (int)BaseRace.Ork, (int)BaseRace.Dwarf, (int)BaseRace.Kamael, (int)BaseRace.Ethria}),
            new HColor(new []{ (int)BaseRace.Human, (int)BaseRace.Elf, (int)BaseRace.DarkElf, (int)BaseRace.Ork, (int)BaseRace.Dwarf})
        };
        private static bool IsValidHairStyle(BaseSex sex, int hairStyle)
        {
            if (hairStyle < 0 || hairStyle > HairStyles.Length) return false;
            return HairStyles[hairStyle].IsAllowed(sex);
        }
        private static bool IsValidHairColor(BaseRace race, int hairColor)
        {
            if (hairColor < 0 || hairColor > HairColors.Length) return false;
            return HairColors[hairColor].IsAllowed(race);
        }
        #endregion

причина по которой я за него не взялся - не мог сообразить что брать за ключ чтобы чекать все сразу в одной сущности а не разбивать на проверки стиля волос цвета к расе и тд. А в итоге получем несколько несвязных зависимостей из за которых все равно придется залезать "в тухлые свичи" та же проверка на мужика к артейке к примеру.
На мой взгяд тут вообще все относительно и мой вариант в плане масштабируемости и поддержке был куда проще. по крайней мере для меня

[Gaikotsu]

Кстати на ПТС те же грабли.
Когда ввели GoD хроники на американских серверах создание персов с неверными параметрами работало и перс был невидимым.

на птс граблей хватает, это да.
притом о них все знают, но нцсофт нихрена их не чинит.

чего стоит возможность к примеру тягать любые мелкие файлы (до 8/16кб размером) с сервера, через пакеты. и дырка эта существует еще с самых первых хроник и не знают о ней по моему уже только самые ленивые.
А ведь всех делов то, добавить проверку к примеру на то, что запрашиваемый файл имеет тип htm/html и все - ничего кроме хтмлок вытащить с сервера не удастся.

[Hack]

Вообще не ожидал что этим еще кто-то запаривался

Действительно, зачем проверять входные данные от пользователя?

[Alay]

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

Код:

private static void Main()
        {
            var sw = new Stopwatch();
            for (var i = 0; i < 20; i++)
            {
                sw.Restart();
                var result = Appearance.CheckAppearence(Appearance.BaseRace.Ethria, Appearance.BaseSex.Female, 183, 1, 0, 0);
                sw.Stop();
                Console.WriteLine($"Тест моего говнокода. Прогон {i + 1}й занял {sw.ElapsedNanoSeconds()} наносекунд. Результат {result}");
            }
            for (var i = 0; i < 20; i++)
            {
                sw.Restart();
                var result = Appearance.CheckAppearenceV2(Appearance.BaseRace.Ethria, Appearance.BaseSex.Female, 183, 1, 0, 0);
                sw.Stop();
                Console.WriteLine($"Тест красивой обертки. Прогон {i + 1}й занял {sw.ElapsedNanoSeconds()} наносекунд. Результат {result}");
            }
            Console.ReadLine();
        }

На первый прогон не обращать внимание - трансляция Jit'ом.
Понятно что цифры не существенные. но все же. а так как win - не система реального времени - более точные цифры мы не получим

[flopix]

Смысл тонкой оптимизации такого кода если он выполняется редко и не крутится в цикле?