[이상혁] 프리코스 미션 제출합니다. #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open

pureliture wants to merge 34 commits into next-step:main from pureliture:lsh1756

README.md

-Original file line number
+Diff line change
@@ -1 +1,39 @@
-    # spring-sunshine-precourse
+    # 날씨 조회 서비스 (Weather Service)
+    ## 기능 목록
+. **도시 좌표 조회**: 입력된 도시 이름(Seoul, Tokyo, NewYork, Paris, London)에 해당하는 위도/경도를 조회한다.
+. **날씨 정보 조회**: Open-Meteo API를 사용하여 해당 좌표의 현재 날씨(온도, 체감 온도, 날씨 코드, 습도, 풍속)를 조회한다.
+. **날씨 요약 생성**: 조회된 날씨 정보를 바탕으로 사용자에게 친화적인 한 줄 요약 문장을 생성한다.
+. **API 응답**: 조회된 날씨 정보와 요약 문장을 JSON 형태로 반환한다.
+. **예외 처리**: 지원하지 않는 도시나 외부 API 오류 발생 시 정해진 포맷의 에러 응답을 반환한다.
+. **설정 외부화**: 도시 좌표 및 날씨 코드를 프로퍼티 파일로 분리하여 관리한다.
+. **회복 탄력성(Resilience)**: 외부 API 호출 시 타임아웃 및 재시도 기능을 통해 네트워크 불안정에 대비한다.
+    ## 구현 전략
+. Jules, Codex 2개의 Sandboxed AI Coding Agent를 활용하여 초기 코드 개발
+. guideline 파일을 통해 Backend Architecture, 개발 규칙 등을 사전에 정의하고 Agent가 이를 참조하여 개발
+. 가능한 사전에 `build.gradle.kts`에 정의되어있던 의존성울 모두 사용하여 기능을 구현
+. Agent가 개발한 코드를 검토하고 필요 시 리팩토링 수행
+    ## 구현 상세
+    * **아키텍처**: Domain Layer는 컴포넌트 기반, Service Layer가 Facade 역할로 조합하는 구조를 적용한다.
+        * `api`: 컨트롤러 및 요청/응답 DTO (검증 및 위임).
+        * `service`: Service Layer, Facade 역할로 여러 컴포넌트를 오케스트레이션.
+        * `component`: 단일 책임을 가진 작은 컴포넌트들 (좌표 매핑, 요약 생성 등).
+        * `domain`: 도메인 모델, 도메인 서비스.
+        * `repository`: 도메인 리포지토리 포트 정의.
+        * `infrastructure`: 외부 API 통신 (Open-Meteo Client) 등 기술 어댑터.
+    * **제약 사항 준수**:
+        * `else`, `switch`, `ternary operator` 사용 금지.
+        * 들여쓰기 2단계 제한.
+        * 메서드 길이 15줄 제한.
+        * Google Java Style Guide (4 spaces indent).
+    * **No-Else/Switch 전략**:
+        * 도시 매핑: `Map<String, Coordinate>` 자료구조를 활용하여 조건문 없이 O(1) 조회.
+        * 날씨 코드 매핑: `Map<Integer, String>` 활용.
+        * 방어적 코딩(Guard Clauses)을 통해 `else` 제거.
+    * **설정 관리**:
+        * `CityProperties`, `WeatherProperties` 클래스를 통해 외부 설정(YAML)을 객체로 바인딩하여 사용.
+    * **회복 탄력성**:
+        * `RestClient` 설정 시 `HttpClient` 타임아웃(Connect, Response) 적용.
+        * `ClientHttpRequestInterceptor`를 구현하여 실패 시 재시도 로직 적용 (Custom Retry Interceptor).

build.gradle.kts

            
                      Original file line number
                      Diff line number
                      Diff line change
                  
    @@ -1,9 +1,7 @@
  
    plugins {

        id("org.springframework.boot") version "3.3.1"

        id("io.spring.dependency-management") version "1.1.5"

        kotlin("plugin.jpa") version "1.9.24"

        kotlin("jvm") version "1.9.24"

        kotlin("plugin.spring") version "1.9.24"

        id("java")

    }

    group = "camp.nextstep.edu"

    @@ -24,21 +22,14 @@ dependencies {
  
        implementation("org.springframework.boot:spring-boot-starter-thymeleaf")

        implementation("org.springframework.boot:spring-boot-starter-validation")

        implementation("org.springframework.boot:spring-boot-starter-web")

        implementation("com.fasterxml.jackson.module:jackson-module-kotlin")

        implementation("org.springframework.boot:spring-boot-starter-reactor-netty")

        implementation("org.flywaydb:flyway-core")

        implementation("org.flywaydb:flyway-mysql")

        implementation("org.jetbrains.kotlin:kotlin-reflect")

        runtimeOnly("com.h2database:h2")

        runtimeOnly("com.mysql:mysql-connector-j")

        testImplementation("org.springframework.boot:spring-boot-starter-test")

        testImplementation("org.jetbrains.kotlin:kotlin-test-junit5")

        testRuntimeOnly("org.junit.platform:junit-platform-launcher")

    }

    kotlin {

        compilerOptions {

            freeCompilerArgs.addAll("-Xjsr305=strict")

        }

        implementation("com.h2database:h2")

        implementation("com.mysql:mysql-connector-j")

        testImplementation("org.springframework.boot:spring-boot-starter-test")

        testImplementation("org.junit.platform:junit-platform-launcher")

    }

    tasks.withType<Test> {

gradle/wrapper/gradle-wrapper.jar

Binary file not shown.

gradle/wrapper/gradle-wrapper.properties

-Original file line number
+Diff line change
@@ -1,6 +1,6 @@
     distributionBase=GRADLE_USER_HOME
     distributionPath=wrapper/dists
-    distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-bin.zip
+    distributionUrl=https\://services.gradle.org/distributions/gradle-8.5-bin.zip
     networkTimeout=10000
     validateDistributionUrl=true
     zipStoreBase=GRADLE_USER_HOME
@@ Expand Down @@

gradlew

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

gradlew.bat

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[이상혁] 프리코스 미션 제출합니다. #7

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Uh oh!

Uh oh!

[이상혁] 프리코스 미션 제출합니다. #7

Are you sure you want to change the base?

Uh oh!

[이상혁] 프리코스 미션 제출합니다. #7

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Uh oh!

Uh oh!