yuina1056 / clanbattlebot Goto Github PK

View Code? Open in Web Editor NEW

3.0 2.0 3.0 356 KB

クランバトル管理Bot

License: MIT License

TypeScript 99.94% Dockerfile 0.06%

clanbattlebot's Introduction

ClanBattleBot

プリンセスコネクト！Re:Diveのクランバトルを管理するためのDiscordBot

機能

メイン機能

凸宣言
凸報告
クラン内凸一覧の確認

順次機能は追加予定。

おまけ

運命のダイスロール

開発環境

node.js v20.2.0
MySQL v8.0.33

その他はpackage.jsonを参照。

使い方

discordBotのユーザー作成は各自行ってください。 (動作サンプルのサーバーは公開していません)

インストール

$ git clone https://github.com/yuina1056/ClanBattleBot.git

作者

@yuina1056

ライセンス

MIT

clanbattlebot's People

Contributors

Stargazers

Watchers

Forkers

drakontia iteduki yonta

clanbattlebot's Issues

クランバトル開催情報を画面から登録できるようにする

凸状況に応じて表示される凸宣言ボタンを変更する

凸宣言ボタンを押した後に表示される凸宣言ボタンを、凸状況に応じて変更されるようにする。

見本

物理と魔法の区別は別にいらないかも。
持ち越しがある場合は、持ち越しと表記される。

prettierからdprintに移行するかどうか検討する

#38 から派生。

https://kakehashi-dev.hatenablog.com/entry/2023/11/27/103000

prettierよりも早いし、VSCode拡張もあるっぽいので移行視野に入れつつ検討

救援ボタンを作る

ロールメンションで救援呼べるが最初。
凸残ってる人を個別メンションできるとベスト

githubActionで色々チェックしたい

トランスコンパイル通るかチェック
pritterとかeslintチェックしたい
- PRで解決済み！感謝ァ！

DB呼び出し周りのリファクタ

凸状況がきちんと反映されない時がある

3凸しても1凸目が凸状況に反映されないことがある。他にも同様のケースが散見される。

コルネ — 今日 00:33
やり方が間違ってたらごめんなさい。1凸の持ち越しが違うのと、２、３凸が入力されてないっぽい？
凸宣言して何凸目か選択して、削りが撃破おしてます
端末はアイポン７（古）

持ち越しを含んだ時の処理が上手くいっていないのか？

優先度高く対応をお願いします。

周回数の管理はクランと分離する

今DBにおいてクランの中に周回数が埋め込まれていますが、毎度リセットする手間が発生するので、イベントごとにした方がいいと思います。

凸状況に残凸メンバーが表示されるようにする

凸状況がきちんと反映されればいいのですが上手くいっていないので、凸状況が分からないと進行ができません。

どちらかの対応をいち早くお願いします。

凸状況と各ボスのところでボス名を表示する

ボスが序数で表示されているので、ボスの名前で表示して誤操作を防ぐようにする

各ボスのHPの管理をDB上でマスタ化する

ついでにボス名とかもマスタにおいて、discordから更新できるように

同凸等で凸をしなかったユーザーの凸宣言をなかったことにしたい

救援等で同凸をしたが、殴らずに終了するパターンがある。
撃破時に凸宣言が残っているユーザーの凸宣言は削除するようにする。

ボスの残HPを登録できるようにする

宣言メッセージとボス状況を分離する

宣言とボス状況とボタンが一緒になっているが、分離した方がいい。

宣言はchannel.sendで送る。

ボス状況はできれば、編集にしたいが無理なら新しく送る

持ち越しが発生しているかの判定が機能していない

以下のところで判定していますが、機能していません。

ClanBattleBot/src/commands/button/report_defeat.ts

Lines 160 to 171 in 622ebab

    
           let isCarryOver = false; 
        
           const reports = await DataSource.getRepository(Report).find({ 
        
             where: { 
        
               userId: user.id, 
        
               eventId: event.id, 
        
               day: declaration.day, 
        
               attackCount: declaration.attackCount, 
        
             }, 
        
           }); 
        
           if (reports.length === 0) { 
        
             isCarryOver = true; 
        
           }

おそらく、取得したreportsのlengthを厳密に0か判定していますが、nullかなんかが返ってきているのを想定できていません。
なので、毎度判定がすり抜けます。
findの返り値を確認して、的確に判定する必要があります。

代理入力が出来る様にする

