Feature/1st assignment #5

YoonAh-dev · 2021-10-10T07:24:24Z

📌 관련 이슈

close #1
close #2
close #3
close #4

📌 변경 사항 및 이유

1주차 과제를 구현했습니다.

📌 PR Point

iOS 15.0를 반영해서 구현했습니다. 괜찮은지 봐주세요!!

📌 참고 사항

없습니다~

pcsoyeon

LGTM 💖

pcsoyeon · 2021-10-10T07:26:11Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/SignupVC.swift

@@ -104,6 +98,27 @@ class SignupVC: UIViewController {
        passwordTextfield.delegate = self
    }

+    private func setupButtonState(_ button: UIButton) {
+        if #available(iOS 15.0, *) {


오호 .. 이 부분인가요 ..

15.0 정말 징하네요..🥲

loinsir

첫 코드 리뷰 남겨봅니다 ㅎㅎ 별로 도움이 안될지라도 잘부탁 드려여

loinsir · 2021-10-10T10:55:16Z

Youtube-Clone/Youtube-Clone/Sources/Supports/AppDelegate.swift

+        let navi = UINavigationController.init(rootViewController: LoginVC())
+        window.rootViewController = navi
+        window.makeKeyAndVisible()
+        window.backgroundColor = .white


여기서 window의 backgroundColor를 white로 지정해 준 이유가 있나요? systemBackground는 안될까요...?

고거 나도 좀 궁금합니다.. 아릐까릐..😵‍💫😵‍💫

여기서 window의 backgroundColor를 white로 지정해 준 이유가 있나요? systemBackground는 안될까요...?

systemBackground도 상관없습니다!
저 부분은 View의 진입점이 되는 rootViewController인 navi의 배경색을 지정해주는 부분인데 어차피 이번 과제에서 다크모드를 구현할 필요도 없고 그래서 white로 배경색을 지정해준거예요.
만약에 다크모드를 구현해야한다면 systemBackground로 하는 게 맞지만 그게 아니라면 white나 systemBackground나 상관이 없어서 white로 색을 지정해줬습니다!

loinsir · 2021-10-10T10:57:29Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/LoginVC.swift

+        logoLabel.snp.makeConstraints {
+            $0.centerX.equalToSuperview()
+            $0.top.equalTo(view.safeAreaLayoutGuide).inset(30)
+        }
+        titleLabel.snp.makeConstraints {
+            $0.centerX.equalToSuperview()
+            $0.top.equalTo(logoLabel.snp.bottom).offset(15)
+        }
+        infoLabel.snp.makeConstraints {
+            $0.top.equalTo(titleLabel.snp.bottom).offset(40)
+            $0.leading.trailing.equalToSuperview().inset(20)
+        }
+        loginStackView.snp.makeConstraints {
+            $0.top.equalTo(infoLabel.snp.bottom).offset(40)
+            $0.leading.trailing.equalToSuperview().inset(30)
+        }
+        signupButton.snp.makeConstraints {
+            $0.top.equalTo(loginStackView.snp.bottom).offset(75)
+            $0.leading.equalToSuperview().inset(30)
+        }
+        signinButton.snp.makeConstraints {
+            $0.top.equalTo(loginStackView.snp.bottom).offset(70)
+            $0.trailing.equalToSuperview().inset(30)
+            $0.width.equalTo(70)
+            $0.height.equalTo(35)


snapKit을 사용하면 이런식으로 깔끔하게 코드로 오토레이아웃을 잡을 수 있군요?! 저도 다음 과제에서 써먹어 보겠습니다!

깔끔하게 오토를 잡을 수 있지만 스토리보드가 있다면 스토리보드에서 잡아주는 게 제일 좋습니다!
저는 스토리보드를 사용하지 않아서 코드로 레이아웃을 직접 구현해준겁니다!
있다면 스보에서 하는게 제일 베스트입니다!!👍

loinsir · 2021-10-10T11:11:08Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/SignupVC.swift

+        if let isNameEmpty = nameTextfield.text?.isEmpty,
+           let isAccountEmpty = accountTextfield.text?.isEmpty,
+           let isPasswordEmpty = passwordTextfield.text?.isEmpty {
+            if isNameEmpty || isAccountEmpty || isPasswordEmpty {
+                signinButton.isEnabled = false
+                signinButton.backgroundColor = .lightGray


Suggested change

if let isNameEmpty = nameTextfield.text?.isEmpty,

let isAccountEmpty = accountTextfield.text?.isEmpty,

let isPasswordEmpty = passwordTextfield.text?.isEmpty {

if isNameEmpty || isAccountEmpty || isPasswordEmpty {

signinButton.isEnabled = false

signinButton.backgroundColor = .lightGray

if nameTextfield.hasText &&

accountTextfield.hasText &&

passwordTextfield.hasText {

signinButton.isEnabled = false

signinButton.backgroundColor = .lightGray

UITextfield.text?.isEmpty 보다는 UITextfield.hasText 속성을 사용하는 건 어떨까요???

요런 지적 너무 좋습니다👍 반영해서 머지할게요!

loinsir · 2021-10-10T11:17:13Z

Youtube-Clone/Youtube-Clone/Sources/Extensions/UIView+.swift

+    func addSubviews(_ views: [UIView]) {
+        views.forEach { self.addSubview($0) }
+    }


forEach로 한번에 추가하는 로직을 extension을 통해 구현하셨네요 기발합니다.👍 저도 extension 잘 써먹어 볼게요!

loinsir · 2021-10-10T11:19:32Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/SignupVC.swift

+    private let nameTextfield = LoginTextField(placeholder: "이름을 입력해주세요")
+    private let accountTextfield = LoginTextField(placeholder: "이메일 또는 휴대전화")
+    private let passwordTextfield = LoginTextField(placeholder: "비밀번호 입력", isSecureTextEntry: true)


textfield를 커스텀 하신게 인상깊어요😁

loinsir · 2021-10-10T11:21:50Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/LoginVC.swift

+    private let logoLabel = UILabel().then {
+        $0.text = "Google"
+        $0.font = .boldSystemFont(ofSize: 40)
+        $0.textColor = .black
+    }


then 라이브러리 말로만 들었었는데 상당히 코드가 간결해지네요. 다음 과제 부터 적용해보겠습니다!

인환선배야 중초보 스터디 때도 말했지만 라이브러리를 쓰지 않고 그냥 구현해보는 것을 추천해!
그리고 나서 라이브러리를 써도 충분해!
나는 손배의 골드그래스파가 아니지만... 읽어보면 좋을 거 같아서
https://brunch.co.kr/@advisor/11

@heerucan 좋은 지적 고마우~

seondal

같은 과제를 했는데.. 전혀 다른 코드의 수준을 보며 정말 놀랐어요.... 윤아님 코드 통해서 스토리보드 없이 생코드로 개발한걸 직접 보면서 정말 많은 걸 배워갑니다.. 존경해요ㅠㅇㅠ

seondal · 2021-10-10T14:33:35Z

README.md

+|`1주차`| iOS 컴포넌트 이해, Xcode 기본 사용법, View 화면 전환| [feature/1st-Assignment](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/tree/feature/1st-Assignment) |   | [💐Basic](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/blob/feature/second-seminar/README/Assignment1-Basic.md) <br/> [✈️Advanced](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/blob/feature/second-seminar/README/Assignment1-Advanced.md) |
 |`2주차`| AutoLayout, StackView, TabBarController | | | |


이 깃허브 링크는 어디에 연결되는건지 궁금해요..!

그러게 저두 궁금한데 혹싀.. 노션링크 좀 전부 다.. 주시게써여?🥶🥶

제가 만든 README파일에 작성해둔 과제 리드미를 저장해두고 있습니다!
그곳으로 이동해주는 링크입니다!!

seondal · 2021-10-10T14:34:23Z

README/Assignment1-Advanced.md

@@ -1 +1,5 @@
 ## 1주차 - iOS Assignment(도전과제)
+
+💐 [1주차 과제 - Advanced 노션 정리 >>>>>](https://yxxnaxxin.notion.site/1-bc06e54b37c44fada320440dd908759e)


노션이랑 리드미 이렇게 예쁘게 정리하시는거 본받고 싶어요..

seondal · 2021-10-10T14:37:50Z

Youtube-Clone/Youtube-Clone/Resources/Assets.xcassets/AccentColor.colorset/Contents.json

+{
+  "colors" : [
+    {
+      "idiom" : "universal"
+    }
+  ],
+  "info" : {
+    "author" : "xcode",
+    "version" : 1
+  }
+}


스토리보드를 사용하지 않고 생 코드로만 작성하시는 이유가 따로 있으실까요?!

코드로만 작성하는 이유가 있다면

변경 사항이 생겼을 때 스토리보드보다 수정하기 좋아요!

UI속성들을 보기 위해서 스토리보드로 왔다갔다 하는 게 싫어요..🥲

storyboard, xib까지 생기면서 Resource 파일 자체의 크기가 커지는 게 싫어요..🥲

복잡한 레이아웃에 경우에는 스토리보드와 코드를 모두 써서 레이아웃을 잡는데 "그럴바에는 코드로만 짜지" 라고 생각이 듭니다!

저스트 저의 생각입니다..🙄
스토리보드도, 코드도 다 할 줄 알아야하고 뭐 하나 안 좋은 게 아니니 스보 많이 많이 사랑해주세요.. 🙌

윤아언니의 말에 덧붙여서 스보/코드 팀의 협업 스타일, 서비스 규모, 뷰의 구성 등등 상황에 따라 적재적소에 사용하는 것이 좋다고 생각합니다 ~ 뭐든 .. no 맹신 .. It's up to you ..

heerucan

듀나레이디 덕에 내 잔디는 오늘도 쑥쑥 자라나.. 그나저나 여전히 손배 코드는 갈곰갈곰하네..
나레이디 우리 캐치미줌보걸즈-듀나걸한테는 항시 발도장을 찍고 가~^^*

heerucan · 2021-10-10T17:14:18Z

README.md

+|`1주차`| iOS 컴포넌트 이해, Xcode 기본 사용법, View 화면 전환| [feature/1st-Assignment](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/tree/feature/1st-Assignment) |   | [💐Basic](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/blob/feature/second-seminar/README/Assignment1-Basic.md) <br/> [✈️Advanced](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/blob/feature/second-seminar/README/Assignment1-Advanced.md) |
 |`2주차`| AutoLayout, StackView, TabBarController | | | |


그러게 저두 궁금한데 혹싀.. 노션링크 좀 전부 다.. 주시게써여?🥶🥶

heerucan · 2021-10-10T17:16:07Z

Youtube-Clone/Youtube-Clone/Sources/Extensions/UIColor+.swift

+    }
+}
+
+internal extension UIColor {


이 internal 키워드는 모란 말야!
해당 모듈 외부에서는 사용되지 않게 하기 위해서 붙여줬군요.
나의 듀나걸 그녀는 제법 쩜 멋져..😵‍💫🥶😊

heerucan · 2021-10-10T17:16:25Z

Youtube-Clone/Youtube-Clone/Sources/Extensions/UITextField+.swift

+        self.leftViewMode = .always
+    }
+
+    func setRightPaddingPoints(_ amount:CGFloat) {


right까지 챙겨주다니 세심걸

heerucan · 2021-10-10T17:16:55Z

Youtube-Clone/Youtube-Clone/Sources/Extensions/UIView+.swift

+    func constraint(_ anchor: NSLayoutDimension, constant: CGFloat) {
+        self.translatesAutoresizingMaskIntoConstraints = false
+        anchor.constraint(equalToConstant: constant).isActive = true
+    }


손배 스냅킷을 썼는데 이거는 왜 해준 것이야?? 요 아이는 왜??

손배 스냅킷을 썼는데 이거는 왜 해준 것이야?? 요 아이는 왜??

Component 내부에서 높이를 잡아주려고 했는데 굳이 높이를 위해서 SnapKit import해서 쓰고 싶지 않아가즤고
constraint(heightAnchor, constant: 100) 요렇게만 쓰면 높이가 지정되는 함수를 굳이 Extension했섬니다.

오 명쾌한 설명, 이해가 퐉 되었따.

heerucan · 2021-10-10T17:17:38Z

Youtube-Clone/Youtube-Clone/Sources/Supports/AppDelegate.swift

+        let navi = UINavigationController.init(rootViewController: LoginVC())
+        window.rootViewController = navi
+        window.makeKeyAndVisible()
+        window.backgroundColor = .white


고거 나도 좀 궁금합니다.. 아릐까릐..😵‍💫😵‍💫

heerucan · 2021-10-10T17:24:59Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/LoginVC.swift

+    private let logoLabel = UILabel().then {
+        $0.text = "Google"
+        $0.font = .boldSystemFont(ofSize: 40)
+        $0.textColor = .black
+    }


인환선배야 중초보 스터디 때도 말했지만 라이브러리를 쓰지 않고 그냥 구현해보는 것을 추천해!
그리고 나서 라이브러리를 써도 충분해!
나는 손배의 골드그래스파가 아니지만... 읽어보면 좋을 거 같아서
https://brunch.co.kr/@advisor/11

heerucan · 2021-10-10T17:26:28Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/LoginVC.swift

+        $0.textColor = .black
+    }
+    private let infoLabel = UILabel().then {
+        $0.text = "Continue to YouTube. You'll also sign in to Google services in your apps & Safari."


영어로 써준 거 좀. . ganzi.hoonzi.girl

heerucan · 2021-10-10T17:28:47Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/SignupVC.swift

+    private lazy var showPasswordButton = UIButton().then {
+        setupButtonState($0)
+        $0.addTarget(self, action: #selector(touchUpShowPassword), for: .touchUpInside)
+    }
+    private lazy var loginStackView = UIStackView().then {
+        $0.alignment = .fill
+        $0.axis = .vertical
+        $0.distribution = .fillEqually
+        $0.spacing = 15
+        $0.addArrangedSubview(nameTextfield)
+        $0.addArrangedSubview(accountTextfield)
+        $0.addArrangedSubview(passwordTextfield)
+    }


이거 private lazy var 써준 이유가 무엇인가영..?⁉️🤷‍♀️

우선 private은 해당 파일의 extension 안에서까지만 불러올 수 있고,
lazy var는 불러오기 전까지 초기화가 되지 않는다. 이 정도까지 알고 있습니다릥.
필요할 때 너 생성시킨다. 흠 button에 붙이는 것은 약간 알 것 같은데 stackView는 모르겠다릐..

우선 private은 해당 파일의 extension 안에서까지만 불러올 수 있고, lazy var는 불러오기 전까지 초기화가 되지 않는다. 이 정도까지 알고 있습니다릥. 필요할 때 너 생성시킨다. 흠 button에 붙이는 것은 약간 알 것 같은데 stackView는 모르겠다릐..

lazy var가 알다시피 모든 뷰들이 다 초기화하고 나서 불러지는 변수인데, 저 두 UI속성에서 필요했던 이유는

showPasswordButton

setupButtonState라는 함수를 사용해야 하는데 사용하려면 저 함수가 초기화된 후에 사용해야하기 때문에 lazy var를 붙여줬습니다.

loginStackView

addArrangedSubview를 사용하기 위해서 then closure 외부에 있는 textfield를 가져와야 합니다. 하지만 textfield들이 var를 쓸 경우에는 아직 초기화되어 있지 않아서 addArrangedSubview에 존재하지 않는 textfield가 들어가서 문제를 일으킵니다. 따라서 lazy var를 사용해서 초기화가 완료된 textfield들이 들어갈 수 있게끔 해준거예요!

heerucan · 2021-10-10T17:31:40Z

Youtube-Clone/Youtube-Clone/Sources/ViewControllers/SignupVC.swift

+
+    private func setupButtonState(_ button: UIButton) {
+        if #available(iOS 15.0, *) {
+            var configuration = UIButton.Configuration.plain()


그래 나레이디가 손배 코드보고 이 부분을 안에 넣어서 써줄 수 있구나.. 깨달았어.
리팩gg~!
근데 15에만 대응해준 거.. 분기처리 그녀는 제법.. 14.7을 쓰는 나에게도 서윗하네..🐝🐝

14.7 후리에게도 image와 font가 적용될 수 있게 쩜 세심한 배려를 해줬즤?

heerucan · 2021-10-10T17:32:46Z

Youtube-Clone/Youtube-Clone/Sources/Views/LoginTextField.swift

+
+import UIKit
+
+class LoginTextField: UITextField {


이렇게 자유자재로 class 만들어서 사용하는 손배야가 난 부럽고 멋져.. 👏 👍 😊 ☺️

이거 인정.합니다. 듀나걸을.당장.우리집으로.

YoonAh-dev added 27 commits October 4, 2021 01:48

➕[ADD] 1차 과제 프로젝트 생성(#1)

4638a4a

✅[CHORE] Storyboard 삭제(#1)

9fafdd0

➕[ADD] Podfile 추가(#1)

8b5588b

➕[ADD] Podfile install(#1)

33ca00a

➕[ADD] Color 추가(#2)

3c75f55

➕[ADD] LoginView 구현 완료(#2)

37884d3

➕[ADD] 회원가입 뷰 완료(#2)

3c6e806

➕[ADD] CheckVC 화면 구현 완료(#2)

e8dc94b

✨[FEAT] 전체 화면 전환 구현 완료(#3)

7cd867c

✨[FEAT] 회원가입 뷰에서 화면 전환 구현(#3)

d176ce3

✨[FEAT] textfield에 적은 이름 화면에 띄우기 구현(#3)

2a04537

✨[FEAT] 로그인 화면 Button Enable 구현(#4)

b8868f6

✨[FEAT] 회원가입 로직 구현 완료(#4)

86b047d

✨[FEAT] 비밀번호 표시 구현(#4)

dd418ae

➕[ADD] Mark 추가

ddfc4e1

📝[DOCS] Update readme

026a297

📝[DOCS] 1주차 Basic Notion 첨부

0a1416d

📝[DOCS] 1주차 도전 과제 Notion 첨부

0b0e381

📝[DOCS] Update Link

e853376

📝[DOCS] Update Link

452be93

✅[CHORE] 15.0 이외의 버전에서도 가능하게 수정

9416878

🔀[MERGE] Merge Pull

f5e0738

➕[ADD] 두번째 세미나 추가

01a6dc5

🚚[MOVE] 세미나 파일 브랜치 이동

f1238bc

📝[DOCS] Update readme

b2def8f

📝[DOCS] Update readme

3fcb77e

✏️ [CORRECT] button 이름 변경

8dbf64a

YoonAh-dev added 💐 듀나 Duna's work 🌈 과제 Assignment 1️⃣ 차 First Assignment labels Oct 10, 2021

YoonAh-dev added this to the 1차 과제 구현 milestone Oct 10, 2021

YoonAh-dev requested review from loinsir, pcsoyeon and seondal October 10, 2021 07:24

YoonAh-dev self-assigned this Oct 10, 2021

pcsoyeon reviewed Oct 10, 2021

View reviewed changes

loinsir approved these changes Oct 10, 2021

View reviewed changes

seondal reviewed Oct 10, 2021

View reviewed changes

heerucan reviewed Oct 10, 2021

View reviewed changes

✅[CHORE] hasText로 코드 변경(#5)

8e89614

YoonAh-dev merged commit 3a9de82 into main Oct 11, 2021

		\|`1주차`\| iOS 컴포넌트 이해, Xcode 기본 사용법, View 화면 전환\| [feature/1st-Assignment](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/tree/feature/1st-Assignment) \| \| [💐Basic](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/blob/feature/second-seminar/README/Assignment1-Basic.md) <br/> [✈️Advanced](https://github.com/29th-WE-SOPT-iOS-Part/ShinYunA/blob/feature/second-seminar/README/Assignment1-Advanced.md) \|
		\|`2주차`\| AutoLayout, StackView, TabBarController \| \| \| \|

		@@ -1 +1,5 @@
		## 1주차 - iOS Assignment(도전과제)

		💐 [1주차 과제 - Advanced 노션 정리 >>>>>](https://yxxnaxxin.notion.site/1-bc06e54b37c44fada320440dd908759e)

Feature/1st assignment #5

Feature/1st assignment #5

Conversation

YoonAh-dev commented Oct 10, 2021

📌 관련 이슈

📌 변경 사항 및 이유

📌 PR Point

📌 참고 사항

pcsoyeon left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

loinsir 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

seondal 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

heerucan 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

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