Beauty Shop

[Mifesto]

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

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

С 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;

        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;
        }

        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;
 
		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;
 
		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]

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

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

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

[Ashe]

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

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

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

[Gaikotsu]

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

[Volond1]

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

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

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

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

[Daan Raven]

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