Code review

🎓 Student: KPI / IM-12 👷 Speciality: Software engineer
💡 Skills (JS) 💡 Skills (Kotlin) 🧻 Projects 📢 Public talks: 0 👀 Open-source contribution
🧑‍💻 Languages: JavaScript, Kotlin 🪙 Rates
💬 telegram: @Nikita_Sutulov 📫 [email protected]

	arrayOf<Piece?>(Rook(WHITE), Knight(WHITE), Bishop(WHITE), Queen(WHITE), King(WHITE), Bishop(WHITE), Knight(WHITE), Rook(WHITE)),
	arrayOf<Piece?>(Pawn(WHITE), Pawn(WHITE), Pawn(WHITE), Pawn(WHITE), Pawn(WHITE), Pawn(WHITE), Pawn(WHITE), Pawn(WHITE)),
	arrayOf<Piece?>(null, null, null, null, null, null, null, null),
	arrayOf<Piece?>(null, null, null, null, null, null, null, null),
	arrayOf<Piece?>(null, null, null, null, null, null, null, null),
	arrayOf<Piece?>(null, null, null, null, null, null, null, null),
	arrayOf<Piece?>(Pawn(BLACK), Pawn(BLACK), Pawn(BLACK), Pawn(BLACK), Pawn(BLACK), Pawn(BLACK), Pawn(BLACK), Pawn(BLACK)),
	arrayOf<Piece?>(Rook(BLACK), Knight(BLACK), Bishop(BLACK), Queen(BLACK), King(BLACK), Bishop(BLACK), Knight(BLACK), Rook(BLACK)),

	private fun doMove(cell: Cell) {
	if (!isMoveStarted && cell.piece != null && cell.piece?.color == currentTeam) {
	selectedCell = cell
	possibleMoves = selectedCell!!.getPossibleMoves()
	if (selectedCell!!.piece is King && !selectedCell!!.piece!!.getIsMoved()) {
	checkForCastling(selectedCell!!, possibleMoves)
	Log.d("Castling", "checking")
	}
	isMoveStarted = true
	moveTimer.start()
	} else if (isMoveStarted && cell.piece != null && cell.piece?.color == currentTeam) {
	selectedCell = cell
	possibleMoves = selectedCell!!.getPossibleMoves()
	if (selectedCell!!.piece is King && !selectedCell!!.piece!!.getIsMoved()) {
	checkForCastling(selectedCell!!, possibleMoves)
	}
	} else if (isMoveStarted && cell.piece?.color != currentTeam) {
	if (possibleMoves.any { pair -> pair.first == cell.getX() && pair.second == cell.getY() }) {
	activity.requireViewById<TextView>(R.id.last_move_tv).apply {
	text = "${getChessCoords(selectedCell!!.getX()!!, selectedCell!!.getY()!!)} ${getChessCoords(cell.getX()!!, cell.getY()!!)}"
	}
	movePiece(selectedCell!!, cell)
	selectedCell = null
	isMoveStarted = false
	switchCurrentTeam()
	resetMoveTimer()
	moveTimer.start()
	}
	}
	}

	private fun checkForCastling(king: Cell, possibleMoves: MutableList<Pair<Int, Int>>) {
	var teamCells: MutableList<Cell>
	var enemyCells: MutableList<Cell>
	if (currentTeam == WHITE) {
	teamCells = whiteCells
	enemyCells = blackCells
	} else {
	teamCells = blackCells
	enemyCells = whiteCells
	}
	val rooks = teamCells.filter { cell -> cell.piece is Rook && !cell.piece!!.getIsMoved() }
	if (rooks.isEmpty()) {
	return
	}
	var startY: Int
	var endY: Int
	var yToMoveKingTo: Int
	val kingX = king.getX()!!
	for (rook: Cell in rooks) {
	var isRookAbleForCastling = true
	if (rook.getY()!! < king.getY()!!) {
	startY = rook.getY()!!
	endY = king.getY()!!
	yToMoveKingTo = king.getY()!! - 2
	} else {
	startY = king.getY()!!
	endY = rook.getY()!!
	yToMoveKingTo = king.getY()!! + 2
	}
	for (y in startY..endY) {
	if (y == rook.getY()!! \|\| y == king.getY()!!) {
	continue
	}
	Log.d("Castling", "checking cell ${getChessCoords(kingX, y)}")
	if (
	cells[kingX][y]!!.piece == null && cells[kingX][y]!!.isUnderAttack(enemyCells) \|\|
	cells[kingX][y]!!.piece != null
	) {
	isRookAbleForCastling = false
	break
	}
	}
	if (!isRookAbleForCastling) {
	continue
	}
	Log.d("Castling", "adding a move")
	possibleMoves.add(Pair(kingX, yToMoveKingTo))
	}
	}

nikitasutulov / chessforandroid Goto Github PK

	for (i in 0..7) {
	for (j in 0..7) {