#219/feature/jsp/backend #222

pknujsp · 2024-03-11T16:39:32Z

📌 관련 이슈

#219 클라이언트, 백엔드 연결 코드 추가(로그인, 회원가입 기능 개발)

✨ 과제 내용

파일 변경이 엄청난데 Ai작업할 때 이리저리 테스트하면서 쓴 나중에 제거할 코드가 많아서 따로 정리할 로그인, 회원가입 부분 파일들만 보시면 될거 같습니다.

AccountSessionRepository : 현재 로그인 세션을 관리
AppDataStore : 이메일, 닉네임 저장
CognitoUserPool : AWS 계정 API 라이브러리 모듈 구현체

원래 있던 TokenRepository나 Token 다루는 클래스는 전부 제거했습니다.

CognitoUserPool에서 로그인 세션, 토큰 관리 등등 다 해줘서 우리가 직접 다룰 필요는 없어졌고,

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

…변경되지 않는 문제 수정

…, k2컴파일러 적용

…t,post 메서드 수정

…urce를 aws에 맞게 수정

… 정보(토큰)를 가지고 있는 레포지토리

…류로 가입이 되지 않는 문제 수정

…essionRepository에 추가

tgyuuAn · 2024-03-12T02:47:30Z

파일 체인지드 미르쓰띤 나중에 카페가서 확인해볼게용

pknujsp · 2024-03-12T07:34:19Z

core/data/src/main/java/com/android/mediproject/core/data/session/AccountSessionRepository.kt

+import kotlinx.coroutines.flow.StateFlow
+
+interface AccountSessionRepository {
+    val lastSavedEmail: Flow<String>


로그인 화면에서 아이디 저장 눌렀을 때 자동입력되는 이메일인데
이거는 SignRepository로 가는게 맞겠네요
이 레포는 현재 로그인 세션 관리하는 목적인데 지금 여기서 안쓰이기도 하고 관련이 없네요

pknujsp · 2024-03-12T07:36:52Z

core/domain/src/main/java/com/android/mediproject/core/domain/GetCommentsUseCase.kt

댓글 관련 코드들은 현재 로그인 세션 상태랑 연동하는 로직을 재 구현해야 하는 부분때문에 큰 공사가 될거 같아
로그인/회원가입 구현부터 먼저하고자 임시로 주석처리 해놓은게 많습니다

pknujsp · 2024-03-12T07:39:05Z

...rc/main/java/com/android/mediproject/core/model/requestparameters/ChangePasswordParameter.kt

+class ChangePasswordParameter(
+    val newPassword: ByteArray,
+)


비밀번호 변경, 닉네임 변경하는 파라미터 클래스는 내부 속성을 단순
래핑하는 정도라서 없애고

속성을 실제 닉네임 변경, 비번 변경하는 메서드의 인자로 넘기는게 좋을 듯한데
어떻게 생각하시나요

core/model/requestparameters 패키지 내 모든 파일에 대해서 같은 내용입니다

@pknujsp 그게 맞는 것 같아요. 그냥 파라미터로 전달하고 datasource에서 요청 보내는 게 훨씬 간단할 것 같네요.

이 부분 수정해서 다음 PR에 반영할게요

pknujsp · 2024-03-12T07:43:57Z

feature/intro/src/main/res/layout/dialog_email_verification.xml

+
+            <TextView
+                android:id="@+id/positiveButton"
+                android:layout_width="0dp"
+                android:layout_height="wrap_content"
+                android:layout_marginEnd="15dp"
+                android:background="@drawable/radius_main_5"
+                android:gravity="center"
+                android:paddingVertical="10dp"
+                android:text="@string/complete"
+                android:textColor="@color/white"
+                android:textSize="18sp"
+                app:layout_constraintBottom_toBottomOf="parent"
+                app:layout_constraintEnd_toEndOf="@id/guideline"
+                app:layout_constraintStart_toStartOf="parent"
+                app:layout_constraintTop_toTopOf="parent" />
+


이 뷰를 눌렀을 때 이메일 인증 코드를 뷰모델에서 받아서 인증처리를 하는데
바인딩 어댑터로 직접 메서드를 호출하려고 하니 구현법이 기억이 안나서
우선적으로 프래그먼트 클래스내에서 직접 클릭리스너를 연결해줬습니다

이런 식으로 넘어가면 분명 까먹긴 합니다만,

그것을 방지하기 위해서 어디 써놓던가 해야겠어요.

까먹을수도 있는 것들 찾아보기 쉽게 이슈에 등록해둘게요

tgyuuAn

시간이 지나고 보니까...

너무 컨벤션이 부족했고.. 코드의 품질이 많이 낮았다는 것을 깨닫게 되네요..

그 만큼 저희가 많이 성장했다는 증거겠죠...?!

소소한 코멘트 남겨봤습니다.

고봉밥 PR은 조금 힘드네요 허허...

고생하셨씁니다잇 ~ 👍👍👍

tgyuuAn · 2024-03-12T07:00:05Z

app/build.gradle.kts

@@ -88,7 +88,6 @@ dependencies {

    implementation(libs.bundles.lifecycles)
    implementation(libs.bundles.materials)
-    implementation(libs.bundles.composes)


compose import 제거하신 이유가 따로 있을까요?

app 모듈에서는 컴포즈 관련 코드가 하나도 없어서 제거하였습니다

tgyuuAn · 2024-03-12T07:02:44Z

core/ai/src/main/java/com/android/mediproject/core/ai/camera/CameraHelper.kt

        cameraExecutor = newSingleThreadExecutor()
-        _mImageAnalyzer =
-            ImageAnalysis.Builder().setBackpressureStrategy(ImageAnalysis.STRATEGY_KEEP_ONLY_LATEST).setTargetRotation(mPreviewView.display.rotation)
-                .setOutputImageFormat(ImageAnalysis.OUTPUT_IMAGE_FORMAT_RGBA_8888).build()
+
+        _mImageAnalyzer = ImageAnalysis.Builder().setOutputImageRotationEnabled(true).setBackpressureStrategy(ImageAnalysis.STRATEGY_KEEP_ONLY_LATEST)
+            .setTargetResolution(Size(720, 1280)).setTargetRotation(Surface.ROTATION_0)
+            .setOutputImageFormat(ImageAnalysis.OUTPUT_IMAGE_FORMAT_RGBA_8888).build()
        mImageAnalyzer.setAnalyzer(cameraExecutor!!) { image ->
            if (_bitmapBuffer == null) _bitmapBuffer = Bitmap.createBitmap(image.width, image.height, Bitmap.Config.ARGB_8888)
            image.use {
                bitmapBuffer.copyPixelsFromBuffer(it.planes[0].buffer)
            }
-            detect(image)
+            detect()
        }

-        _mPreview = Preview.Builder().setTargetRotation(mPreviewView.display.rotation).build()
+        _mPreview = Preview.Builder().setTargetResolution(Size(720, 1280)).setTargetRotation(Surface.ROTATION_0).build()
        mCameraProvider.bindToLifecycle(fragmentLifeCycleOwner, mCameraSelector, mPreview, mImageAnalyzer)
        mPreview.setSurfaceProvider(mPreviewView.surfaceProvider)
    }


이 부분 사소하긴한데,

저희가 컨벤션으로 정하지 않았떤 부분이라서..

이렇게 메소드 체이닝이 일어날 때는 개행을 넣어서 구분을 쉽게 하는 것은 어떨까요 ??!

맞아요 전반적으로 코드 가독성이 떨어지네요
개행 넣어서 가독성 높이고 불필요한 코드는 다시 정리하겠습니다

tgyuuAn · 2024-03-12T07:03:11Z

core/ai/src/main/java/com/android/mediproject/core/ai/camera/CameraHelper.kt

+        data class Object(
+            val boundingBox: RectF,
+            val confidence: Int,
+            val label: String,
+        )


object를 interface안에 넣으신 이유가 이씅ㄹ까요?

OnDetectionListener의 목적이 뷰에서 약의 위치에 사각형 박스를 그리는 건데요.
이 때 화면 상에서 약의 위치 좌표값를 담고있는게 Object 클래스인데
이 인터페이스에서만 쓰이기 때문에 관계를 확실하게 정해주고자 이렇게 작성했습니다.

실제로 어떻게 쓰이고 있는지 코드 개선점 고민 해볼겸 작성해봤습니다.

밑에 사진처럼 Ai가 약을 인식했을때 뷰에서 약에 대해서 바운딩 박스를 쳐주기 위한 정보를 약 별로 담는 건데요.

이 클래스가 있는 OnDetectionListener는 카메라를 보여주는 MedicinesDetectorFragment에서 구현해서 쓰이고 있습니다. 이 프래그먼트에서 **onDetect()**를 호출받았을 때 뷰에 그리게 되어있어요.

코드를 읽어보니까 더 개선할 수 있는 코드가 다수인 거 같습니다. 기능이 동작하니 지금 당장 수정은 하지 않는게 좋을거 같고 다음에 MVP단계에서 개선해보겠습니다.

OnDetectionListener의 목적이 뷰에서 약의 위치에 사각형 박스를 그리는 건데요. 이 때 화면 상에서 약의 위치 좌표값를 담고있는게 Object 클래스인데 이 인터페이스에서만 쓰이기 때문에 관계를 확실하게 정해주고자 이렇게 작성했습니다.

실제로 어떻게 쓰이고 있는지 코드 개선점 고민 해볼겸 작성해봤습니다.

밑에 사진처럼 Ai가 약을 인식했을때 뷰에서 약에 대해서 바운딩 박스를 쳐주기 위한 정보를 약 별로 담는 건데요.

이 클래스가 있는 OnDetectionListener는 카메라를 보여주는 MedicinesDetectorFragment에서 구현해서 쓰이고 있습니다. 이 프래그먼트에서 **onDetect()**를 호출받았을 때 뷰에 그리게 되어있어요.

코드를 읽어보니까 더 개선할 수 있는 코드가 다수인 거 같습니다. 기능이 동작하니 지금 당장 수정은 하지 않는게 좋을거 같고 다음에 MVP단계에서 개선해보겠습니다.

조오옿습니다. !

tgyuuAn · 2024-03-12T07:03:35Z

core/ai/src/main/java/com/android/mediproject/core/ai/camera/CameraHelper.kt

+                        it.boundingBox,
+                        it.confidence,
+                        it.label,
+                    )
                },
                detectionResultEntity.inferencedImageSize.width, detectionResultEntity.inferencedImageSize.height,
            )
        }
    }

    fun interface OnDetectionListener {


와우 fun interface는 어떤 기능인가요..?

abstract fun이랑 비슷한건가..

인터페이스에 메서드가 하나일 때
이걸 다른 메서드에 람다 인자로 넣어서 편하게 쓸수가 있게 돼요

tgyuuAn · 2024-03-12T07:05:45Z

core/ai/src/main/java/com/android/mediproject/core/ai/classification/MedicineClassifierImpl.kt

+    private var _aiModels: List<Module>? = null
+    private val aiModels get() = _aiModels!!

-    private var _aiModel: Module? = null
-    private val aiModel get() = _aiModel!!
-
-    private var _metaData: ClassificationMetaDataEntity? = null
+    private var _metaData: List<ClassificationMetaDataEntity>? = null


이거 초기값 null로 주신 이유가 있나요?

binding같은 것들은 nulll로 줘서 onDestroy 에서 null다시 대입해서 메모리 회수하는데,

이 부분은 제가 잘 모르는 부분이라 그런데.. 다시 null 처리해서 메모리를 반환할 필요는 없을까요 ?

MedicineClassifierImpl는 약을 실제로 Ai가 분류하는 코드를 가지고 있는 클래스입니다.

이 클래스에서 80MB 가량되는 Ai모델을 읽어와서 메모리 상에 올리는 로직을 포함하고 있어요.

말하신 것 처럼 멤버변수에 null을 넣는 경우는 이 인스턴스가 필요없어서 지울 때 일텐데

모델 파일 용량이 좀 크다보니 null처리로 메모리에서 삭제, 할당을 여러 번 반복하는 건 비효율적인거 같아서
메모리에 계속 올려두고자 지금처럼 구현했고 object클래스로 작성했습니다.

코드 다시 보니 밑에 처럼 get필드가 있는데 꼭 필요한게 아니라서
이걸 없애고 Delegates.notNull()를 사용해서 코드를 좀 더 줄여보겠습니다.

private var _aiModels: List<Module>? = null private val aiModels get() = _aiModels!! private var _metaData: List<ClassificationMetaDataEntity>? = null private val metaData get() = _metaData!!

@pknujsp 아니면 lateinit var같은 것을 써도 되지 않을까요 한번만 대입하는 거라면

그렇네요
타입이 int같은게 아니라서 위임말고 lateinit쓰는게 맞아요

tgyuuAn · 2024-03-12T07:30:24Z

...etwork/src/main/java/com/android/mediproject/core/network/datasource/sign/LoginDataSource.kt

+class LoginRequest(
+    val email: String,
+    val password: ByteArray,
+)
+
+class LoginResponse(
+    val userSession: CognitoUserSession,
+    val attr: CognitoUserDetails,
+    val newDevice: CognitoDevice? = null,
+)


얘네도 data class도 아니고 갑자기 한 파일 안에 여러 개의 DTO들이 있어요.

저희 원래 이렇게 했었었나요 넘 오래되서 기억이..

플젝 시작할 때 컨벤션을 구체적으로 크게 정하고 가질 않았었어요
이 부분 중요성 크게 느끼고 가네요

tgyuuAn · 2024-03-12T07:43:35Z

...rk/src/main/java/com/android/mediproject/core/network/datasource/sign/LoginDataSourceImpl.kt

+        return getSession(request).fold(
+            onSuccess = { userSession ->
+                getUserAttr(userSession).fold(
+                    onSuccess = { userAttr ->
+                        Result.success(LoginResponse(userSession, userAttr))
+                    },
+                    onFailure = { Result.failure(it) },
+                )
+            },
+            onFailure = { Result.failure(it) },
+        )
+    }


