replace #66

Yaruma3341 · 2019-12-08T19:45:57+01:00

Yaruma3341 hat

2019-12-08 19:45:57 +01:00

kommentiert

Mitwirkender

Keine Beschreibung angegeben.

Lixfel wurde von Yaruma3341 2019-12-08 19:45:57 +01:00 zugewiesen

Yaruma3341 hat sich das Issue 2019-12-08 19:45:57 +01:00 selbst zugewiesen

Lixfel hat 2019-12-09 17:58:02 +01:00 Änderungen angefragt

Lixfel hat einen Kommentar hinterlassen

Besitzer

Bitte mach die Änderungen, die ich unter Geänderte Dateien im Code angemerkt habe.

FightSystem_Main/src/de/steamwar/fightsystem/FightSystem.java Veraltet

						
				@ -108,2 +108,4 @@

				        new PreRunningCountdown();

				        Fight.getRedTeam().replaceSync();

				        Fight.getBlueTeam().replaceSync();

Lixfel hat

2019-12-09 17:54:59 +01:00

kommentiert

Besitzer

Bitte verschiebe das Aufsplitten in Red und Blue Team in die Klasse Fight

FightSystem_Main/src/de/steamwar/fightsystem/fight/FightTeam.java

						
				@ -302,0 +308,4 @@

				        for(int x = minPoint.getBlockX(); x <= maxPoint.getBlockX(); x++) {

				            for(int z = minPoint.getBlockZ(); z <= maxPoint.getBlockZ(); z++) {

				                for(int y = minPoint.getBlockY(); y <= maxPoint.getBlockY(); y++) {

Lixfel hat

2019-12-09 17:57:23 +01:00

kommentiert

Besitzer

Du machst SYNC!!! ein paar 1000 Schleifendurchläufe, obwohl das Feature nur gelegentlich aktiv ist?

Bitte überprüfe am Beginn der Methode, ob überhaupt replaced werden soll. (Ja mach daraus 2 Funktionsaufrufe, füge halt 2/3 Argumente hinzu wie replaceFrom, replaceTo, shouldReplace.

Du machst SYNC!!! ein paar 1000 Schleifendurchläufe, obwohl das Feature nur gelegentlich aktiv ist? Bitte überprüfe am Beginn der Methode, ob überhaupt replaced werden soll. (Ja mach daraus 2 Funktionsaufrufe, füge halt 2/3 Argumente hinzu wie replaceFrom, replaceTo, shouldReplace.