我有一个包含MutableList的类,该类对该列表进行了更改。我定义了一个吸气剂,以允许外部模块获取列表的副本。问题是,每当我访问该类中的字段时,最终都会更改列表副本而不是实际列表。如何在不通过定义的类中的getter的情况下访问列表?这是否被认为是不好的做法?
class Server(private val port: Int): Runnable {
val clients = mutableListOf<SocketChannel>()
get() = field.toMutableList()
private val sel = Selector.open()
private val serverChannel = NIOServerChannel(port, sel).get()
private val engine = ServerEngine(sel, clients)
private val acceptor = ChannelAcceptor(serverChannel, clients)
private val reader = ChannelReader()
private val writer = ChannelWriter()
private val middleware = Middleware()
override fun run() {
buildEngine()
println("Starting server on port: $port")
engine.run()
}
fun use(func: (String, Response) -> Unit) = middleware.add(BiConsumer(func))
private fun buildEngine() {
reader.process = middleware.get()
engine.add(acceptor)
engine.add(reader)
engine.add(writer)
}
}
答案 0 :(得分:1)
问题是这一行:get() = field.toMutableList()
查看definition of toMutableList
:它将创建一个新集合。只需返回field
或完全省略吸气剂即可。
答案 1 :(得分:1)
访问自动创建的后备字段似乎没有任何约定。 Kotlin docs on backing fields声明一个明确的后备属性:
private val _clients = mutableListOf<SocketChannel>()
val clients: MutableList<SocketChannel>
get() = _clients.toMutableList()
注意:我不喜欢这种语法,因为它强制您记住公开的字段(通过_前缀名称访问它们)。我可能会使用clientsInternal
,因此我的IDE可以推荐它,并且我可以选择而无需回溯来添加_。
答案 2 :(得分:1)
如果您的访问权限是单线程的,我认为您可以通过以下操作避免创建副本:
private val clientsList = mutableListOf<SocketChannel>()
val clients: List<out SocketChannel>
get() = clientsList as List<out SocketChannel>
答案 3 :(得分:0)
@JvmField
val clients = mutableListOf<SocketChannel>()
fun getClients() = clients.toMutableList()