L2 MMOLAB (Public Work)

Их важность = 0. Конфиги пилят команды которые проживут не очень долго и все это по принципу ctrl+c и ctrl+v.
У любой разработки смотрятся сначала главные аспекты, а не второстепенные которые ну по сути ничего с собой не несут.
А причем тут это? Я про оформление в кб говорил, но не как сайта. И если так, то можно спокойно оформить и по своему redmine.
Кпд работы над сборкой = 0
 
Now the repository available to view / Изменения исходного кода теперь доступны для просмотра
Вы не можете просматривать ссылку пожалуйста воспользуйтесь следующими ссылками Вход или Регистрация

Вы не можете просматривать ссылку пожалуйста воспользуйтесь следующими ссылками Вход или Регистрация
 
Реакции: kick

Код: 
package org.mmocore.gameserver.handler.bbs.custom.privateOffice.services.player.appearance;
import handler.bbs.abstracts.AbstractCommunityBoard;
import org.mmocore.gameserver.handler.bbs.custom.privateOffice.engine.annotations.Init;
import org.mmocore.gameserver.handler.bbs.custom.privateOffice.engine.enums.Services;
import org.mmocore.gameserver.handler.bbs.custom.privateOffice.engine.interfaces.IService;
import org.mmocore.gameserver.handler.bbs.custom.privateOffice.services.player.appearance.enums.EFace;
import org.mmocore.gameserver.handler.bbs.custom.privateOffice.services.player.appearance.enums.ETypes;
import org.apache.commons.lang3.ArrayUtils;
import org.mmocore.gameserver.configuration.config.community.CBasicConfig;
import org.mmocore.gameserver.model.entity.events.impl.SiegeEvent;
import org.mmocore.gameserver.network.lineage.components.CustomMessage;
import org.mmocore.gameserver.object.Player;
import org.mmocore.gameserver.utils.Log;

import java.util.Optional;

/**
* @author Mangol
* @since 29.02.2016
*/
@Init(service = Services.change_face)
public class ChangeFaceService extends AbstractCommunityBoard implements IService
{
    @Override
    public Services getService()
    {
        return Services.change_face;
    }

    @Override
    public void content(final Player player, final String bypass, final Object... params)
    {
        final ETypes[] face = EFace.getFace(player.getPlayerTemplateComponent().getPlayerRace(), player.getPlayerTemplateComponent().getPlayerSex());
        String htm = getCache().getHtml(CBasicConfig.BBS_PATH + "/cabinet/service/player/changeFace/changeFace.htm", player);
        String table = getCache().getHtml(CBasicConfig.BBS_PATH + "/cabinet/service/player/changeFace/table.htm", player);
        String button = getCache().getHtml(CBasicConfig.BBS_PATH + "/cabinet/service/player/changeFace/button.htm", player);
        final StringBuilder buttons = new StringBuilder();
        for(int correctCount = 0; correctCount <= face.length - 1; )
        {
            if(face[correctCount].ordinal() == player.getAppearanceComponent().getFace())
            {
                correctCount++;
                continue;
            }
            String generateButton = button;
            generateButton = generateButton.replace("<?face_name?>", String.valueOf(face[correctCount].name().toUpperCase()));
            generateButton = generateButton.replace("<?face?>", String.valueOf(face[correctCount].name()));
            buttons.append(generateButton);
            correctCount++;
        }
        table = table.replace("<?buttons?>", buttons.toString());
        htm = htm.replace("<?tables?>", table);
        htm = htm.replace("<?correct_face?>", ETypes.getName(player.getAppearanceComponent().getFace()).toUpperCase());
        htm = htm.replace("<?price?>", String.valueOf(getService().getItemCount()));
        htm = htm.replace("<?item_name?>", getItemName(player.getLanguage(), getService().getItemId()));
        htm = htm.replace("<?price_wear?>", String.valueOf(getService().getWearItemCount()));
        htm = htm.replace("<?item_name_wear?>", getItemName(player.getLanguage(), getService().getWearItemId()));
        separateAndSend(htm, player);
    }

