feat: 4주차 미션_묵은지 by mookeunji05 · Pull Request #32 · SWU-UMC/10th_Android

mookeunji05 · 2026-04-08T03:47:58Z

📌 PR 제목

4주차 미션

🔗 관련 이슈

Closes #이슈번호

✨ 변경 사항

기능1 추가
UI 수정
버그 수정

🔍 테스트

테스트 완료
에러 없음

📸 스크린샷 (선택)

첫번쨰	두번째	세번째

🚨 추가 이슈

sua710

잘 구현 하신 것 같아요!

sua710 · 2026-04-08T10:50:46Z

Week3/app/src/main/java/com/example/week3/DataStoreManager.kt

+class DataStoreManager(private val context: Context) {
+    private val gson = Gson()
+    private val PRODUCT_LIST_KEY = stringPreferencesKey("product_list")
+    private val WISHLIST_KEY = stringPreferencesKey("wishlist_key")


DataStore로 변경 잘하신거 같아요!!

Lemon0610 · 2026-04-08T10:58:05Z

Week3/app/src/main/java/com/example/week3/PurchaseFragment.kt

+class PurchaseFragment : Fragment(R.layout.fragment_purchase) {
+
+    private lateinit var dataStoreManager: DataStoreManager
+
+    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
+        super.onViewCreated(view, savedInstanceState)
+        dataStoreManager = DataStoreManager(requireContext())
+
+
+        val tabLayout = view.findViewById<TabLayout>(R.id.purchase_tab_layout)
+
+
+        if (tabLayout.tabCount < 4) {
+            tabLayout.addTab(tabLayout.newTab().setText("Sale"))
+        }
+
+        tabLayout.addOnTabSelectedListener(object : TabLayout.OnTabSelectedListener {
+            override fun onTabSelected(tab: TabLayout.Tab?) {
+
+                when (tab?.position) {
+                    0 -> { /* 전체 */ }
+                    1 -> { /* Tops & T-Shirts */ }
+                    2 -> { /* Shoes */ }
+                    3 -> { /* Sale */ }
+                }
+            }
+            override fun onTabUnselected(tab: TabLayout.Tab?) {}
+            override fun onTabReselected(tab: TabLayout.Tab?) {}
+        })
+
+
+        val heartBtn = view.findViewById<ImageView>(R.id.iv_socks)
+
+        heartBtn.setOnClickListener {
+            val currentProduct = Product("Nike Everyday Plus", "$10", R.drawable.socks)
+            toggleWishlist(currentProduct)
+        }
+    }
+
+    private fun toggleWishlist(product: Product) {
+        viewLifecycleOwner.lifecycleScope.launch {
+            // 1. 현재 위시리스트 가져오기
+            val currentWishlist = dataStoreManager.getWishlist().first().toMutableList()
+
+            // 2. 이미 있으면 제거, 없으면 추가
+            val isExist = currentWishlist.any { it.name == product.name }
+            if (isExist) {
+                currentWishlist.removeAll { it.name == product.name }
+                Toast.makeText(context, "위시리스트에서 제거되었습니다.", Toast.LENGTH_SHORT).show()
+            } else {
+                currentWishlist.add(product)
+                Toast.makeText(context, "위시리스트에 추가되었습니다!", Toast.LENGTH_SHORT).show()
+            }
+
+            dataStoreManager.saveWishlist(currentWishlist)
+        }
+    }
+}


아이템이 가로로 2개씩 뜨도록 만드는게 좋을 것 같아용

jeongkyueun · 2026-04-08T14:26:52Z

Week2(Final)/app/src/main/res/layout/fragment_home.xml

+    <TextView
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="9월 4일 목요일"


android:text 항목들 strings에 옮기면 유지보수에 좋을 거 같아요 ! 과제 수행하시느라 수고 많으셨습니다~!!

kimdoyeon1234

TabLayout과 DataStore를 연동해서 실시간으로 하트 아이콘 상태를 바꾸는 로직을 아주 잘 구현하셨네요! 다만 현재 레이아웃이 하드코딩 방식이라 레몬 말씀대로 가로 2줄 배치를 구현하기엔 한계가 있어 보입니다!!

이번 기회에 RecyclerView를 도입해서 더 유연하고 효율적인 리스트 구조로 3주차 피드백 반영과 함께 리팩토링 해보는것도 좋을거 같습니다! 고생 많으셨습니다!

kimdoyeon1234 · 2026-04-13T15:55:10Z

Week3/app/src/main/res/layout/fragment_purchase.xml

