Entonces quiero acceder a isInBounds
Navegación.kt:
data class Coordinate(val x:Int , val y:Int){ val isInBounds = x >= 0 && y >= 0 operator fun plus(other:Coordinate) = Coordinate(x + other.x, y + other.y) }Pero cuando intento acceder a él, aparece una referencia sin resolver.
Juego.kt
private fun move(directionInput: String) = try { val direction = Direction.valueOf(directionInput.uppercase()) val newPosition = direction.updateCoordinate(player.currentPosition) if (!newPosition.isInBounds){ } else{ } } catch (e: Exception){ } enum class Direction(private val coordinate: Coordinate){ NORTH(Coordinate(0 , -1)), EAST(Coordinate(1 , 0)), SOUTH(Coordinate(0 , 1)), WEST(Coordinate(-1 , 0)); fun updateCoordinate(playerCoordinate:Coordinate) { coordinate + playerCoordinate } }El error está en la Direction . Tenga en cuenta que no especificó un tipo de retorno para updateCoordinate y usó un cuerpo de bloque para ello. Esto significa que implícitamente devuelve Unit . Entonces updateCoordinate calcula coordinate + playerCoordinate , descarta el resultado y devuelve Unit . Esto hace que newPosition sea Unit , y eso obviamente no tendrá una propiedad isInBounds .
Está claro que eso no es lo que quieres. En su lugar, debe declarar updateCoordinate con un cuerpo de expresión:
fun updateCoordinate(playerCoordinate:Coordinate) = coordinate + playerCoordinateo si te gustan los cuerpos en bloque:
fun updateCoordinate(playerCoordinate:Coordinate): Coordinate { return coordinate + playerCoordinate } Esta es solo mi opinión, pero creo que es más legible tener updateCoordinate como un método en Coordinate , llamado movedTowards :
data class Coordinate(val x: Int, val y: Int) { operator fun plus(coord: Coordinate) = ... fun movedTowards(direction: Direction) = this + direction.coordinate // you'd have to make direction.coordinate public }Su función updateCoordinate en realidad no logra nada porque calcula algo sin hacer nada con el resultado, como devolverlo. Por lo tanto, implícitamente devuelve Unit, por lo que newPosition es solo una referencia a Unit.
Cambie la función para devolver la nueva coordenada:
fun updateCoordinate(playerCoordinate: Coordinate): Coordinate { return coordinate + playerCoordinate }Recomiendo darle a esta función un mejor nombre. La palabra "actualizar" implica que está mutando la instancia que se le pasa en lugar de calcular una nueva.
Lógicamente, dado que esta función es algo que agrega algo a una coordenada, tendría más sentido para la legibilidad y el razonamiento fácil de su código para invertirlo (cambiar el receptor y el parámetro). Así que sacaría esta función de la clase de enumeración y la convertiría en una función de extensión de coordenadas.
fun Coordinate.moved(direction: Direction): Coordinate { return this + direction.coordinate } pero tendrá que hacer que la propiedad de coordinate de la Dirección sea pública o internal .