[Week8] Jamie #41

HwangJaemin49 · 2024-11-29T11:42:10Z

📌 이슈 번호

ex) #1

💻 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)
회원가입, 로그인/로그아웃 기능 구현
앨범 좋아요 기능 추가

📢 기타 사항

SuHyeon00

아래 코멘트에 남겨진 부분대로 수정한 후 실행해보았을 때 기능들은 제대로 동작하는 것 같아요! 현재 flo 프로젝트의 각 클래스들이 어떤 기능을 하는 것인지 파악할 필요가 있어 보입니다! 각 기능에 맞추어 폴더도 정리해보시는 걸 추천드려요~ 실습 하시느라 수고하셨습니다!

SuHyeon00 · 2024-12-05T08:24:30Z

flo-clone/app/src/main/res/layout/activity_main.xml

+        android:id="@+id/main_player_cl"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:background="@color/player_bg"


해당 리소스를 찾을 수 없다고 뜹니다! colors.xml 파일에 정의된 컬러를 연결해주시거나 새로 색상 정의해주세요~

SuHyeon00 · 2024-12-05T08:28:18Z

flo-clone/app/src/main/java/com/example/flo_clone/SavedAlbumFragment.kt

@@ -0,0 +1,60 @@
+package com.example.flo


현재 SavedAlbumFragment 파일의 패키지 설정이 com.example.flo로 되어있어서 LockerRVAdapter 클래스에서 SavedAlbumFragment를 임포트하는 부분과 충돌이 나고 있습니다. 해당 패키지를 com.example.flo_clone으로 수정해주시거나 LockerRVAdapter에 임포트문을 수정해주세요!

SuHyeon00 · 2024-12-05T08:28:37Z

flo-clone/app/src/main/java/com/example/flo_clone/ui/locker/LockerRVAdapter.kt

+
+import androidx.fragment.app.Fragment
+import androidx.viewpager2.adapter.FragmentStateAdapter
+import com.example.flo_clone.SavedAlbumFragment


이부분입니다~

SuHyeon00 · 2024-12-05T08:33:45Z

flo-clone/app/src/main/AndroidManifest.xml

+        <activity
+            android:name=".MainActivity"
+            android:exported="true"
+            android:label="@string/app_name">
+            <intent-filter>
+                <action android:name="android.intent.action.MAIN" />
+
+                <category android:name="android.intent.category.LAUNCHER" />
+            </intent-filter>
+        </activity>
+
+        <activity
+            android:name=".SongActivity"
+            android:exported="true" />


MainActivity와 SongActivity를 mainfest 파일에 추가해주신 것처럼 새로운 액티비티를 사용하신다면 (LoginActivity, SingUpActivity) 이 파일에 항상 추가해주셔야지 화면에 불러올 수 있습니다!

SuHyeon00 · 2024-12-05T08:44:54Z

flo-clone/app/src/main/java/com/example/flo_clone/SignUpActivity.kt

+            signUp()
+            finish()


finish() 함수를 이곳에 두게 된다면 signUp이 제대로 되지 않았을 때에도 회원가입 화면이 종료되기 때문에 finish는 회원가입이 성공했을 경우에만 현재 액티비티가 종료되도록 설정해주세요!

SuHyeon00 · 2024-12-05T09:03:42Z

flo-clone/app/src/main/java/com/example/flo_clone/MainActivity.kt

+import androidx.media3.common.util.Log
+import androidx.media3.common.util.UnstableApi


로그 클래스는 android.util에서 임포트해주세요!

SuHyeon00 · 2024-12-05T09:08:34Z

flo-clone/app/src/main/java/com/example/flo_clone/SavedAlbumFragment.kt

+            override fun onRemoveSong(songId: Int) {
+                albumDB.albumDao().getLikedAlbums(getJwt())
+            }


클릭 리스너 네이밍과 실제로 동작하는 기능이 매치되지 않는 것 같아요. AlbumLockerRVAdapter의 onRemove 함수 동작을 살펴보면 더보기 아이콘을 눌렀을 때 좋아요했던 앨범이 좋아요가 해제되어야 하는 것 같은데 현재는 로그인 한 유저의 좋아요한 앨범 목록을 불러오는 기능으로 구현되어있는 것 같네요. 해당 부분 수정 부탁드려요!

추가로 파라미터 네이밍도 song이 아닌 album과 관련된 기능이기 때문에 수정할 필요가 있어 보입니다..!

SuHyeon00 · 2024-12-05T09:25:54Z

flo-clone/app/src/main/java/com/example/flo_clone/ui/home/HomeFragment.kt

+//        binding.homeAlbumImgIv1.setOnClickListener {
+//            var albumFragment = AlbumFragment()
+//            albumFragment.arguments = Bundle().apply {
+//                putString("title", binding.homeAlbumTitleTv1.text.toString())
+//                putString("singer", binding.homeAlbumSingerTv1.text.toString())
+//            }
+//            (context as MainActivity)
+//                .supportFragmentManager
+//                .beginTransaction()
+//                .replace(R.id.nav_host_fragment_activity_main, albumFragment)
+//                .commitAllowingStateLoss()
+//        }


사용하지 않는 코드는 지워주세요!

SuHyeon00 · 2024-12-05T09:30:41Z

flo-clone/app/src/main/java/com/example/flo_clone/ui/album/AlbumRVAdapter.kt

+        companion object {
+            fun onItemClick(album: Album) {
+
+            }
+
+            fun onRemoveAlbum(position: Int) {
+
+            }
+        }
+    }


이부분은 뭘까요..?!

SuHyeon00 · 2024-12-05T09:32:53Z

flo-clone/app/src/main/java/com/example/flo_clone/ui/home/HomeFragment.kt

+            }
+
+            override fun onRemoveAlbum(position: Int) {
+                albumRVAdapter.removeItem(position)


앨범 삭제와 관련된 로직이 homeFragment에서 동작할 기능이 없고 AlbumRVAdapter 내부에서만 동작하면 되는 기능이라면 굳이 클릭 리스너로 외부에서 설정할 필요 없이 AlbumRVAdpater의 onRemove 함수를 정의하셔서 해당 함수로 연결해주시면 될 것 같아요~

HwangJaemin49 added 8 commits November 13, 2024 11:49

[week6] Flo Clone Coding

5b817c0

[week6] Flo Clone Coding

d2d49d9

[week6] Flo Clone Coding

2bfa87a

[week6] .idea directory delete

d57b16c

7주ì°[week7] Database

cbc7b96

[week7] Database

2f4e6ca

[week 7] 좋아요 기능 추가

c08df4f

[week8] 회원가입, 로그인/로그아웃, 앨범 좋아요

f0f2264

SuHyeon00 self-requested a review December 5, 2024 08:17

SuHyeon00 assigned HwangJaemin49 Dec 5, 2024

SuHyeon00 approved these changes Dec 5, 2024

View reviewed changes

		import androidx.media3.common.util.Log
		import androidx.media3.common.util.UnstableApi

[Week8] Jamie #41

Are you sure you want to change the base?

[Week8] Jamie #41

Uh oh!

Conversation

HwangJaemin49 commented Nov 29, 2024

📌 이슈 번호

💻 작업 내용

📢 기타 사항

Uh oh!

SuHyeon00 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants