Beauty Shop

[Mifesto]

Ты бы уточнил о чём говоришь.

С 90% вероятностью что про это:

Код:

package l2m.gameserver.data.xml.holder;

import gnu.trove.TIntObjectHashMap;
import l2m.commons.data.xml.AbstractHolder;
import l2m.gameserver.model.Player;
import l2m.gameserver.model.base.Race;
import l2m.gameserver.model.base.Sex;
import l2m.gameserver.model.beautyshop.BeautyShopSet;

/**
* @author: Gaikotsu
*/
public final class BeautyShopHolder extends AbstractHolder
{
    private static final BeautyShopHolder _instance = new BeautyShopHolder();

    public static BeautyShopHolder getInstance()
    {
        return _instance;
    }

    private TIntObjectHashMap<BeautyShopSet> _sets = new TIntObjectHashMap<>();


    public void addSet(BeautyShopSet set)
    {
        if (_sets.containsKey(set.getId()))
            warn("Duplicate set declaration, set id - " + set.getId());

        _sets.put(set.getId(), set);
    }

    public BeautyShopSet getSet(int id)
    {
        return _sets.get(id);
    }

    public BeautyShopSet getSet(Player player)
    {
        int id = -1;

        [COLOR="Red"]switch (player.getRace())
        {
            case human:
                id = player.getClassId().isMage() ? (player.getSex() == Sex.MALE.ordinal() ? 8 : 9) : (player.getSex() == Sex.MALE.ordinal() ? 0 : 1);
                break;
            case elf:
                id = player.getSex() == Sex.MALE.ordinal() ? 6 : 7;
                break;
            case darkelf:
                id = player.getSex() == Sex.MALE.ordinal() ? 2 : 3;
                break;
            case orc:
                id = player.getClassId().isMage() ? (player.getSex() == Sex.MALE.ordinal() ? 12 : 13) : (player.getSex() == Sex.MALE.ordinal() ? 10 : 11);
                break;
            case dwarf:
                id = player.getSex() == Sex.MALE.ordinal() ? 4 : 5;
                break;
            case kamael:
                id = player.getSex() == Sex.MALE.ordinal() ? 14 : 15;
                break;
        }[/COLOR]

        return getSet(id);
    }

    @Override
    public int size()
    {
        return _sets.size();
    }

    @Override
    public void clear()
    {
        _sets.clear();
    }

    @Override
    public void log()
    {
        info(String.format("loaded %d beauty shop set(s) count.", size()));
    }
}

[Ashe]

Mifesto, а где же "древо ифов вместо кейса?)" в таком случае?

[Mifesto]

Я дал уже исправленный мною файл, древо ифов выше в теме.

Код:

package l2p.gameserver.data.xml.holder;

import gnu.trove.map.hash.TIntObjectHashMap;
import l2p.commons.data.xml.AbstractHolder;
import l2p.gameserver.model.Player;
import l2p.gameserver.model.base.Race;
import l2p.gameserver.model.base.Sex;
import l2p.gameserver.model.beautyshop.BeautyShopSet;

public final class BeautyShopHolder extends AbstractHolder
{
    private static final BeautyShopHolder _instance = new BeautyShopHolder();
    private static final TIntObjectHashMap<BeautyShopSet> _sets = new TIntObjectHashMap<BeautyShopSet>();

    public static BeautyShopHolder getInstance()
    {
        return _instance;
    }

    public void addSet(BeautyShopSet set)
    {
        if (_sets.containsKey(set.getId()))
            warn("Duplicate set declaration, set id - " + set.getId());

        _sets.put(set.getId(), set);
    }

    public BeautyShopSet getSet(int id)
    {
        return _sets.get(id);
    }

    public BeautyShopSet getSet(Player player)
    {
        int id = -1;

        [COLOR="Red"]if (player.getRace() == Race.HUMAN)
        {
            if (!player.getClassId().isMage())
                id = player.getSex() == Sex.MALE ? 0 : 1;
            else
                id = player.getSex() == Sex.MALE ? 8 : 9;
        }
        else if (player.getRace() == Race.DARK_ELF)
            id = player.getSex() == Sex.MALE ? 2 : 3;
        else if (player.getRace() == Race.DWARF)
            id = player.getSex() == Sex.MALE ? 4 : 5;
        else if (player.getRace() == Race.ELF)
            id = player.getSex() == Sex.MALE ? 6 : 7;
        else if (player.getRace() == Race.ORC)
        {
            if (!player.getClassId().isMage())
                id = player.getSex() == Sex.MALE ? 10 : 11;
            else
                id = player.getSex() == Sex.MALE ? 12 : 13;
        }
        else if (player.getRace() == Race.KAMAEL)
            id = player.getSex() == Sex.MALE ? 14 : 15;[/COLOR]

        return getSet(id);
    }

    @Override
    public int size()
    {
        return _sets.size();
    }

    @Override
    public void clear()
    {
        _sets.clear();
    }

    @Override
    public void log()
    {
        info(String.format("loaded %d beauty shop set(s) count.", size()));
    }
}

[linliss]

Mifesto, а где же "древо ифов вместо кейса?)" в таком случае?

при маленьком количестве условий, что свич, что иф одно и то же...

[Ashe]

при маленьком количестве условий, что свич, что иф одно и то же...

Спасибо, буду знать

[Gaikotsu]

ну вот выше уже все сказали - при малом количестве вариантов особой разницы нет, так что уже чисто дело вкуса.

[Volond1]

Думаю все в курсе, что в Lindvior ввели Beauty Shop (салон красоты), предназначенный для изменения внешнего вида персонажей (прически, цвета волос, лица).

Ну вот собственно и все. Если я что и не описал или не пояснил - это в основном мелочи, до которых вы сами сможете без проблем додуматься.

Пояснил, разжевал ну вообще суперски. Большущее тебе СПАСИБИЩЕ.

[Daan Raven]

Респект автору огромный!!!