giganticminecraft / seichiassist Goto Github PK

View Code? Open in Web Editor NEW

96.0 10.0 36.0 87.76 MB

ギガンティック☆整地鯖の独自要素を司るプラグイン / A Minecraft(Spigot) Plugin used for Gigantic Seichi Server. Address -> play.seichi.click

Home Page: https://discord.gg/gigantic-minecraft

License: GNU General Public License v3.0

Java 4.24% Scala 94.41% Dockerfile 0.07% Batchfile 0.03% Shell 1.26%

minecraft-plugin spigot-plugin minecraft-server

seichiassist's Introduction

SeichiAssist

前提プラグイン

前提プラグイン(整地鯖内製)

RegenWorld リポジトリ | jar

コントリビューション

どのように貢献するかはCONTRIBUTING.mdへ移動しました。

慣れている開発者向け：

自分のGitHubアカウントへfork
git clone --recursive https://github.com/${YOUR_GITHUB_ID}/SeichiAssist
cd SeichiAssist
git checkout -b ${YOUR_BRANCH_NAME} develop
git commit -am "feat: 〇〇を実装"
- コミットメッセージはコンベンショナルコミットを推奨
- 言語は英語または日本語を推奨
git push
GiganticMinecraftのレポジトリに向かってPull Requestを作成
CIとコードレビューを待つ
マージされる
CDが実行される
(本番サーバーへのデプロイはGiganticMinecraftのメンバーがPRによって行う)

利用条件

GPLv3ライセンスでの公開です。ソースコードの使用規約等はGPLv3ライセンスに従います。
当リポジトリのコードの著作権はunchamaが所有しています。
独自機能の追加やバグの修正等、ギガンティック☆整地鯖(以下、当サーバー)の発展への寄与を目的としたコードの修正・改変を歓迎しています。その場合、当サーバーのDiscordコミュニティに参加して、当コードに関する詳細なサポートを受けることが出来ます。

ライセンス

このプラグインは、Apache 2.0ライセンスで配布されている以下の製作物が含まれています。

AJD4JP 日本用カレンダー処理 Javaクラスライブラリ

seichiassist's People

Contributors

Stargazers

Watchers

seichiassist's Issues

プレイヤーデータが取得完了する前にブロックを掘った場合、所持マナの量が0.0になる不具合

タイトルの通り。

プレミアムエフェクトを解除してもメニューのブロックが岩盤から変化しない

出典: https://discordapp.com/channels/237758724121427969/573131925158494220/573176038759071793

sub_home日本語名称(sub_home.name)がDB上で文字化け

どこかの文字コードの問題と思われる。
ちなみに英語名称は問題ない。

整地鯖専用の進捗を作成する

1.12から進捗を自由に造ることが出来るようになったので、整地鯖特有の進捗を作成する。

アイディアプロジェクトからだが、具体的な進捗の内容に関しては何も決まっていないため、そこから決定することになりそう。

MineStackに紫色の彩釉テラコッタがない

https://discordapp.com/channels/237758724121427969/573131925158494220/573134075024375828 による。

[SQL]statementやresultsetの閉じ忘れ

statementやresultsetの閉じ忘れを所々忘れているので閉じるようにする。
閉じ忘れててもGC時に閉じるらしいけど、その分GCが重くなるらしい……とgoogle先生が仰っておりました。

木の棒メニュー1ページ目

木の棒メニュー2ページ目

土地保護関連メニュー

サーバ間移動メニュー

サーバ間移動メニュー
- アルカディアサーバ移動ボタン(0)
  - 押下でアルカディアサーバへ
- エデンサーバ移動ボタン(1)
  - 押下でエデンサーバへ
- ヴァルハラサーバ移動ボタン(2)
  - 押下でヴァルハラサーバへ
- 公共施設サーバ移動ボタン(8)
  - 押下で公共施設サーバへ

実績・二つ名システムメニュー

パッシブスキルブックメニュー

パッシブスキルメニュー
- 複数種類同時破壊スキルボタン(0)
  - 押下で複数種類同時破壊スキル(ON/OFF)
- チェスト破壊切り替えボタン(1)
  - 押下でチェスト破壊切り替え(ON/OFF)
- GiganticBerserkボタン(9)
  - 押下で進化(?)

プレミアムエフェクト引き換え時にプレミアムポイントが減算されない

取り急ぎ

Mavenリポジトリにあるjarはローカル依存ではなくリポジトリからダウンロードするようにする