fold() 를 쓰지 않고 onSuccess, onFailure를 사용해서 똑같은 로직을 만들 수 있는데

혹시 fold() 를 처음에 사용하신 이유가 있을까요?!

저도 아무것도 모르고 사용하다가 진호님이랑 프로젝트 하면서 고쳤어서 ㅠㅠ

pknu-wap/WAPP#73 (comment)

getSeesion(), getUserAttr() 이 두 개가 반환 타입이 달라서 onSuccess로 구현하기에는 좀 어려움이 있어서 fold를 썼어요.

mapCatching로 바꾸고 getUserAttr() 결과가 실패일 때 예외를 던져서 지금 코드의 fold 내에 fold에서 동작하는 거랑
똑같은 동작을 하도록 바꿔서 반영할게용

override suspend fun login(request: LoginRequest): Result<LoginResponse> = getSession(request).mapCatching { userSession -> LoginResponse(userSession, getUserAttr(userSession).getOrElse { throw it }) }

tgyuuAn · 2024-03-12T07:45:04Z

...ork/src/main/java/com/android/mediproject/core/network/datasource/user/UserDataSourceImpl.kt

+    override suspend fun changeNickname(changeNicknameParameter: ChangeNicknameParameter): Flow<Result<ChangeNicknameResponse>> = channelFlow {
+        awsNetworkApi.changeNickname(changeNicknameParameter).onResponse()
+            .fold(onSuccess = { Result.success(it) }, onFailure = { Result.failure(it) }).also { trySend(it) }
+    }
+
+    override suspend fun changePassword(changePasswordParameter: ChangePasswordParameter): Flow<Result<ChangePasswordResponse>> = channelFlow {
+        /*      val password = WeakReference(aesCoder.encodePassword(changePasswordParameter.email, changePasswordParameter.newPassword)).get()!!
+              awsNetworkApi.changePassword(ChangePasswordParameter(password.toCharArray())).onResponse()
+                  .fold(onSuccess = { Result.success(it) }, onFailure = { Result.failure(it) }).also { trySend(it) }*/
+    }