+    <ScrollView
+        android:id="@+id/purchase_scroll_view"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent">
+
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:orientation="vertical"
+            android:padding="16dp">
+
+            <FrameLayout
+                android:layout_width="match_parent"
+                android:layout_height="300dp">
+                <ImageView
+                    android:id="@+id/iv_socks"
+                    android:layout_width="match_parent"
+                    android:layout_height="match_parent"
+                    android:src="@drawable/socks"
+                    android:scaleType="centerInside"
+                    android:background="#F9F9F9" />
+                <ImageView
+                    android:id="@+id/btn_heart1"
+                    android:layout_width="40dp"
+                    android:layout_height="40dp"
+                    android:layout_gravity="bottom|end"
+                    android:layout_margin="12dp"
+                    android:src="@drawable/ic_heart_empty"
+                    android:padding="8dp"/>
+            </FrameLayout>
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="Training Crew Socks" android:textColor="#888888" android:textSize="14sp" android:layout_marginTop="16dp" />
+            <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Nike Everyday Plus Cushioned" android:textColor="@color/black" android:textSize="22sp" android:textStyle="bold" android:layout_marginTop="4dp" />
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="$10" android:textColor="@color/black" android:textSize="14sp" android:layout_marginTop="8dp" android:layout_marginBottom="40dp" />
+
+            <FrameLayout
+                android:layout_width="match_parent"
+                android:layout_height="300dp">
+                <ImageView
+                    android:id="@+id/iv_socks2"
+                    android:layout_width="match_parent"
+                    android:layout_height="match_parent"
+                    android:src="@drawable/socks"
+                    android:scaleType="centerInside"
+                    android:background="#F9F9F9" />
+                <ImageView
+                    android:id="@+id/btn_heart2"
+                    android:layout_width="40dp"
+                    android:layout_height="40dp"
+                    android:layout_gravity="bottom|end"
+                    android:layout_margin="12dp"
+                    android:src="@drawable/ic_heart_empty"
+                    android:padding="8dp"/>
+            </FrameLayout>
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="Training Crew Socks" android:textColor="#888888" android:textSize="14sp" android:layout_marginTop="16dp" />
+            <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Nike Everyday Plus Cushioned2" android:textColor="@color/black" android:textSize="22sp" android:textStyle="bold" android:layout_marginTop="4dp" />
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="$10" android:textColor="@color/black" android:textSize="14sp" android:layout_marginTop="8dp" android:layout_marginBottom="40dp" />
+
+            <FrameLayout
+                android:layout_width="match_parent"
+                android:layout_height="300dp">
+                <ImageView
+                    android:id="@+id/iv_socks3"
+                    android:layout_width="match_parent"
+                    android:layout_height="match_parent"
+                    android:src="@drawable/socks"
+                    android:scaleType="centerInside"
+                    android:background="#F9F9F9" />
+                <ImageView
+                    android:id="@+id/btn_heart3"
+                    android:layout_width="40dp"
+                    android:layout_height="40dp"
+                    android:layout_gravity="bottom|end"
+                    android:layout_margin="12dp"
+                    android:src="@drawable/ic_heart_empty"
+                    android:padding="8dp"/>
+            </FrameLayout>
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="Training Crew Socks" android:textColor="#888888" android:textSize="14sp" android:layout_marginTop="16dp" />
+            <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Nike Everyday Plus Cushioned3" android:textColor="@color/black" android:textSize="22sp" android:textStyle="bold" android:layout_marginTop="4dp" />
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="$10" android:textColor="@color/black" android:textSize="14sp" android:layout_marginTop="8dp" android:layout_marginBottom="40dp" />
+
+            <FrameLayout
+                android:layout_width="match_parent"
+                android:layout_height="300dp">
+                <ImageView
+                    android:id="@+id/iv_socks4"
+                    android:layout_width="match_parent"
+                    android:layout_height="match_parent"
+                    android:src="@drawable/socks"
+                    android:scaleType="centerInside"
+                    android:background="#F9F9F9" />
+                <ImageView
+                    android:id="@+id/btn_heart4"
+                    android:layout_width="40dp"
+                    android:layout_height="40dp"
+                    android:layout_gravity="bottom|end"
+                    android:layout_margin="12dp"
+                    android:src="@drawable/ic_heart_empty"
+                    android:padding="8dp"/>
+            </FrameLayout>
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="Training Crew Socks" android:textColor="#888888" android:textSize="14sp" android:layout_marginTop="16dp" />
+            <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Nike Everyday Plus Cushioned4" android:textColor="@color/black" android:textSize="22sp" android:textStyle="bold" android:layout_marginTop="4dp" />
+            <TextView android:layout_width="wrap_content" android:layout_height="wrap_content" android:text="$10" android:textColor="@color/black" android:textSize="14sp" android:layout_marginTop="8dp" android:layout_marginBottom="40dp" />
+
+        </LinearLayout>
+    </ScrollView>


현재 ScrollView 안에 유사한 형태의 아이템 레이아웃(FrameLayout 및 TextView들)이 반복적으로 직접 선언되어 있습니다. 아이템이 늘어날 경우 코드 관리가 어려워지고 메모리 효율도 떨어질 수 있어요. RecyclerView와 GridLayoutManager(spanCount = 2)를 사용하면 레몬 의견처럼 가로 2줄 배치도 쉽게 구현할 수 있고, 코드도 훨씬 깔끔해질 것 같습니다!

kimdoyeon1234 · 2026-04-13T15:56:11Z

Week3/app/src/main/java/com/example/week3/PurchaseFragment.kt

