[Feat] Assignment4 #8

daminoworld · 2022-05-15T14:53:25Z

🌱 작업한 내용

기본 과제

인스타그램 클론앱 내 로그인, 회원가입을 Alamofier를 사용해 API 기반 연결 구현
도전 과제 - 1
GET 통신을 이용해서 picsum 사이트의 api를 이용해 이미지 url을 담고 있는 Json 데이터 가져오기

도전과제 - 2 가져온 이미지 url을 이용해 이미지를 가져와 피드 이미지에 띄워주는건데 시간이 없어 구현하지 못했습니다..
시간 될 때 추가적으로 구현해보겠습니다ㅠ

🌱 PR Point

PR Point 1

기본 과제에 있는거는 로그인, 회원가입에 성공했을때 메시지만 알림창으로 띄워주는것이였는데
실패했을때 응답을 status 코드로 분기처리하고 메시지를 변경해 왜 실패 했는지
( ex. 중복 아이디(회원가입), 비밀번호 틀림(로그인))를 표시해 주었고
특히 회원가입에서 비밀번호까지 입력완료하고 아이디 중복으로 실패했을때
중복 알림 메시지와 함께 아이디 입력페이지로 다시 돌아가게 해서 과제 내에서 UX 경험을 개선하려고 노력했습니다^^
그러나 signUpPasswordViewController에서 분기처리를 할때
통신함수( signUp() ) 를 실행하고 하고 그 안에서 응답 코드에 따라
성공의 유무를 판단하는 Bool 타입 변수 isSignUpSuccess로 값을 변경해주어
그 값에 따라 어떻게 화면 전환을 할지 결정하려고 했는데
비동기적으로 signUp 함수가 끝나기도 전에 ( isSignUpSuccess가 바뀌기도 전에)
기존 값으로 화면전환이 이루어지는 문제가 발생해서

결과적으로 아래와 같이 아예 응답을 받는 함수 안에 분기처리로 case에 따라 화면 전환이 되도록 했는데 이런식으로 구현하는게 좀 아닌것 같아서 피드백 궁금합니다!ㅠ

📸 스크린샷

아이디 중복	회원가입 성공

아이디 불일치	로그인 성공

📮 관련 이슈

Resolved: [FEAT] iOS 4차 과제 #7

빈줄 삭제

- APIConstants, NetworkResult, LoginModel, UserService 파일 추가 및 LoginViewController에 통신 코드 추가

- 로그인, 회원가입 API 연결 - 로그인 성공 및 실패 별 알림 메시지 및 동작 변경 - 회원가입 성공 및 실패 별 알림 메시지 동작 변경

- 가입 완료시와 마찬가지로 서버 통신 함수 내부에 - response code에 따른 화면전환 처리

- Get 통신으로 Picsum API 를 통해 Json으로 데이터 가져오기 까지 구현함

devxsby

과제 제출하시느라 늦은 시각까지 고생하셨습니다. !!!

서버와 연결을 하는 것에 관해 생각을 많이 해보신게 느껴져요 👍

추가로 gitignore에 코코아팟이 없어서 관련 파일이 다 올라오는 것 같은데 추가하시면 좋을 것 같아용 😁

PR 포인트에 적은 사항도 제가 아는 부분에서 코멘트 남겼는데 해결이 될 진 모르겠으나, 계속 고민해보고 추후에 리팩토링으로 개선된 코드가 되면 더 좋을 것 같습니다~~~

이제 당근.. 하러 가시죠... 👩🏻‍🌾🥕

devxsby · 2022-05-17T09:57:38Z

iOS_assignment3/iOS_assignment3/Auth/Login/LoginViewController.swift

+        passwordTextField.isSecureTextEntry.toggle()
+        passwordEyeButton.isSelected.toggle()
+        let eyeImage = passwordEyeButton.isSelected ? "icn_password_shown" : "icn_password_hidden"
+        passwordEyeButton.setImage(UIImage(named: eyeImage), for: .normal)
+        passwordEyeButton.tintColor = .clear


passwordEyeButton.isSelected.toggle() 이 코드 말고는 다 스토리보드에서 지정할 수 있는 걸로 아는데, 코드말고 스보로 해보시는 것도 추천드립니당~~

devxsby · 2022-05-17T09:58:08Z

iOS_assignment3/iOS_assignment3/Auth/Login/LoginViewController.swift

+        guard let name = userNameTextField.text else { return  }
+        guard let email = userNameTextField.text else { return }
+        guard let password = passwordTextField.text else { return }


guard let 구문 좋네요 👍

devxsby · 2022-05-17T09:59:32Z

iOS_assignment3/iOS_assignment3/Auth/Login/LoginViewController.swift