여기도 사실 flow를 사용할 필요가 전혀 없는 부분이었네요..

흑흑,.. 이 부분도 다 suspend로 고쳐야하는 부분인 것 같아요.

저 이거 개발할 때 코루틴을 제대로 첨 써봐서 무지성으로 flow때려박은 경향이 있어요
channelFlow때문에 서로 고생했던 기억이 나네요

5월 이네요 ㄷㄷ

tgyuuAn · 2024-03-12T07:46:23Z

core/ui/src/main/java/com/android/mediproject/core/ui/base/view/Subtitle.kt

-    lateinit var title: TextView
-    lateinit var redPoint: TextView
-    lateinit var inputData: EditText
+    var title: TextView
+    var redPoint: TextView
+    var inputData: EditText


여기 lateinit 지우신 이유가 있나요?

바인딩 어댑터 연결시키다가 객체 초기화 오류나서 잠시 빼놨었는데
최종적으로 lateinit 이게 문제가 없는 부분이라 다시 복구시키는걸 깜빡쓰했네요

tgyuuAn · 2024-03-12T07:46:54Z

...re/camera/src/main/java/com/android/mediproject/feature/camera/MedicinesDetectorViewModel.kt

-    private val _captureState = MutableSharedFlow<InferenceState<CapturedDetectionEntity>>(replay = 1, extraBufferCapacity = 1)
-    val captureState get() = _captureState.asSharedFlow()
+    private val _captureState = MutableSharedFlow<InferenceState<CapturedDetectionEntity>>(replay = 1, extraBufferCapacity = 0)
+    val captureState: SharedFlow<InferenceState<CapturedDetectionEntity>> = _captureState

    private val vibrateDuration = 50L