    @Override
    public void request(final Player player, final String bypass, final Object... params)
    {
        if(player == null)
        {
            return;
        }
        if(bypass.contains("wear"))
        {
            final String[] wearStr = bypass.split(":");
            if(wearStr.length < 6)
            {
                player.sendPacket(new CustomMessage("bbs.service.changeFace.noCorrectWear"));
                useSaveCommand(player);
                return;
            }
            if(player.getEvent(SiegeEvent.class) != null)
            {
                player.sendPacket(new CustomMessage("bbs.service.changeFace.isEventSiegeWear"));
                useSaveCommand(player);
                return;
            }
            if(!player.isInPeaceZone())
            {
                player.sendPacket(new CustomMessage("bbs.service.changeFace.noIsInPeaceZoneWear"));
                useSaveCommand(player);
                return;
            }
            final Optional<ETypes> face = ETypes.value(wearStr[5]);
            final ETypes[] faces = EFace.getFace(player.getPlayerTemplateComponent().getPlayerRace(), player.getPlayerTemplateComponent().getPlayerSex());
            if(!face.isPresent() || !ArrayUtils.contains(faces, face.get()))
            {
                player.sendPacket(new CustomMessage("bbs.service.changeFace.noCorrectWear"));
                useSaveCommand(player);
                return;
            }
            if(!getCheckAndPick(player, getService().getWearItemId(), getService().getWearItemCount(), true))
            {
                useSaveCommand(player);
                return;
            }
            wear(player, face.get());
            return;
        }
        final String[] styleStr = bypass.split(":");
        if(styleStr.length < 5)
        {
            player.sendPacket(new CustomMessage("bbs.service.changeFace.noCorrectRequest"));
            useSaveCommand(player);
            return;
        }
        if(player.getEvent(SiegeEvent.class) != null)
        {
            player.sendPacket(new CustomMessage("bbs.service.changeFace.isEventSiege"));
            useSaveCommand(player);
            return;
        }
        if(!player.isInPeaceZone())
        {
            player.sendPacket(new CustomMessage("bbs.service.changeFace.noIsInPeaceZone"));
            useSaveCommand(player);
            return;
        }
        final Optional<ETypes> face = ETypes.value(styleStr[4]);
        final ETypes[] faces = EFace.getFace(player.getPlayerTemplateComponent().getPlayerRace(), player.getPlayerTemplateComponent().getPlayerSex());
        if(!face.isPresent() || !ArrayUtils.contains(faces, face.get()))
        {
            player.sendPacket(new CustomMessage("bbs.service.changeFace.noCorrect"));
            useSaveCommand(player);
            return;
        }
        if(!getCheckAndPick(player, getService().getItemId(), getService().getItemCount(), true))
        {
            useSaveCommand(player);
            return;
        }
        reply(player, face.get());
    }

    @Override
    public void wear(final Player player, final Object... params)
    {
        final ETypes types = (ETypes) params[0];
        player.getCustomPlayerComponent().setFaceWear(types.ordinal(), true);
        player.getCustomPlayerComponent().startRemoveFaceTask();
        useCommand(player, getService().getContentBypass());
        Log.service("ChangeFaceService", player, "wear face to " + types.name() + ".");
    }

    @Override
    public void reply(final Player player, final Object... params)
    {
        final ETypes types = (ETypes) params[0];
        //Остановить подборку если она есть.
        player.getCustomPlayerComponent().stopRemoveFaceTask();
        player.getAppearanceComponent().setFace(types.ordinal());
        player.broadcastUserInfo(true);
        useSaveCommand(player);
        player.sendPacket(new CustomMessage("bbs.service.successfully"));
        Log.service("ChangeFaceService", player, "changed face to " + types.name() + ".");
    }
}
 

Вложения

  • upload_2016-4-22_11-27-37.png
    8,4 КБ · Просмотры: 136
Могу лишь пожелать терпения, не бросайте разработку если уж начали, делайте до конца.
 
честно говоря, я не люблю критиковать, ибо только творцу ясен замысел
но я скажу что я вижу, я вижу попытку посмотреть на тот же уй, только сбоку и сказать что это не уй))) или дайте мне определение вашего понимания "конфиг"
еще я заметил выпендреж в стиле понимания ООП ввиде применения абстрактов и интерфейсов, но профессионалы всегда понимают, где уместно применять методологию ООП, а где нет.
в вашем примере, я считаю, это не уместно, слишком просто, слишком не глубоко

P.S. хотя вы меня можете удивить, показав содержимое IService
 
А там секрет с валидациями, и кто будет писать все через else if, else if





И все это ведет лишь в 1 место.
 
Torch Chief написал(а):
выпендреж
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
понимания ООП
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
ввиде применения абстрактов и интерфейсов
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
абстрактов
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
абстрактов
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
абстрактов
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
выпендреж
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
слишком просто
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
слишком не глубоко
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
применять методологию ООП
Нажмите, чтобы раскрыть...

Torch Chief написал(а):
методологию
Нажмите, чтобы раскрыть...

И все сразу встало на свои места!
 

    GenCloud

    Баллов: -22
    Данный пользователь ведет себя не адекватно, и не может общаться с пользователями
velafrys написал(а):
Господи Иисусе, что это за говно ?
Нажмите, чтобы раскрыть...
Не знаю, мб человек не догоняет что они не так просто так лежат, и там скорей всего есть менеджер который всем этим управляет, дабы не дергать instanceOf
.
 
давайте оставим все как есть) я пытаюсь взывать к философии java, а вы продолжаете мыслить на прикладном уровне.

да - так можно делать,
да - так будет работать,
но интерфейсы не предназначены для решения задач в таком ключе
это не вопрос правильности, это вопрос философии java
 
Что такое трейты, и есть ли они в java?
 
а можно почистить тему? а то тут какой-то срач намечается) а я хотел просто публиковать ход разработки(
 

Ну так расскажите нам про философию, про правильный подход.
Век живи - век учись.
 
Войдите или зарегистрируйтесь для ответа.
Меню
Вход
Регистрация