メンバーの凸宣言および削り/撃破入力を進行メンバーが代理で入力できるようにする。
（ひとまずは進行メンバーでなくても、誰でもできてもいいかもしれない）

解凍処理を実装する

解凍とは、凸宣言をしている状態で救援などで凸宣言の対象であるボスが撃破された際に凸宣言をキャンセルさせるものである。
解凍操作自体は、凸宣言のキャンセルで実施出来るが、~~よりわかりやすくするために別途解凍ボタンを設置した方がいいと思われる~~操作のし忘れなどの防止に実装しておいた方がいい。
処理としては、解凍対象のボスに凸宣言されているものを一律で消去すれば問題ないと考えられる。

宣言する際にクランが違っていても宣言できてしまう

別のクランので凸宣言しようとした場合、失敗になるのが望ましいがそうならない。
クランなどの情報は引っ張ってきているがバリデーションされていないので、バリデーションするようにする。

命名規則を揃えたい

ぶれてるので直したい。import周り直すときに一緒にやる。

transactionをほぼほぼ張ってないので入れる

タイトル通り

周数判定を段階に合わせて判定できるように修正する

現状は２周離れてしまった場合に宝箱のみ凸できないような実装となっている。
段階が上がるタイミングでは全ての周回数が揃う形になるため、段階ごとに考慮する処理を追加する。

Node.jsのバージョンを固定する

node-versionファイルを置くと使用するnode.jsのバージョンを固定できます
https://zenn.dev/yassh/articles/5a9c761c10aaed

Reportに持ち越しかの判定だけでなく、持ち越しが発生したかの判定が必要

今、持ち越しかの判定でisCarryOverがあると思いますが、その凸で持ち越しが発生したかの判定も必要だと思われます。
あとで集計する際に削りか撃破だけだと持ち越しが発生しているかの区別をするのが困難です。
なので、hasCarryOverという発生判定を付ける必要があると思います。

discord.jsのバージョンアップ対応

各種dependabotが更新できない状態になっている理由がdiscord.jsのバージョンアップできてないからなので対応する。

凸状況で持越があることを表示する

[2撃破/5削り/3削り]

上のような場合において、持越凸が残っている場合は下のように持越が残っていることを表記するようにする。

[2撃破-/5削り/3削り]

prisma
typeORM

mysql> select * from declaration where userId=44;
+-----+--------+--------+---------+--------+-----+-----+-------------+------------+----------------------------+----------------------------+
| id  | clanId | userId | eventId | bossId | lap | day | attackCount | isFinished | CreatedAt                  | UpdatedAt                  |
+-----+--------+--------+---------+--------+-----+-----+-------------+------------+----------------------------+----------------------------+
| 225 |      2 |     44 |       4 |     10 |  23 |   1 |           1 |          1 | 2023-11-26 02:27:01.761509 | 2023-11-26 02:32:58.000000 |
| 233 |      2 |     44 |       4 |     10 |  24 |   1 |           1 |          1 | 2023-11-26 02:49:08.021522 | 2023-11-26 02:53:53.000000 |
| 236 |      2 |     44 |       4 |      7 |  24 |   1 |           2 |          1 | 2023-11-26 03:01:41.279601 | 2023-11-26 03:02:22.000000 |
| 238 |      2 |     44 |       4 |      8 |  23 |   1 |           3 |          1 | 2023-11-26 03:34:13.863365 | 2023-11-26 03:35:54.000000 |
| 442 |      2 |     44 |       4 |      8 |  25 |   2 |           1 |          1 | 2023-11-27 00:16:02.421635 | 2023-11-27 00:16:09.000000 |
| 443 |      2 |     44 |       4 |     10 |  26 |   2 |           2 |          1 | 2023-11-27 00:17:12.895175 | 2023-11-27 00:21:20.000000 |
| 450 |      2 |     44 |       4 |      7 |  26 |   2 |           3 |          1 | 2023-11-27 00:36:31.669690 | 2023-11-27 00:36:37.000000 |
| 453 |      2 |     44 |       4 |      7 |  27 |   2 |           3 |          1 | 2023-11-27 00:38:16.744103 | 2023-11-27 00:47:49.000000 |
| 458 |      2 |     44 |       4 |      8 |  26 |   2 |           1 |          1 | 2023-11-27 00:52:04.103379 | 2023-11-27 00:59:52.000000 |
| 579 |      2 |     44 |       4 |      7 |  27 |   3 |           1 |          1 | 2023-11-27 20:57:14.222633 | 2023-11-27 20:59:49.000000 |
| 583 |      2 |     44 |       4 |      8 |  28 |   3 |           2 |          1 | 2023-11-27 21:01:00.217214 | 2023-11-27 21:05:55.000000 |
| 592 |      2 |     44 |       4 |     10 |  29 |   3 |           3 |          1 | 2023-11-27 21:31:49.933460 | 2023-11-28 22:43:21.000000 |
| 829 |      2 |     44 |       4 |      7 |  31 |   4 |           1 |          0 | 2023-11-28 22:35:41.776629 | 2023-11-28 22:35:41.776629 |
+-----+--------+--------+---------+--------+-----+-----+-------------+------------+----------------------------+----------------------------+
13 rows in set (0.00 sec)