이런 상수는 대문자로 처리 했어야 했는데 너무 뒤늦게 발견했네요..

이거 범인누구죠 문제네요
상수화 하겠습니다 ㅋㅋㅋ

pknujsp added 30 commits August 18, 2023 00:48

#214 댓글 탭 띄워질 때 스크롤 고정되도록 기능 추가

40e83bb

#214 댓글 입력 화면 레이아웃 수정, 입력 화면 다이얼로그로 나타나게 변경

23d8593

#214 답글 대상 댓글 정보를 답글 입력 다이얼로그에서 표시되도록 수정, 약 상세 정보 화면에서 상단 툴바 색상이 제대로 …

aadb372

…변경되지 않는 문제 수정

#212 약 실시간 검출 시 라벨과 확률을 boundingbox와 함께 그리도록 개선

ed9c227

#182 검출/분류 로직 수정, 불필요한 Compose 의존성 제거하기 위해 모듈 설정 변경, 라이브러리 최신 버전 업데이트…

e6671d1

…, k2컴파일러 적용

갑자기 발생한 알수없는 빌드 오류로 수정 후 덮어씌움

53ceaa8

#219 AwsNetworkApi.kt 내 코드를 기존 API 연결코드에서 aws api연결을 위한 코드로 수정, 댓글 ge…

ce3ce28

…t,post 메서드 수정

#219 CommentsDataSource.kt내에서 댓글 가져오는 메서드 인자를 새로운 api에 맞게 수정

b3870e1

#219 TokenRepository.kt 삭제하고 연관된 코드 삭제 작업

783b1cb

#219 feature:aws모듈에 있던 모든 파일 core:network모듈 내 sign패키지로 이동, SignDataSo…

c0d80b5

…urce를 aws에 맞게 수정

#219 aws api에 맞게 SignRepositoryImpl 코드 전부 수정

702470c

#219 TokenRepository.kt 를 생성하여 SignRepositoryImpl에 구현하도록 함, 현재 로그인 세션…

2ae4679

… 정보(토큰)를 가지고 있는 레포지토리

#219 의약품 정보에서 댓글 화면에 연결된 MedicineCommentsViewModel내 코드를 수정한 레포지토리에 맞게 변경

895d963

#219 로그인/회원가입 뷰모델 로직 새로운 api에 맞게 수정, 가입 이메일 검증 로직 VerifyEmail.kt추가

da0254c

#219 VerifyEmail 완성

63c088e

#219 이메일 검증 로직을 SignRepository에서 처리하도록 함, 로그인/가입 응답 객체 변경

c1d8270

#219 UserInfoRepositoryImpl에서 @Inject를 불필요하게 사용하여 제거

24a12e2

#219 TokenRepository -> AccountSessionRepository 이름 변경, 로그인 상태 체크 필드 추가

db48a95

#219 SignInState -> LoginState 이름 변경

b1754d7

#219 UserInfoRepository.kt 삭제

163e062

#219 SignRepositoryImpl에서 AccountSessionRepository 분리

5f11a9a

#219 RepositoryModule 정리 : SignRepositoryImpl대신 SignRepository를 노출시키게 변경

392aa47

#219 UserInfoDataSource 관련 클래스 삭제, UserDataSourceImpl 불필요한 주석 제거

c8c8c93

#219 AppDataStore내 계정ID 관련 필드 삭제 ServerNetwork 모듈 내 쓰이지 않는 인스턴스 제공자 제거

e5146c2

#219 UserRepository내 메서드 반환 타입 flow제거, 구현 메서드 공란으로 변경

c162168

#219 의약품 댓글 처리 MedicineCommentsViewModel에서 발생하는 오류 코드 모두 주석처리