+            case .success(let data):
+                guard let data = data as? LoginResponse else { return }
+
+                var message:String
+                // LoginResponse의 상태에 따른 메시지 분기처리
+                switch data.status {
+                case 404:
+                    message = "일치하는 아이디가 없습니다."
+                    self.isLoginSuccess = false
+                case 409:
+                    message = "비밀번호가 일치하지 않습니다."
+                    self.isLoginSuccess = false
+                default:
+                    message = data.message
+                    self.isLoginSuccess = true
+                    guard let signInCompletedVC = UIStoryboard(name: "SignInCompleted", bundle: nil).instantiateViewController(withIdentifier: "SignInCompletedViewController") as? SignInCompletedViewController else {return}
+                    signInCompletedVC.userName = self.userNameTextField.text
+
+                    self.navigationController?.pushViewController(signInCompletedVC, animated: true)


switch 구문에서 case .success 코드 안에 또 switch 구문으로 request에러 타입을 넣으신 거는 어떤 이유이실까요...? PR 포인트1에 관한 내용일까요??

success = 통신 됐을때 라고 생각하고 그냥 requestErr를 따로 구분 안하고 로그인 메시지만 구분하려고 switch구문에 같이 넣어줬습니다..근데 이것도 따로 request에러를 구분해줘야겠네요ㅎㅎ 좋은 지적 감사합니다ㅎㅎ

devxsby · 2022-05-17T10:12:11Z

iOS_assignment3/iOS_assignment3/Service/UserService.swift

+        // 성공 시에는 넘겨받은 데이터를 decode(해독)하는 함수를 호출
+        // statusCode가 200~400대 일때는 통신에 이상있는것이 아니라 둘다 isValidLoginData함수를 호출
+        case 200..<300: return isVaildLoginData(data: data)
+        case 400..<500: return isVaildLoginData(data: data)


isVaildLoginData함수는 .success(decodedData as Any)를 리턴하기 때문에,
여기 부분은 isInVaildLoginData 함수를 따로 만들어서 .requestErr(decodedData)을 리턴하도록 하는게 좋을 것 같습니다..!

private func isInVaildSignUpData(data: Data) -> NetworkResult<Any> { let decoder = JSONDecoder() guard let decodedData = try? decoder.decode(SignUpResponse.self, from: data) else { return .pathErr } return.requestErr(decodedData) }

아 네네 맞습니다 역시 수빈선배 바로 찾으셨네요ㅎㅎ 사실 처음에 .success가 통신이 됐나 안됐나라고 생각해서 success 됐을때도 그냥 isValid 함수를 실행시켜줬는데 다시 보니까 수빈님 말씀하신 것처럼 함수를 따로 만들어서 requestErr를 구분해야겠더라구요,,, 매번 친절한 지적 좋아요ㅎㅎ 시간날때 다른것까지 같이 리팩토링 해보겠습니다,,😆

daminoworld added 5 commits May 10, 2022 09:44

# [DEL] <빈줄 삭제>

433608f

빈줄 삭제

# ➕[ADD] <4차 세미나 코드 적용>

a2ed337

- APIConstants, NetworkResult, LoginModel, UserService 파일 추가 및 LoginViewController에 통신 코드 추가

# [�FEAT] <4차 기본과제> (#7)

d925b38

- 로그인, 회원가입 API 연결 - 로그인 성공 및 실패 별 알림 메시지 및 동작 변경 - 회원가입 성공 및 실패 별 알림 메시지 동작 변경

# [FIX] <로그인 알림 수정> (#7)

2556fa4

- 가입 완료시와 마찬가지로 서버 통신 함수 내부에 - response code에 따른 화면전환 처리

# [FEAT] <4차 기본, 도전-1과제> (#7)

f73f239

- Get 통신으로 Picsum API 를 통해 Json으로 데이터 가져오기 까지 구현함

daminoworld added the 4차 과제 label May 15, 2022

daminoworld requested review from wngus4296, Taehyeon-Kim, hyesuuou, devxsby and Suyeon9911 May 15, 2022 14:53

daminoworld self-assigned this May 15, 2022

devxsby approved these changes May 17, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] Assignment4 #8

[Feat] Assignment4 #8

daminoworld commented May 15, 2022 •

edited

Loading

devxsby left a comment

devxsby May 17, 2022

devxsby May 17, 2022

devxsby May 17, 2022

daminoworld May 17, 2022

devxsby May 17, 2022 •

edited

Loading

daminoworld May 17, 2022

[Feat] Assignment4 #8

Are you sure you want to change the base?

[Feat] Assignment4 #8

Conversation

daminoworld commented May 15, 2022 • edited Loading

🌱 작업한 내용

🌱 PR Point

📸 스크린샷

📮 관련 이슈

devxsby left a comment

Choose a reason for hiding this comment

devxsby May 17, 2022

Choose a reason for hiding this comment

devxsby May 17, 2022

Choose a reason for hiding this comment

devxsby May 17, 2022

Choose a reason for hiding this comment

daminoworld May 17, 2022

Choose a reason for hiding this comment

devxsby May 17, 2022 • edited Loading

Choose a reason for hiding this comment

daminoworld May 17, 2022

Choose a reason for hiding this comment

daminoworld commented May 15, 2022 •

edited

Loading

devxsby May 17, 2022 •

edited

Loading