Hey guys, which one of these approaches you think would be better in terms of performance, memory, e...

# codereview

[Diego Almeida de Oliveira]

Hey guys, which one of these approaches you think would be better in terms of performance, memory, etc?
1

class Foo {

	private val logger: Logger by lazy { Logger(getApplicationContext()) } 

	fun aFunctionCalledMultipleTimes() {
		logger.doSomething()
	}
}

2

class Foo {

	fun aFunctionCalledMultipleTimes() {
		val logger = Logger(getApplicationContext())
		logger.doSomething()
	}
}

[Shawn]

optimizing references to loggers is like the textbook definition of overoptimization

[Shawn]

1 makes way more sense logically and is far more ergonomic

[Daniel]

You can even ditch the by lazy . Provided the Logger does not do something crazy on initialization