AppModule 내의 변수 접근 범위 변경 외 몇 가지

GitHubBrowserApp/app/src/main/java/com/omjoonkim/app/mission/di/AppModule.kt

Lines 19 to 56 in 126ed77

    
           class AppModule(private val app: App) { 
        
               val applicationContexxt get() = app.applicationContext 
        
               val gson by lazy { Gson() } 
        
               val apiClient : APIClientType by lazy { APIClient(apiService) } 
        
               private val loggingInterceptor by lazy { HttpLoggingInterceptor().apply { level = HttpLoggingInterceptor.Level.BODY } } 
        
               private val okHttpClient by lazy { 
        
                   OkHttpClient.Builder() 
        
                       .readTimeout(10, TimeUnit.SECONDS) 
        
                       .connectTimeout(10, TimeUnit.SECONDS) 
        
                       .addInterceptor(loggingInterceptor) 
        
                       .build() 
        
               } 
        
               private val apiService : APIService by lazy { 
        
                   Retrofit.Builder() 
        
                       .client(okHttpClient) 
        
                       .baseUrl(APIService.EndPoint.baseUrl) 
        
                       .addCallAdapterFactory(RxJava2CallAdapterFactory.create()) 
        
                       .addConverterFactory(GsonConverterFactory.create(gson)) 
        
                       .build() 
        
                       .create(APIService::class.java) 
        
               } 
        
               val gitHubDataRepository : GitHubRepositoryType by lazy { GitHubDataRepository(apiClient) } 
        
               val enviorment by lazy { 
        
                   Environment( 
        
                       apiClient, 
        
                       gitHubDataRepository, 
        
                       gson 
        
                   ) 
        
               } 
        
           }

AppModule.kt 내의 변수 접근 범위 변경

gson / apiClient / gitHubDataRepository 변수 모두 environment 라는 변수를 통해 제공이 되고 있습니다.
위의 변수들을 private 으로 두지 않는다면 environment 로서 따로 제공하는 의미가 없을 듯 합니다.

apiClient 변수 제거

gitHubDataRepository 는 apiClient 를 Wrapping 하여 기능을 하고있습니다.
즉 environment 변수에서 apiClient 를 따로 제공할 필요가 없습니다.
실제로 environment 변수에서 apiClient 를 받아서 사용하는 경우가 존재하지 않습니다.

변수 오타 수정

enviroment -> environment

APIService 내의 Endpoint object

GitHubBrowserApp/app/src/main/java/com/omjoonkim/app/mission/network/APIService.kt

Lines 9 to 20 in 126ed77

    
           interface APIService { 
        
               object EndPoint { 
        
                   const val baseUrl = "https://api.github.com" 
        
               } 
        
               @GET("/users/{userName}") 
        
               fun getUserInfo(@Path("userName") userName: String): Single<User> 
        
               @GET("/users/{userName}/repos") 
        
               fun getUserRepos(@Path("userName") userName: String): Single<List<Repo>> 
        
           }

Endpoint object 제거

Endpoint 라는 object name 에 baseUrl 밖에 존재하지 않습니다.
endpoint 변수를 추가한다면 필요한 object 가 되겠으나 현재 그렇게 사용하고 있지 않으니 제거하는 것이 어떨까요?

baseUrl 위치 변경

baseUrl 자체를 build.gradle 내의 buildConfigField 변수로 정의해놓는 것이 어떨까요?

omjoonkim / githubbrowserapp Goto Github PK

githubbrowserapp's Issues

AppModule 내의 변수 접근 범위 변경 외 몇 가지

AppModule.kt 내의 변수 접근 범위 변경

apiClient 변수 제거

변수 오타 수정

APIService 내의 Endpoint object

Endpoint object 제거

baseUrl 위치 변경

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	class AppModule(private val app: App) {

	val applicationContexxt get() = app.applicationContext

	val gson by lazy { Gson() }

	val apiClient : APIClientType by lazy { APIClient(apiService) }

	private val loggingInterceptor by lazy { HttpLoggingInterceptor().apply { level = HttpLoggingInterceptor.Level.BODY } }

	private val okHttpClient by lazy {
	OkHttpClient.Builder()
	.readTimeout(10, TimeUnit.SECONDS)
	.connectTimeout(10, TimeUnit.SECONDS)
	.addInterceptor(loggingInterceptor)
	.build()
	}

	private val apiService : APIService by lazy {
	Retrofit.Builder()
	.client(okHttpClient)
	.baseUrl(APIService.EndPoint.baseUrl)
	.addCallAdapterFactory(RxJava2CallAdapterFactory.create())
	.addConverterFactory(GsonConverterFactory.create(gson))
	.build()
	.create(APIService::class.java)
	}

	val gitHubDataRepository : GitHubRepositoryType by lazy { GitHubDataRepository(apiClient) }

	val enviorment by lazy {
	Environment(
	apiClient,
	gitHubDataRepository,
	gson
	)
	}
	}

	interface APIService {
	object EndPoint {
	const val baseUrl = "https://api.github.com"
	}

	@GET("/users/{userName}")
	fun getUserInfo(@Path("userName") userName: String): Single<User>

	@GET("/users/{userName}/repos")
	fun getUserRepos(@Path("userName") userName: String): Single<List<Repo>>

	}