mysql> select * from report where userId=44;
+-----+--------+--------+---------+--------+-----+-----+-------------+--------+-------------+----------------------------+----------------------------+----------+
| id  | clanId | userId | eventId | bossId | lap | day | attackCount | damage | isCarryOver | CreatedAt                  | UpdatedAt                  | isDefeat |
+-----+--------+--------+---------+--------+-----+-----+-------------+--------+-------------+----------------------------+----------------------------+----------+
| 225 |      2 |     44 |       4 |      5 |  24 |   1 |           1 |      0 |           0 | 2023-11-26 02:53:53.628717 | 2023-11-26 02:53:53.628717 |        0 |
| 227 |      2 |     44 |       4 |      2 |  24 |   1 |           2 |      0 |           0 | 2023-11-26 03:02:22.214453 | 2023-11-26 03:02:22.214453 |        0 |
| 229 |      2 |     44 |       4 |      3 |  23 |   1 |           3 |      0 |           0 | 2023-11-26 03:35:54.421798 | 2023-11-26 03:35:54.421798 |        0 |
| 415 |      2 |     44 |       4 |      3 |  26 |   2 |           1 |      0 |           0 | 2023-11-27 00:16:09.701205 | 2023-11-27 00:16:09.701205 |        1 |
| 416 |      2 |     44 |       4 |      5 |  26 |   2 |           2 |      0 |           0 | 2023-11-27 00:21:20.600527 | 2023-11-27 00:21:20.600527 |        0 |
| 421 |      2 |     44 |       4 |      2 |  27 |   2 |           3 |      0 |           0 | 2023-11-27 00:36:37.708355 | 2023-11-27 00:36:37.708355 |        1 |
| 423 |      2 |     44 |       4 |      2 |  27 |   2 |           3 |      0 |           0 | 2023-11-27 00:41:09.925472 | 2023-11-27 00:55:50.985687 |        0 |
| 428 |      2 |     44 |       4 |      3 |  26 |   2 |           1 |      0 |           0 | 2023-11-27 00:52:17.660724 | 2023-11-27 00:52:17.660724 |        0 |
| 545 |      2 |     44 |       4 |      2 |  27 |   3 |           1 |      0 |           0 | 2023-11-27 20:57:27.418599 | 2023-11-27 21:00:42.967177 |        0 |
| 550 |      2 |     44 |       4 |      3 |  28 |   3 |           2 |      0 |           0 | 2023-11-27 21:05:24.190807 | 2023-11-27 21:05:49.896531 |        0 |
| 558 |      2 |     44 |       4 |      5 |  29 |   3 |           1 |      0 |           0 | 2023-11-27 21:36:13.519448 | 2023-11-27 21:36:13.519448 |        0 |
| 793 |      2 |     44 |       4 |      2 |  32 |   4 |           3 |      0 |           0 | 2023-11-28 22:43:21.778156 | 2023-11-28 22:43:21.778156 |        1 |
+-----+--------+--------+---------+--------+-----+-----+-------------+--------+-------------+----------------------------+----------------------------+----------+
12 rows in set (0.00 sec)

それぞれの一番下の行。
isFinishedがTrueになっていない。
1凸目なのに3凸目になっている。
宣言は拾えているのに、途中ですり替わった？

周回数に依存せず凸内容を修正できるようにする

現状の凸リセットだと、凸の修正で再度宣言報告をすると周回数がずれてしまうことがある。
周回数に影響させずに個人の凸報告を修正できるようにしたい。

	let isCarryOver = false;
	const reports = await DataSource.getRepository(Report).find({
	where: {
	userId: user.id,
	eventId: event.id,
	day: declaration.day,
	attackCount: declaration.attackCount,
	},
	});
	if (reports.length === 0) {
	isCarryOver = true;
	}