refactor movePiece
This commit is contained in:
@@ -104,7 +104,7 @@ public class DDDView extends GameAdaptator {
|
|||||||
@Override
|
@Override
|
||||||
public void onMove(Move move) {
|
public void onMove(Move move) {
|
||||||
// update world internal positions
|
// update world internal positions
|
||||||
this.world.movePiece(this.world.getPiece(move.getStart()), move.getFinish());
|
this.world.movePiece(this.world.getPiece(move.getStart()), move);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void run() {
|
public void run() {
|
||||||
|
|||||||
@@ -4,6 +4,7 @@ import java.util.ArrayList;
|
|||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
|
||||||
import chess.model.Coordinate;
|
import chess.model.Coordinate;
|
||||||
|
import chess.model.Move;
|
||||||
import chess.model.Piece;
|
import chess.model.Piece;
|
||||||
|
|
||||||
public class World {
|
public class World {
|
||||||
@@ -30,13 +31,18 @@ public class World {
|
|||||||
|
|
||||||
public void addPiece(PieceEntity entity, Coordinate coordinate) {
|
public void addPiece(PieceEntity entity, Coordinate coordinate) {
|
||||||
addEntity(entity);
|
addEntity(entity);
|
||||||
movePiece(entity, coordinate);
|
setPieceCoords(entity, coordinate);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void movePiece(PieceEntity entity, Coordinate coordinate) {
|
private void setPieceCoords(PieceEntity entity, Coordinate coordinate) {
|
||||||
pieces[coordinate.toIndex()] = entity;
|
pieces[coordinate.toIndex()] = entity;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public void movePiece(PieceEntity entity, Move move) {
|
||||||
|
setPieceCoords(entity, move.getFinish());
|
||||||
|
setPieceCoords(null, move.getStart());
|
||||||
|
}
|
||||||
|
|
||||||
public PieceEntity getPiece(Coordinate coordinate) {
|
public PieceEntity getPiece(Coordinate coordinate) {
|
||||||
return pieces[coordinate.toIndex()];
|
return pieces[coordinate.toIndex()];
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user