+
+        val tabLayout = view.findViewById<TabLayout>(R.id.purchase_tab_layout)
+        val scrollView = view.findViewById<ScrollView>(R.id.purchase_scroll_view)
+
+
+        val heartButtons = listOf<ImageView>(
+            view.findViewById(R.id.btn_heart1),
+            view.findViewById(R.id.btn_heart2),
+            view.findViewById(R.id.btn_heart3),
+            view.findViewById(R.id.btn_heart4)
+        )


현재 heartButtons를 리스트로 만들기 위해 findViewById를 개별적으로 호출하고 있는데, 만약 아이템이 100개라면 관리가 불가능해집니다. RecyclerView를 도입해서 Adapter 내부에서 ViewBinding으로 처리하도록 개선하면 좋을거같습니다!

kimdoyeon1234 · 2026-04-13T15:57:27Z

Week3/app/src/main/java/com/example/week3/PurchaseFragment.kt

+    private fun toggleWishlist(product: Product) {
+        viewLifecycleOwner.lifecycleScope.launch {
+            val currentWishlist = dataStoreManager.getWishlist().first().toMutableList()
+            val isExist = currentWishlist.any { it.name == product.name }
+
+            if (isExist) {
+                currentWishlist.removeAll { it.name == product.name }
+                Toast.makeText(context, "위시리스트 제거", Toast.LENGTH_SHORT).show()
+            } else {
+                currentWishlist.add(product)
+                Toast.makeText(context, "위시리스트 추가!", Toast.LENGTH_SHORT).show()
+            }
+            dataStoreManager.saveWishlist(currentWishlist)
+        }


Fragment 내부에서 dataStoreManager를 직접 호출해 위시리스트를 추가/삭제하는 로직을 수행하고 있습니다. 나중에 앱이 커지면 유지보수가 힘들 수 있으니, 데이터 처리는 ViewModel로 옮기고 Fragment는 관찰만 하는 구조로 하면은 좋을거 같습니다..

mookeunji05 added 15 commits March 19, 2026 18:14

기록

7df3da1

feat: 감정우표 구현

f87d7ad

감정 우표 UI 및 클릭 이벤트 구현 완료감정 우표 UI 및 클릭 이벤트 구현 완료

ed0d9fa

감정 우표 UI 및 클릭 이벤트 구현 완료감정 우표 UI 및 클릭 이벤트 구현 완료

9b20c30

feat:감정우표 ui구현

930ddfb

2주차 미션 제출

c74ff8e

fix: README.md 충돌 해결 및 복구

147b9fb

fix: 2주차 피드백 반영 및 코드 수정

a0cb87c

상품 목록 기능 및 위시리스트 매니저 추가

8858995

Week 2 과제 제출

247767c

2주차 미션

611b5f7

2주차미션

342f644

feat: 3주차 미션

9b171d5

fix: 3주차 미션 수정

501f24f

4주차 미션

7592b48

mookeunji05 changed the title ~~4주차 미션_묵은~~ 4주차 미션_묵은지 Apr 8, 2026

mookeunji05 changed the title ~~4주차 미션_묵은지~~ feat:4주차 미션_묵은지 Apr 8, 2026

mookeunji05 changed the title ~~feat:4주차 미션_묵은지~~ feat: 4주차 미션_묵은지 Apr 8, 2026

sua710 reviewed Apr 8, 2026

View reviewed changes

Lemon0610 reviewed Apr 8, 2026

View reviewed changes

jeongkyueun approved these changes Apr 8, 2026

View reviewed changes

feat: 미션4 수정

81c6fec

kimdoyeon1234 reviewed Apr 13, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 4주차 미션_묵은지#32

feat: 4주차 미션_묵은지#32
mookeunji05 wants to merge 16 commits intomainfrom
mookeunji-4

mookeunji05 commented Apr 8, 2026 •

edited by kimdoyeon1234

Loading

Uh oh!

sua710 left a comment

Uh oh!

sua710 Apr 8, 2026

Uh oh!

Lemon0610 Apr 8, 2026

Uh oh!

jeongkyueun Apr 8, 2026

Uh oh!

kimdoyeon1234 left a comment

Uh oh!

kimdoyeon1234 Apr 13, 2026

Uh oh!

kimdoyeon1234 Apr 13, 2026

Uh oh!

kimdoyeon1234 Apr 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

mookeunji05 commented Apr 8, 2026 • edited by kimdoyeon1234 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 PR 제목

🔗 관련 이슈

✨ 변경 사항

🔍 테스트

📸 스크린샷 (선택)

🚨 추가 이슈

Uh oh!

sua710 left a comment

Choose a reason for hiding this comment

Uh oh!

sua710 Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

Lemon0610 Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

jeongkyueun Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

kimdoyeon1234 left a comment

Choose a reason for hiding this comment

Uh oh!

kimdoyeon1234 Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

kimdoyeon1234 Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

kimdoyeon1234 Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

mookeunji05 commented Apr 8, 2026 •

edited by kimdoyeon1234

Loading