そうした方がプラグインバージョンの管理が楽

リファクタリングでデータベース初期化のコードが壊れてしまっている

#44 による変更で、playerdataの初期化が正しく行われないようになっています。

元の動作は、playerdata以外のテーブルをまず初期化し、その後プラグインクラスにstaticでくっついたフィールドを変更し、そのデータに基づいてplayerdataを初期化する、というものでした。

playerdataテーブルのカラムが読み込まれたデータに依存しているため、初期化を逐次的に行うか、テーブルスキーマを全部変更してしまうかの修正が必要です。

Maven -> Gradleに乗り換え

#64 によりFlywayを入れたいが、fat jarをmavenで生成したいとなると設定が煩雑になりがちなのでいっそgradleにしたい

PlayerHeadの初回読み込み時にサーバーがプチフリーズする

1.10.2 -> 1.12.2にした際にPlayerHead読み込みの挙動が変わり、読み込み開始〜完了までの間サーバーがフリーズしている模様。特に大量のPlayerHeadを読み込む各種ランキング表示時の影響が大きい。

PlayerHeadの読み込みを一律非同期化させる必要がある。

新たなステータスSeichiEXPの追加

MCLvがExperienceManagerでオーバーフローしているため、MCLv1200程度で経験値が増えなくなるというバグが発生するという報告があるので別な独自ステータスを用意したい。
そこでMCEXPと相互変換可能なステータス、SeichiEXPを提案する

例えば、MCEXPを消費する独自機能(fly,自分の頭取り出すやつ)をSeichiEXPを消費するように変更する。

参照：https://red.minecraftserver.jp/issues/2528

現状

現状では、MenuInventoryDataクラスでInventoryを管理し、PlayerInventoryListenerにおいてInventoryの操作を感知し動作を行わせている。

問題点

現状で述べたような方式を採用すると、

Inventoryのデータと処理が分離し分かりずらい
Inventoryのデータを作成する際にPlayerDataを利用したい時に困ってしまう(今のようにPlayerを引数に取り、Inventoryを返すような関数を大量に作成する必要がある)
Inventoryのデータとして保持しているItemStackの作成のコード量が多く、重複が多い(メニュー移動の矢印等)

以上のような問題が発生すると考える。

改善策・設計

そこで以上の問題点を解決すべくこのissueを作成した。まずは検討している設計について述べる。
まずはInventoryを構成するSlotの概念を導入する。

Slot

SlotとはBukkitのInventoryの1つのマスのことである。
SlotはItemStackとBukkitのInventoryにおける配置番号を持つ。
interfaceにより設計され、以下の継承クラスを持つ。

BaseSlot:何の機能も有しないSlotであり、Inventoryに設置できるItemStackとして利用できる。
Button:動作を付与できるSlotであり、Inventoryに設置し、MenuのButtonとして利用できる。動作の付与方法は後述する。

上記二つのクラスは共にBuilderを持ち、容易に生成することが可能である。

AbstractSlotBuilder: Slotのベースとなる抽象Builderクラス。再帰的ジェネリクスを利用し具象型クラスのインスタンスを返すことができるようにしている。
- SlotBuilder:BaseSlotのBuilder。
- ButtonBuilder:ButtonのBuilder。

ItemStackBuilder

ItemStackを容易に生成することができるBuilder。SlotBuilderとともに利用することを想定している。

BaseIconComponent: ItemStackの各情報を集約し、各Builderクラスに必要なインスタンスを付与する。
AbstractItemStackBuilder: 各Builderのベースとなる抽象Builderクラス。
- ItemStackBuilder : ItemStackのBuilderクラス。
- SkullItemStackBuilder : Skullを作成するためのBuilderクラス。

具体例

Buttonの生成

public static Button playerInfo = ButtonBuilder
        .from(
            SkullItemStackBuilder
                .of()
                .playerSkull()
                .title(data -> Text.of(data.name + "の統計データ"))
                .lore(PlayerData::getPlayerInfoLore)
        )
        .at(0)
        .handler(new SlotActionHandler(
            Triggers.RIGHT_CLICK,
            //TODO: WIP
            event -> {}
        ))
        .build();

Menuの生成

public static Menu stickMenu = ChestMenuBuilder.of()
                .size(4 * 9)
                .slots(
                    PlayerDataButtons.playerInfo
                )
                .build();

Gachaコマンドの表示・入力に関する不具合

