Skip to content

Commit

Permalink
Merge pull request #196 from Team-Notitime/docs/189-README
Browse files Browse the repository at this point in the history
docs: LearningPoint.md 수정 (#189)
  • Loading branch information
easyhz authored Sep 3, 2024
2 parents c9d58e6 + 5153f6c commit 97fd95a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions docs/LearningPoint.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,13 @@

### Problem

a) 공지 발행의 특성상 다양한 옵션 데이터를 관리해야 했는데, 이로 인해 상태 관리가 쉽지 않았습니다.
b) 이미지 수정 시, 기존 이미지 URL에 새 이미지를 덮어씌우는 방식으로 진행했는데, URL의 데이터는 변경되었지만 UI에서 변경 사항이 반영되지 않는 이슈가 발생했습니다.
1. 공지 발행의 특성상 다양한 옵션 데이터를 관리해야 했는데, 이로 인해 상태 관리가 쉽지 않았습니다.
2. 이미지 수정 시, 기존 이미지 URL에 새 이미지를 덮어씌우는 방식으로 진행했는데, URL의 데이터는 변경되었지만 UI에서 변경 사항이 반영되지 않는 이슈가 발생했습니다.

### Try

a) 먼저 `ViewModel` 의 책임을 분리하고, 옵션 데이터를 `sealed class` 로 정의하여 일관성 있는 구조를 구축했습니다. 또한, `LinkedHashMap` 을 활용하여 옵션 선택 여부 등의 상태를 관리함으로써 타입 안정성과 확장성을 확보했습니다.
b) `이미지 url`, 보이는 이미지 크기, `description` 등은 바뀌지 않고 url에 해당하는 이미지 데이터만 바뀌는 거라서 이미 캐싱 된 이미지를 사용함으로써 나타나는 이슈였습니다. 이미지 캐시 정책을 수정하여 문제를 해결하였습니다.
1. 먼저 `ViewModel` 의 책임을 분리하고, 옵션 데이터를 `sealed class` 로 정의하여 일관성 있는 구조를 구축했습니다. 또한, `LinkedHashMap` 을 활용하여 옵션 선택 여부 등의 상태를 관리함으로써 타입 안정성과 확장성을 확보했습니다.
2. `이미지 url`, 보이는 이미지 크기, `description` 등은 바뀌지 않고 url에 해당하는 이미지 데이터만 바뀌는 거라서 이미 캐싱 된 이미지를 사용함으로써 나타나는 이슈였습니다. 이미지 캐시 정책을 수정하여 문제를 해결하였습니다.

### 느낀점

Expand Down

0 comments on commit 97fd95a

Please sign in to comment.