Week 1 assignment #6

dev-jungbin · 2022-04-10T03:42:44Z

🌱 작업한 내용

기본 과제

코드베이스 프로젝트 세팅
UI 구성
- 필요한 Extension 추가
- 재사용 컴포넌트 추가
  - DefaultTextField
  - BlueBtnWithTextField
- BaseVC, BaseNC 추가
- 로그인, 회원가입(두 개의 뷰 재사용), 완료 화면 구성
화면 전환
데이터 전달

🌱 PR Point

두 번 이상 쓰이는 부분(TextField, Blue Button, 회원가입 1, 2 화면)을 재사용하는 식으로 구성
❓ 근데... 코드로 Button 올리니까 터치했을 때 버튼이 눌리는 듯한 효과(?)가 안 나와여... 이거 어케 하ㅈㅕ...

📸 스크린샷

구현 내용	스크린샷
기본 과제

📮 관련 이슈

Resolved [FEAT] iOS 1차 과제 - 기본 #2

hwangJi-dev

정정빈천재

hwangJi-dev · 2022-04-10T04:33:56Z

Clone-Instagram/Clone-Instagram/Scene/Welcome/WelcomeVC.swift

+        titleLabel.snp.makeConstraints {
+            $0.top.equalTo(view.safeAreaLayoutGuide.snp.top).offset(270)
+            $0.leading.equalTo(view.safeAreaLayoutGuide.snp.leading).offset(20)
+            $0.trailing.equalTo(view.safeAreaLayoutGuide.snp.trailing).offset(-20)


요거 offset말고 inset으로 주면 inset(20)하면 된다~? 참고행!!

@hwangJi-dev 아!!!!!! 황지은은 천잰가?? 오케바리!

@hwangJi-dev 내 피알도 놀러와줘 선배~~

wngus4296

나도선배 아요가 쥐리는 이유가 여기 있었네요 ;;
은주선배가 마크 구문 추천해줬는데 여기서 보네여.. 저도 얼른 적용해보겠습니다 ~ 😎

여기도 내일 다시 찾아올게요

jane1choi

정빈걸...너는 그냥 천재야...
이런식으로하면 어? 그냥 앱스토어 정복밖에 못한다니까??
코베로 깔끔하게 레이아웃도 잘잡아줬당 굿굿👍🏻
담주에 inset 활용해서 레이아웃 야무지게 잡아보자😆🥰
수고했어🔥🔥

jane1choi · 2022-04-10T10:45:19Z

Clone-Instagram/Clone-Instagram/Extension/UITextField+.swift

+import UIKit
+
+extension UITextField {
+


그냥 파일자체가 저랑 똑같네요^__^
그냥 지은키드들이네요^^

jane1choi · 2022-04-10T10:46:25Z

Clone-Instagram/Clone-Instagram/Scene/SignIn/SignInVC.swift

+        $0.contentMode = .scaleAspectFit
+    }
+    private let userNameTextField = DefaultTextField().then {
+        $0.placeholder = "전화번호, 사용자 이름 또는 이메일"


디폴트 스타일 적용한 Custom Class 만들어서 쓰는거 좋네요👍🏻👍🏻

jane1choi · 2022-04-10T10:48:10Z

Clone-Instagram/Clone-Instagram/Scene/SignIn/SignInVC.swift

+    private func setTapSignUpBtn() {
+        signUpBtn.press {
+            let signUpUserNameVC = SignUpVC()
+            signUpUserNameVC.signUpViewType = .makeUserName


enum 활용 짱이다 너....
역시 nadoAlert의 창시자..

jane1choi · 2022-04-10T10:52:28Z

Clone-Instagram/Clone-Instagram/Scene/SignUp/SignUpVC.swift

+                welcomeVC.userName = self.userName
+                self.present(welcomeVC, animated: true)
+                self.navigationController?.popToRootViewController(animated: true)
+            }


와 이거 진짜 미친거 아닐까?? 어어???
이렇게 야물딱+효율적으로 케이스 나눠서 화면전환을 해준다고??
진짜 대박인데??어??? 대박대박..
특히 코드베이스 기반에서 똑같은 구조의 뷰 두개 있는 상황이니까 진짜 효율적인 화면전환 방식인거 같애!!🔥

와 인정정빈.. 천잰가... 나도 쇽샥 해도 될까 혹시 ~~~~~ 쩐다 쩐다

@Suyeon9911 쇽샥해용🌟💫

jane1choi · 2022-04-10T10:52:56Z

Clone-Instagram/Clone-Instagram/Scene/SignUp/SignUpVC.swift

+            titleLabel.text = "비밀번호 만들기"
+            contentLabel.text = "비밀번호를 저장할 수 있으므로 iCloud 기기에서 로그인 정보를 입력하지 않아도 됩니다."
+            textField.placeholder = "비밀번호"
+        }


천재다 정정빈..

noah0316

나도선배 최고~~!!!
정빈선배 최고~~!!!!👍

noah0316 · 2022-04-11T10:04:56Z

Clone-Instagram/Clone-Instagram/Base/BaseVC.swift

+
+import UIKit
+
+class BaseVC: UIViewController {


BaseVC 상속받아서 쓰는 방식 좋네요~~!!

noah0316 · 2022-04-11T10:07:11Z

Clone-Instagram/Clone-Instagram/Scene/SignIn/SignInVC.swift

+import Then
+import SnapKit
+
+class SignInVC: BaseVC {


이 친구의 서브클래스가 필요하지 않으면 앞에 final 키워드를 붙이는건 어떨까요?
컴파일러가 이 키워드를 보고 static dispatch를 한다면 성능이 향상 될 것 같습니다~!!

@noah0316 오 좋은 제안 감사요!!!!!! 도전과제 전에 반영하겠슴!!!!!!!

noah0316 · 2022-04-11T10:08:41Z

Clone-Instagram/Clone-Instagram/Scene/SignIn/SignInVC.swift

+}
+
+// MARK: - UI
+extension SignInVC {


UI 관련된 부분은 extension으로 따로 빼주시니 분리가 잘되어있어 보기 편하네요.. 짱짱~!!👍👍

noah0316 · 2022-04-11T10:09:21Z

Clone-Instagram/Clone-Instagram/Scene/SignIn/SignInVC.swift

+            $0.top.equalTo(view.safeAreaLayoutGuide.snp.top).offset(80)
+            $0.leading.equalTo(view.safeAreaLayoutGuide.snp.leading).offset(40)
+            $0.trailing.equalTo(view.safeAreaLayoutGuide.snp.trailing).offset(-40)
+            $0.height.equalTo(80)


순서 통일성 있는거 편안하네요...

noah0316 · 2022-04-11T10:11:07Z

Clone-Instagram/Clone-Instagram/Extension/UITextField+.swift

+    var isEmpty: Bool {
+        if text?.isEmpty ?? true {
+            return true
+        }
+        return false
+    }


UITextField의 hasText를 사용하면 이 친구는 안써도 되지 않나요?! 궁금합니다~!!🧐

@dev-madilyn 오 그런 메소드가 있는 줄 몰랐음!!~~~~!!! 땡큐!

noah0316 · 2022-04-11T12:27:03Z

Clone-Instagram/Clone-Instagram/Scene/SignIn/SignInVC.swift

+    }
+
+    private func setTapSignInBtn() {
+        signInBtn.press {


press method가 escaping closure로 되고, 클로져를 프로퍼티에 저장하고 있어서
press 메소드를 사용할 때 [weak self]를 넣어주지 않으면 이곳에서 메모리 누수가 발생할 것 같네요~!!

@noah0316 혹시 인간 윜셀프 이신지.. 여기저기 위크셀프 이동강연 다니시네요 ㅋㅋ

@noah0316 오 확인여 ㅋㅋㅋ 인간 메모리세이버 이시네요...

dev-jungbin added 21 commits April 7, 2022 23:55

Create: Clone-Instagram project

9956695

Chore: 코드베이스 프로젝트 세팅

1736ef4

Add: SnapKit, Then SPM 추가

bcf0c5b

Add: BaseVC, SignInVC

fd9fb4b

Add: Instagram 로고 에셋

50d6170

Move: Scene/SignInVC

19e506c

Add: UIView, UITextField, UIButton Extension

90bdd05

Add: Custom Textfield - DefaultTextField

b09ef3b

Add: Custom Button - BlueBtnWithText

b3e40a9

Feat: SignInVC UI

ad1fcf1

Add: SignUpVC

d6e6169

Chore: Foldering - SignIn / SignUp

037978c

Chore: set root view controller to navigation controller

4db2dcf

Add: BaseNC

ae75572

Chore: BaseNC 활용하여 SceneDelegate 수정

a1e9fc1

Add: BaseVC에 hideNavigationBar, pop 함수 추가

f4a6151

Chore: SignInVC에 회원가입 버튼 액션 추가, Navi bar 숨기는 코드 추가

3321cf2

Feat: Design SignUpVC

7050274

Chore: 회원가입 뷰 type 나누어 값 세팅

e09e270

Feat: Design WelcomeVC UI

5d2f414

Feat: 화면 전환 및 데이터 전달 구현

71e4e84

dev-jungbin added the 📱 assignment - basic 기본 과제 label Apr 10, 2022

dev-jungbin requested review from wngus4296, jane1choi, noah0316, yungu0010, sssua-0928 and ZaidKang April 10, 2022 03:42

dev-jungbin self-assigned this Apr 10, 2022

hwangJi-dev reviewed Apr 10, 2022

View reviewed changes

wngus4296 reviewed Apr 10, 2022

View reviewed changes

jane1choi approved these changes Apr 10, 2022

View reviewed changes

noah0316 reviewed Apr 11, 2022

View reviewed changes

dev-jungbin added 4 commits April 12, 2022 02:23

Chore: Offset 음수 코드 Inset으로 수정

d9e4e7b

Chore: final 키워드 필요한 곳에 적용

6984087

Remove: UITextField + isEmpty

d0ae7a0

Chore: [weak self]로 메모리 누수 막기

c1886dc

dev-jungbin merged commit 43fe8db into main Apr 13, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Week 1 assignment #6

Week 1 assignment #6

dev-jungbin commented Apr 10, 2022 •

edited

Loading

hwangJi-dev left a comment

hwangJi-dev Apr 10, 2022

dev-jungbin Apr 10, 2022

jane1choi Apr 10, 2022

wngus4296 left a comment •

edited

Loading

jane1choi left a comment

jane1choi Apr 10, 2022

jane1choi Apr 10, 2022

jane1choi Apr 10, 2022

jane1choi Apr 10, 2022

Suyeon9911 Apr 11, 2022

dev-jungbin Apr 11, 2022

jane1choi Apr 10, 2022

noah0316 left a comment

noah0316 Apr 11, 2022

noah0316 Apr 11, 2022

dev-jungbin Apr 11, 2022

noah0316 Apr 11, 2022

noah0316 Apr 11, 2022

noah0316 Apr 11, 2022

dev-jungbin Apr 11, 2022

noah0316 Apr 11, 2022

Suyeon9911 Apr 11, 2022

dev-jungbin Apr 11, 2022

		import UIKit

		extension UITextField {

Week 1 assignment #6

Week 1 assignment #6

Conversation

dev-jungbin commented Apr 10, 2022 • edited Loading

🌱 작업한 내용

기본 과제

🌱 PR Point

📸 스크린샷

📮 관련 이슈

hwangJi-dev 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

wngus4296 left a comment • edited Loading

Choose a reason for hiding this comment

jane1choi 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

Choose a reason for hiding this comment

noah0316 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

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

dev-jungbin commented Apr 10, 2022 •

edited

Loading

wngus4296 left a comment •

edited

Loading