/gacha helpで表示されるコマンド使用方法一覧における/gacha addmsは以下のようになっている

SeichiAssist/src/com/github/unchama/seichiassist/commands/gachaCommand.java

Lines 57 to 58 in 8c5b9b4

    
           sender.sendMessage(ChatColor.RED + "/gacha addms <名前> <レベル> <ID>"); 
        
           sender.sendMessage("指定したガチャリストのIDを指定した名前とレベル(実際のレベルではないことに注意)でMineStack用ガチャリストに追加");

一方でコードを見てみると、違った説明が記載されており、引数チェックも誤っている。

SeichiAssist/src/com/github/unchama/seichiassist/commands/gachaCommand.java

Lines 327 to 337 in 8c5b9b4

    
           }else if(args[0].equalsIgnoreCase("addms")){ 
        
           	if(args.length != 3){ 
        
           		sender.sendMessage("/gacha addms <名前> <ID>  のように、追加したいアイテムの名前とIDを入力してください"); 
        
           		return true; 
        
           	} 
        
           	int level = Util.toInt(args[2]); 
        
           	int num = Util.toInt(args[3]); 
        
           	Gachaaddms(sender, args[1],level,num); 
        
           	return true;

SQLリファクタリング後、ランキングテーブルの更新に失敗する

SeichiAssist/src/main/java/com/github/unchama/seichiassist/task/PlayerDataBackupTaskRunnable.java

Line 50 in b5bcb51

//ランキングデータをセット

連続投票カウントの更新ロジックが誤っている

LoadPlayerDataTaskRunnable.java

Date TodayDate = sdf.parse(sdf.format(cal.getTime()));
Date LastDate = sdf.parse(lastvote);
long TodayLong = TodayDate.getTime();
long LastLong = LastDate.getTime();

long datediff = (TodayLong - LastLong)/(1000 * 60 * 60 * 24 );
if(datediff <= 1 || datediff >= 0){
	playerdata.ChainVote = rs.getInt("chainvote");
}else{
	playerdata.ChainVote = 0;
}

PlayerDataManipulator.java

Date TodayDate = sdf.parse(sdf.format(cal.getTime()));
Date LastDate = sdf.parse(lastvote);
long TodayLong = TodayDate.getTime();
long LastLong = LastDate.getTime();

long datediff = (TodayLong - LastLong)/(1000 * 60 * 60 * 24 );
if(datediff <= 1 || datediff >= 0){
    count ++ ;
}else {
    count = 1;
}

これらのif文はともに恒真式です。結果として、chainvoteが必ずインクリメントされるようになっています。
また、ロジックが全く同じなのでどこかに切り出せます。

ループの中で文字列を連結しているところをStringBuilderに置き換える

TL:DR; 中間生成物の文字列が発生しない→GCも起きない→多少早くなるかもしれない。
(+=や+で一時的に生成された文字列は結果として使われないのでGCされて無駄になるだけ)

/src/main/java/Sql.javaのリファクタリング

分割、ロジックの分離など

整地レベルアップでガチャ券を報酬として配布する機能の追加

Lv50までの初心者を対象として、レベルアップ時に段階的に報酬(ガチャ券)を用意するというもの。
既にそのレベルを超えているプレイヤーにも受け取れるようにする。

二つ名選択メニューの戻るボタンの不具合

二つ名システムのメニューにおいて、２ページ目から１ページ目へ戻るボタンが表示されない。

マナ妖精からの呼び名の変更機能追加

メビウスからの呼び名を変更できるように、マナ妖精からの呼び名を変更できるようにする。

凝固スキルの説明のミス

凝固スキルの必要ポイントを変えようとしてメニューに表示される必要ポイントを変更したが、結局変更しなかったので、現状食い違っている。
私のミスなので早急に修正したい。

[SQL]接続復旧処理がダブる

プレイヤーが2人以上且つDB接続が切れてしまった状態でPlayerDataBackupTaskRunnableが走ると、ensureConnection()が非同期でプレイヤー人数分実行されてしまっている。
試しにDBをモニタリングしたところ、コネクション数がプレイヤー数分張られてしまっている模様。気持ち悪いのでそのうち直す。

synchronized拡張子とやらを使えばいけるらしいが、同一インスタンス内からの呼び出しのみ正常に機能するとか。なんもわからん
https://www.google.co.jp/search?q=synchronized+java+%E6%84%8F%E5%91%B3&ie=UTF-8&oe=UTF-8&hl=ja-jp

CircleCIビルドで、scpの出力を全て隠す

公開されるべきではないURLなどが出力に含まれる可能性があるので、stdoutとstderrを全部/dev/nullにパイプするべき。

MineStackのアイテム登録の簡略化

現時点ではソースコードに全アイテムを記載してMineStackのアイテムを管理している。
この方法では、ソースコード編集作業を行わなければならず、開発者が必要となる。

そのため、現在のMineStackのガチャアイテム登録のシステムのような、手に持ったアイテムをコマンドを打つことによってMineStackに登録できるような機能を追加したい。

メビウス図鑑機能作成

メビウスをただ集めるだけでは面白さに欠けるため、メビウスの図鑑が欲しい。
追加する情報の例は以下の通り。

今までに発見したメビウスの数
（出来るのなら発見した鯖やワールドも）
最大レベルまで育てたメビウスの数
崩壊してしまったメビウスの数
付与したエンチャント一覧
発見したセリフ一覧
（メビウスが発言した回数なども）
（テクスチャ案が通れば）テクスチャのバリエーションの一覧

参照：https://red.minecraftserver.jp/issues/4059

プレイヤー間における、アイテムのMinestacks内での取引・受け渡しを楽にできるような機能の追加

必要な根拠としては、「大量なアイテムの取引・受け渡し」をするとき、既存のトレードシステムでは最大で64×16このアイテムしか1度に受け渡しできません。
さらに、地面にアイテムを撒いて受け取るにも、サーバに負荷がかかってしまうことが容易に想像できます。
しかし、Minestacks内での取引・受け渡しが可能になれば、大量のアイテムを1度に移動でき、サーバに負荷も与えにくくなると思われます。
また、手間が減るため、プレイヤー間の取引もより活発になることが想像できます。

From RedMine#2505

各種メニューのページ送り・戻りの不具合

時々、各種メニューのページの送り・戻りが出来なくなる事がある。大変利便性が悪い。
これはAPIの制限によりページの送り・戻りのボタンとして使用している頭が読み込めなくなってしまうためである。

以下が現在使用しているコード。getOwner()で正しいOwnerが取得できなくなる事が原因である。

itemstackcurrent.getType().equals(Material.SKULL_ITEM) && ((SkullMeta)itemstackcurrent.getItemMeta()).getOwner().equals("MHF_ArrowRight")

これを読み込みの成否にかかわらず、使用できるようにしたい。

今、ガチャの方の「手に持ったアイテムがマインスタックに追加される」機能、バグってるのかなんなのか不可能なんですけど…

ガチャはもともと手に持って追加するコマンドがあるんですよ！（コードどこに書いてあるか忘れた:innocent: ）　でも正常に動いてないんです:joy:
レベルは収納レベルのことだと思いますが私もよくわからないです…でも無視してちゃんと登録されてるなら良いのでは:thinking:

交換券を無理やり突っ込もうとしたんですけどうまく格納されなくて…

(#plugin開発で kawakuroさんによる)

サーバ毎に表示非表示を切り替えることができる。

指定方法はConfigにて行う。Configにはサーバを識別するservernumが存在するため、それを使用すると良い

SeichiAssist/config.yml

Lines 12 to 14 in 8f602e4

    
           # サーバ識別設定 
        
           # 複数サーバ構成にする場合はこの値をサーバごとに変更します 
        
           servernum: '1'

「スキルでのチェスト破壊は整地ワールドでのみ有効です」をアクションバーに表示する

WorldGuardのメッセージみたいな感じに

スキルで破壊したときに出るエフェクトが違う

例えば、一括破壊で白樺の原木を壊したときに、オークの原木の破壊エフェクトが出る。

	sender.sendMessage(ChatColor.RED + "/gacha addms <名前> <レベル> <ID>");
	sender.sendMessage("指定したガチャリストのIDを指定した名前とレベル(実際のレベルではないことに注意)でMineStack用ガチャリストに追加");

	}else if(args[0].equalsIgnoreCase("addms")){
	if(args.length != 3){
	sender.sendMessage("/gacha addms <名前> <ID> のように、追加したいアイテムの名前とIDを入力してください");
	return true;
	}

	int level = Util.toInt(args[2]);
	int num = Util.toInt(args[3]);
	Gachaaddms(sender, args[1],level,num);

	return true;

	# サーバ識別設定
	# 複数サーバ構成にする場合はこの値をサーバごとに変更します
	servernum: '1'