:two: is the best one, but you shouldn't write get...

# codereview

[kristofdho]

2️⃣ is the best one, but you shouldn't write getters like that, just use a property:

class MainViewModel @Inject constructor(private val movieRepository: MovieRepository) : ViewModel() {

    val movies: LiveData<List<MovieEntry>> = movieRepository.getMovies()
}

1️⃣ only makes sense if the list was mutable and you needed to edit it internally, & the getter exposes an immutable list
3️⃣ you shouldn't put function declarations in init blocks