Elias Müller MineTec
0 Followers · 0 Following
Repositories
1
 Projects Packages Public Activity Starred Repositories
2
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
MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 17:05:45 +00:00
master-customAdvancements

mein Vorschlag hier wäre du throwst nach dem custom fehler einfach nochmal die exception throw e und packst in die Objects.requireNonNull das angefragte rein. Dann ist alles immer ersichtlich.

MineTec suggested changes for CraftAttack/spawn#4 2024-09-29 17:03:36 +00:00
master-customAdvancements

sieht gut aus :)

MineTec commented on pull request CraftAttack/spawn#4 2024-09-29 16:57:35 +00:00
master-customAdvancements

das else if konstrukt ist nicht schön...