4121a73

#219 임시조치 : 로그인, 회원가입과 직접 연결되지 않는 클래스에서 발생하는 오류 코드 모두 주석처리

0a93b2d

#219 이메일 인증 요청 다이얼로그 함수 추가

2232597

#219 이메일 인증 요청 뷰모델 추가, 요청 다이얼로그 레이아웃 추가

5d462e9

#219 SignRepositoryImpl에서 이메일 인증 로직 구현하도록 수정

4229089

pknujsp added 10 commits March 11, 2024 18:47

#219 이메일 인증 다이얼로그 함수에서 프래그먼트로 변경

9a9b223

#219 이메일 인증 다이얼로그 작업 완료

f1b3adc

#219 SignDataSource 에서 로그인, 회원가입 기능을 클래스로 분리

a94249a

#219 수정한 DataSource에 맞게 SignRepository 수정

221d45c

#219 feature:intro 모듈 내 로그인, 회원가입 클래스 새로운 패키지로 이동, 회원가입 시 닉네임 속성 이름 오…

170a2a5

…류로 가입이 되지 않는 문제 수정

#219 로그인 UI 상태 클래스 LoginViewModel에서 따로 뺌, 로그인 테스트 성공

5a92d9a

#219 AccountSessionRepository에서 현재 연결된 계정 정보와 세션을 StateFlow로 관리하도록 함

52e07cb

#219 앱 프로세스를 다시 시작하였을 떄 세션정보를 다시 로드하는 로직을 구현할 loadSession()을 AccountS…

1bca783

…essionRepository에 추가

#219 사용자 상세 정보(CognitoUserDetails)를 로그인시에 가져오도록 수정

ad97ca5

#219 사용자 상세 정보(CognitoUserDetails) 로드 테스트 성공

cb5c9e7

pknujsp added 박준성 안드로이드 백엔드 PR작성중 labels Mar 11, 2024

pknujsp self-assigned this Mar 11, 2024

pknujsp requested a review from tgyuuAn March 11, 2024 16:40

pknujsp commented Mar 12, 2024

View reviewed changes

tgyuuAn reviewed Mar 12, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#219/feature/jsp/backend #222

#219/feature/jsp/backend #222

pknujsp commented Mar 11, 2024 •

edited

Loading

tgyuuAn commented Mar 12, 2024

pknujsp Mar 12, 2024

pknujsp Mar 12, 2024 •

edited

Loading

pknujsp Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn left a comment

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 13, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 13, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 13, 2024

pknujsp Mar 13, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024 •

edited

Loading

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024 •

edited

Loading

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

tgyuuAn Mar 12, 2024

pknujsp Mar 12, 2024

#219/feature/jsp/backend #222

Are you sure you want to change the base?

#219/feature/jsp/backend #222

Conversation

pknujsp commented Mar 11, 2024 • edited Loading

📌 관련 이슈

✨ 과제 내용

파일 변경이 엄청난데 Ai작업할 때 이리저리 테스트하면서 쓴 나중에 제거할 코드가 많아서 따로 정리할 로그인, 회원가입 부분 파일들만 보시면 될거 같습니다.

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

tgyuuAn commented Mar 12, 2024

Choose a reason for hiding this comment

pknujsp Mar 12, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tgyuuAn left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

실제로 어떻게 쓰이고 있는지 코드 개선점 고민 해볼겸 작성해봤습니다.

코드를 읽어보니까 더 개선할 수 있는 코드가 다수인 거 같습니다. 기능이 동작하니 지금 당장 수정은 하지 않는게 좋을거 같고 다음에 MVP단계에서 개선해보겠습니다.

Choose a reason for hiding this comment

실제로 어떻게 쓰이고 있는지 코드 개선점 고민 해볼겸 작성해봤습니다.

코드를 읽어보니까 더 개선할 수 있는 코드가 다수인 거 같습니다. 기능이 동작하니 지금 당장 수정은 하지 않는게 좋을거 같고 다음에 MVP단계에서 개선해보겠습니다.

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pknujsp Mar 12, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pknujsp Mar 12, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pknujsp commented Mar 11, 2024 •

edited

Loading

pknujsp Mar 12, 2024 •

edited

Loading

pknujsp Mar 12, 2024 •

edited

Loading

pknujsp Mar 12, 2024 •

edited

Loading