Feature/jClient #1289

schwKatz · 2023-07-22T02:11:55Z

新役職「クライアント」追加
　ジャッカル陣営のマッドスニッチ

※ロールベース利用に修正

設定
・ベントを使用できる
・ベントクールダウン
・ベント内での最大時間
・クライアントのタスク数
・インポスターと同じ視界を持つ
・タスクが終わるとジャッカルからも視認できる
・ジャッカル死亡後のモード
　なし、後追い

yurinakira · 2023-07-22T02:30:20Z

README-EN.md

+Team : Neutral(Jackal)<br>
+Basis : Crewmate or Engineer<br>
+
+The Client belong to team Jackal, like Inposter's Madmates.<br>


採用可否に関わらず)
Inpost"o"rですね

誤字です
（説明の流用元の誤りをそのまま引き継いでしまいました）
修正します

Impostorでは......？

すみません。
翻訳全般見直しします。
（コピペしたところ色々バグってます・・・）

Hyz-sui

採用可否がまだなので大雑把な指摘にとどめておきます

README-EN.md

Hyz-sui · 2023-07-22T10:00:51Z

README-EN.md

+Team : Neutral(Jackal)<br>
+Basis : Crewmate or Engineer<br>
+
+The Client belong to team Jackal, like Inpostor's Madmates.<br>


文脈的にここのインポスターは複数だと思うのでImpostors' Madmatesが正しいと思いますが，別にインポスターがマッドメイトを所有しているわけではないのでMadmates for Impostors(インポスターにとってのマッドメイトのように，...)あたりが適切に思います

翻訳系は全般的に見直します

Hyz-sui · 2023-07-22T10:00:54Z

README-EN.md

+| Client Can Use Vent                      |
+| Client Vent Cooldown                     |
+| Client Max Time In Vents (s)             |
+| Client Tasks                             |
+| Client Have Impostor Vision              |
+| Can see task finished Client from Jackal |
+| After Jackal　Dead Mode                  |


全角スペースが混ざっているのと，翻訳とオプション名がズレているのをなんとかしてほしいです

修正します

README.md

Hyz-sui · 2023-07-22T10:01:04Z

README.md

+ジャッカル陣営に属しますが、クライアントとジャッカルはお互い誰かはわかりません。<br>
+タスクを全て完了させるとジャッカルを認識できるようになります。<br>


日英のreadmeで情報量にかなり差があるようですが，何かそうした理由はありますか？

翻訳系は全般的に見直します

Hyz-sui · 2023-07-22T10:01:44Z

Roles/Neutral/JClient.cs

+    "JClientAfterJackalDeadMode.None",
+    "JClientAfterJackalDeadMode.Following"


インデントに違和感がありますが，なにか理由はありますか？

不備です。インデント揃えます。

Hyz-sui · 2023-07-22T10:01:48Z

Roles/Neutral/JClient.cs

+        AURoleOptions.EngineerCooldown = JClient.VentCooldown;
+        AURoleOptions.EngineerInVentMaxTime = JClient.VentMaxTime;


JClient.が冗長です

修正します。

Hyz-sui · 2023-07-22T10:01:54Z

Roles/Neutral/JClient.cs

+    {
+        seen ??= seer;
+        if (!CanAlsoBeExposedToJackal ||
+            !seer.Is(CustomRoles.Jackal) || seen.GetRoleClass() is not JClient jclient ||


特別な事情がなければキャメルケースで命名してほしいです

jclient ⇒ jClient の指摘でよかったでしょうか？

そんな感じです
それかシンプルにclientでもいいと思います

clientで統一したいと思います

Hyz-sui · 2023-07-22T10:02:00Z

Roles/Neutral/JClient.cs

+        //ジャッカル死亡時のクライアント状態変化
+        if (AfterJackalDead == AfterJackalDeadMode.None) return;
+
+        var jackal = Main.AllPlayerControls.ToArray().Where(pc => pc.Is(CustomRoles.Jackal)).FirstOrDefault();


ToArrayはないほうがいいと思います

処理冗長ですね。修正します。

Hyz-sui · 2023-07-22T10:02:05Z

Roles/Neutral/JClient.cs

+        if (jackal != null && !jackal.Data.IsDead &&
+            !Main.AfterMeetingDeathPlayers.ContainsKey(jackal.PlayerId)) return;
+
+        Logger.Info($"jackal:dead, mode:{AfterJackalDead}", "JClient.AfterMeetingTasks");
+
+        if (Player.Data.IsDead || !MyTaskState.IsTaskFinished) return;


IsAliveではなくIsDeadを使っている理由はありますか？

旧処理の修正漏れです。IsAlive使用します。

（JClient）

（.ToArray()）

.IsAlive()を使用

　JClientInfoLong 　設定名の変更　一部翻訳言語の追記

　カウント、勝利条件の追記　役職説明の修正　設定名のゲーム内設定名との統合　英語翻訳の見直し

schwKatz added 9 commits July 19, 2023 22:27

クライアント基礎部分（ロールクラス）

dbd6f52

ベント関連設定の追加

d8d61f5

タスク完了によるジャッカル視認を追加

330a0b6

インポスター視界設定追加

1778dd6

ジャッカル陣営での勝利に追加

687a047

イントロでのジャッカル陣営表示

9499ee0

ジャッカルからも視認できる設定追加

674c7d5

ジャッカル死後のモード追加（後追い）

e67b929

README修正

f416f6c

yurinakira reviewed Jul 22, 2023

View reviewed changes

README修正（誤字）

c775df3

Hyz-sui reviewed Jul 22, 2023

View reviewed changes

schwKatz added 9 commits July 22, 2023 22:01

CanVent OptionをGeneralOptionへ変更

2ecb5fc

冗長表現の修正

e8d8a48

（JClient）

冗長処理の修正

709db33

（.ToArray()）

生存/死亡判定変更

d390f02

.IsAlive()を使用

インデント調整

8240f12

役職、設定の説明修正

28c6c6d

　JClientInfoLong 　設定名の変更　一部翻訳言語の追記

READMEの修正

a82be5a

　カウント、勝利条件の追記　役職説明の修正　設定名のゲーム内設定名との統合　英語翻訳の見直し

変数名をキャメルケースで記載

a67a498

説明、設定名の未翻訳部分修正

a4dff1f

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/jClient #1289

Feature/jClient #1289

schwKatz commented Jul 22, 2023

yurinakira Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

yurinakira Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui left a comment

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

Hyz-sui Jul 22, 2023

schwKatz Jul 22, 2023

		ジャッカル陣営に属しますが、クライアントとジャッカルはお互い誰かはわかりません。<br>
		タスクを全て完了させるとジャッカルを認識できるようになります。<br>

		"JClientAfterJackalDeadMode.None",
		"JClientAfterJackalDeadMode.Following"

		AURoleOptions.EngineerCooldown = JClient.VentCooldown;
		AURoleOptions.EngineerInVentMaxTime = JClient.VentMaxTime;

Feature/jClient #1289

Are you sure you want to change the base?

Feature/jClient #1289

Conversation

schwKatz commented Jul 22, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hyz-sui left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment