Elias Müller MineTec
0 Followers · 0 Following
Repositories
1
 Projects Packages Public Activity Starred Repositories
2
MineTec approved CraftAttack/spawn#5 2024-10-06 13:51:39 +00:00
develop-chatReply
MineTec approved CraftAttack/spawn#5 2024-10-06 13:28:22 +00:00
develop-chatReply
MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 13:25:52 +00:00
develop-chatReply

das kann noch über tooOldConversations und dort verwendet werden, auch bei dem removeAll

MineTec suggested changes for CraftAttack/spawn#5 2024-10-06 11:49:54 +00:00
develop-chatReply

bot

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:48:00 +00:00
develop-chatReply

wenn die Klasse nicht mehr verwendet wird kann sie raus

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:46:21 +00:00
develop-chatReply

String.format

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:45:42 +00:00
develop-chatReply

wird in der reply methode auch aufgerufen, macht ggf sinn die chatMessages referenz als objektvariable zu halten.

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:43:49 +00:00
develop-chatReply

hier kann durchaus ein else verwendet werden beim if

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:41:38 +00:00
develop-chatReply

klammer weg und != verwenden

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:40:54 +00:00
develop-chatReply

.appendNewline()

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:40:32 +00:00
develop-chatReply

wenn du einen weg findest finalComponent nicht überschreiben zu müssen brauchst du den array quark nicht.

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:35:41 +00:00
develop-chatReply

(ㆆ _ ㆆ)

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:32:14 +00:00
develop-chatReply

gibt es einen Grund warum die Variable hier außerhalb allokiert wird aber nur innerhalb des if statements genutzt wird?

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:31:05 +00:00
develop-chatReply

hier steht die gleiche logik wie in zeile 58

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:29:16 +00:00
develop-chatReply

"erhalten" statt "bekommen" klingt denke ich besser

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:25:50 +00:00
develop-chatReply

das senden einzelner messages ist problematisch...

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:24:06 +00:00
develop-chatReply

this.sendWhisper

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:21:22 +00:00
develop-chatReply

initialisierung und sofortig bedingtes überschreiben würde ich als bad practice einstufen...

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:16:27 +00:00
develop-chatReply

die null checks überall sind nicht schön, es macht vielleicht eher sinn am anfang einmal zu prüfen ob ein Eintrag vorhanden ist und wenn nicht ne leere Liste zu initialisieren..

MineTec commented on pull request CraftAttack/spawn#5 2024-10-06 11:15:08 +00:00
develop-chatReply

hier wird ganz oft this.replyMapping.get(sender) verwendet, rausziehen in eine eigene variable z.B. replyMap = this.replyMapping.get(sender)