Elias Müller MineTec

0 Followers · 0 Following

• Germany
• https://mhsl.eu
• Joined on 2021-11-02

Repositories

1

Projects Packages Public Activity Starred Repositories

2

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)

MineTec pushed to master at CraftAttack/spawn 2024-10-05 16:40:54 +00:00

346847d2b2 Merge pull request 'master-customAdvancements' (#4) from master-customAdvancements into master

e3b07aa62f solved pr comments

da33e6e592 added more custom advancements, added pending advancements

1aebae6cd5 added fleischerchest and pixelblock advancements

Compare 4 commits »

MineTec deleted branch master-customAdvancements from CraftAttack/spawn 2024-10-05 16:40:54 +00:00

MineTec merged pull request CraftAttack/spawn#4 2024-10-05 16:40:52 +00:00

master-customAdvancements

MineTec approved CraftAttack/spawn#4 2024-10-05 16:40:39 +00:00

master-customAdvancements

MineTec opened issue CraftAttack/website#31 2024-10-03 19:56:28 +00:00

Anmeldebutton auf startseite bei geschlossener Anmeldung austauschen