omjoonkim/GitHubBrowserApp

APIService 내의 Endpoint object

Closed this issue · 1 comments

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 변수로 정의해놓는 것이 어떨까요?

코드 수정 되었습니당. :)