🔀 :: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 #216

junseopark0331 · 2023-12-21T09:50:05Z

💡 개요

급식이 없을때 View에서 급식이 없습니다를 나타내는 기능 구현

📃 작업내용

급식이 없을때 View에서 급식이 없습니다를 나타내는 기능 구현했습니다.

🔀 변경사항

DotoriLabel 타입을 배열로 하는 mealsView를 생성하여 meals배열이 비어있을때
"급식이 없습니다"를 View의 윗부분으로 올리기 위해 공백과 함께 배열에 추가했습니다.

🙋‍♂️ 질문사항

mealViews.append(DotoriLabel("급식이 없습니다."))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))

이 코드 너무 안예뻐보이는데 다른 방법이 있으면 알려주세요..!

🍴 사용방법

🎸 기타

baekteun · 2023-12-21T09:57:39Z

PR 제목에 앞에 prefix로 그냥 커밋 따라가기보다는 🔀 :: [#이슈넘버] 로 하는게 좋을거같아요
PR 제목에 [이슈넘버] => [#이슈넘버] 로 바꿔야할거같아요
커밋에 [#이슈넘버] prefix 안달아주신거같아용

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift

baekteun · 2023-12-21T09:59:28Z

혹시 mac에 swiftlint 설치해놓으셨나요? lint warning을 방치한채로 코드가 작성된거같아서 물어봐요

baekteun · 2023-12-21T10:07:03Z

혹시 make setup 명령어 안치셨나용?

baekteun · 2023-12-21T10:23:57Z

mealViews.append(DotoriLabel("급식이 없습니다."))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))

StackView에 계속 모든 View를 추가하는 것는 코드적인 부분만이 아니라 성능 관련해서도 이슈가 있을 수 있겠네요.

baekteun · 2023-12-21T10:24:41Z

PR 설명에 이미지 넣을때 <img src="" width="150" /> 이렇게 넣어주시면 이미지 보기 편해요

uuuunseo · 2023-12-21T14:52:12Z

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift

+            mealViews.append(DotoriLabel(""))
+            mealViews.append(DotoriLabel(""))
+            mealViews.append(DotoriLabel(""))
+            mealViews.append(DotoriLabel(""))
+            mealViews.append(DotoriLabel(""))
+            mealViews.append(DotoriLabel(""))


append 코드가 없으면 오류가 나나요? 왜 이렇게 코드를 작성한 건지 궁금해요!

디자인에 "급식이 없습니다"가 View의 윗부분에 있어서 비어있는 DotoriLabel을 채워주면서 "급식이 없습니다"를 뷰의 윗부분에 보여주려고 저렇게 추가했어요!

오호 그럼 ""를 저렇게 View여러개로 보내는 거 보다

Suggested change

mealViews.append(DotoriLabel(""))

mealViews.append(DotoriLabel(""))

mealViews.append(DotoriLabel(""))

mealViews.append(DotoriLabel(""))

mealViews.append(DotoriLabel(""))

mealViews.append(DotoriLabel(""))

Array(repeating: DotoriLabel(""), count: 7)

이렇게 한 번 진행해보는건 어떨까요?!

…ckView.swift Co-authored-by: baegteun <[email protected]>

junseopark0331 · 2023-12-22T02:55:41Z

PR 제목에 앞에 prefix로 그냥 커밋 따라가기보다는 🔀 :: [#이슈넘버] 로 하는게 좋을거같아요

PR 제목에 [이슈넘버] => [#이슈넘버] 로 바꿔야할거같아요

커밋에 [#이슈넘버] prefix 안달아주신거같아용

pr제목은 수정했고 앞으로 커밋올릴때 [#이슈넘버] 형태로 올릴게요 !

junseopark0331 · 2023-12-22T02:57:03Z

혹시 mac에 swiftlint 설치해놓으셨나요? lint warning을 방치한채로 코드가 작성된거같아서 물어봐요

아니요...ㅎ 설치가 되어있는줄 알았는데 안되어 있었더라구요 ㅠ
다시 설치했습니다 !

junseopark0331 · 2023-12-22T02:57:28Z

혹시 make setup 명령어 안치셨나용?

명령어 치고 다시 올렸어요 !

junseopark0331 · 2023-12-22T02:59:41Z

PR 설명에 이미지 넣을때 이렇게 넣어주시면 이미지 보기 편해요

감사합니다 이미지 크기 줄였어요 !

junseopark0331 · 2023-12-22T03:00:16Z

mealViews.append(DotoriLabel("급식이 없습니다."))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))

StackView에 계속 모든 View를 추가하는 것는 코드적인 부분만이 아니라 성능 관련해서도 이슈가 있을 수 있겠네요.

흠... 성능 관련해서는 생각을 못해봤네요 ㅠ 어떻게 하면 좋을까요..?

baekteun · 2023-12-26T13:18:55Z

StackView에 계속 모든 View를 추가하는 것는 코드적인 부분만이 아니라 성능 관련해서도 이슈가 있을 수 있겠네요.

흠... 성능 관련해서는 생각을 못해봤네요 ㅠ 어떻게 하면 좋을까요..?

일단 이대로 할까요..?

baekteun · 2023-12-26T13:20:03Z

StackView에 계속 모든 View를 추가하는 것는 코드적인 부분만이 아니라 성능 관련해서도 이슈가 있을 수 있겠네요.

흠... 성능 관련해서는 생각을 못해봤네요 ㅠ 어떻게 하면 좋을까요..?

아니면 StackView에 추가하기보다 AutoLayout 같은거로 지정해놓고 isHidden 값을 바꿔주는 방법도 있을거같아요

baekteun

@uuuunseo 가 올린 DotoriLabel 추가하는 코드 간소화만 하면 될거같아요

baekteun · 2023-12-28T00:07:36Z

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift

-            mealViews.append(DotoriLabel(""))
-            mealViews.append(DotoriLabel(""))
-            mealViews.append(DotoriLabel(""))
+            mealViews.append(contentsOf: Array(repeating: DotoriLabel(""), count: 7))


위에서 공백인 Label 수는 6개인데 count도 6이여야할거같아요

junseopark0331 · 2023-12-28T00:09:11Z

@uuuunseo 가 올린 DotoriLabel 추가하는 코드 간소화만 하면 될거같아요

40e2194

이렇게 해보았는데요,

이렇게 "급식이 없습니다"가 제일 위에 나오지가 않아요 ㅠ
count에 숫자를 늘려봐도 글씨가 위로 추가가 안되는데 어떻게 수정하면 좋을까요??

baekteun · 2023-12-28T00:25:42Z

이렇게 "급식이 없습니다"가 제일 위에 나오지가 않아요 ㅠ count에 숫자를 늘려봐도 글씨가 위로 추가가 안되는데 어떻게 수정하면 좋을까요??

어 안되나요?

junseopark0331 · 2023-12-28T00:33:56Z

이렇게 "급식이 없습니다"가 제일 위에 나오지가 않아요 ㅠ count에 숫자를 늘려봐도 글씨가 위로 추가가 안되는데 어떻게 수정하면 좋을까요??

어 안되나요?

네네 ㅠ

junseopark0331 · 2023-12-28T01:09:44Z

이렇게 "급식이 없습니다"가 제일 위에 나오지가 않아요 ㅠ count에 숫자를 늘려봐도 글씨가 위로 추가가 안되는데 어떻게 수정하면 좋을까요??

어 안되나요?

네네 ㅠ

@baekteun @uuuunseo

mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))
mealViews.append(DotoriLabel(""))

일단 이 방식으로 할까요..??

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift

baekteun · 2023-12-29T03:30:49Z

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift

+        var mealViews: [DotoriLabel] = []
+        if meals.isEmpty {
+            let emptyLabels = (1...6).map { _ in DotoriLabel("") }
+            mealViews.append(DotoriLabel("급식이 없습니다."))
+            mealViews.append(contentsOf: emptyLabels)
+        } else {
+            mealViews = meals.map { DotoriLabel($0) }
+        }


Suggested change

var mealViews: [DotoriLabel] = []

if meals.isEmpty {

let emptyLabels = (1...6).map { _ in DotoriLabel("") }

mealViews.append(DotoriLabel("급식이 없습니다."))

mealViews.append(contentsOf: emptyLabels)

} else {

mealViews = meals.map { DotoriLabel($0) }

}

let mealViews: [DotoriLabel] = if meals.isEmpty {

[DotoriLabel("급식이 없습니다.")] + (1...6).map { _ in DotoriLabel("") }

} else {

meals.map { DotoriLabel($0) }

}

이게 Swift 5.9 문법인데 Github Action에서 Swift 버전이 5.9인지는 모르겠긴하겠네요
아무튼 이 새 문법 어떤가요

♻️ 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정

f5c3c3a

junseopark0331 added the 1️⃣Priority: High 우선순위 상 label Dec 21, 2023

junseopark0331 requested review from baekteun, kimsh153, minaje0917, MelBIP and uuuunseo as code owners December 21, 2023 09:50

junseopark0331 linked an issue Dec 21, 2023 that may be closed by this pull request

홈에서 급식이 없을때 없다는 것 알려주기 #184

Closed

baekteun reviewed Dec 21, 2023

View reviewed changes

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift Outdated Show resolved Hide resolved

baekteun reviewed Dec 21, 2023

View reviewed changes

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift Outdated Show resolved Hide resolved

uuuunseo reviewed Dec 21, 2023

View reviewed changes

Update Projects/Feature/HomeFeature/Sources/Scene/View/MealContentSta…

d215573

…ckView.swift Co-authored-by: baegteun <[email protected]>

junseopark0331 changed the title ~~♻️ [184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정~~ 🔀:: [#216] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 Dec 22, 2023

junseopark0331 changed the title ~~🔀:: [#216] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정~~ 🔀:: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 Dec 22, 2023

junseopark0331 added 2 commits December 22, 2023 11:48

♻️ ::[#184] HomeFeature / 공백제거

bb47081

♻️ MealContentStackView / 공백 제거

fca551f

baekteun approved these changes Dec 27, 2023

View reviewed changes

♻️::[#184] 반복문을 사용하여 빈 배열을 추가합니다.

40e2194

♻️::[#184] tuist fetch를 안해서 날라간 GAuth 추가

d59a02d

baekteun reviewed Dec 28, 2023

View reviewed changes

♻️::[#184] 빈 DotoriLabel 6개를 emptyLabels에 추가하여 mealViews에 추가해줍니다.

32c7ec0

baekteun reviewed Dec 29, 2023

View reviewed changes

Projects/Feature/HomeFeature/Sources/Scene/View/MealContentStackView.swift Outdated Show resolved Hide resolved

♻️::[#184] emptyLabels를 if문 안에 추가합니다.

4568f75

baekteun reviewed Dec 29, 2023

View reviewed changes

junseopark0331 added 3 commits December 29, 2023 14:02

♻️::[#184] 새로운 문법으로 mealViews를 수정

185978b

♻️ ::[#184] HomeFeature / Swift 5.8 대응

bed3483

♻️ ::[#184] HomeFeature / 공백제거

45fc15d

junseopark0331 merged commit 4106da4 into master Jan 4, 2024
4 checks passed

junseopark0331 deleted the 184-Notify-WhenMealIsEmpty branch January 4, 2024 09:27

baekteun changed the title ~~🔀:: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정~~ 🔀 :: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 Mar 15, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔀 :: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 #216

🔀 :: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 #216

junseopark0331 commented Dec 21, 2023 •

edited

Loading

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

uuuunseo Dec 21, 2023

junseopark0331 Dec 22, 2023

uuuunseo Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

baekteun commented Dec 26, 2023

baekteun commented Dec 26, 2023

baekteun left a comment

baekteun Dec 28, 2023

junseopark0331 commented Dec 28, 2023

baekteun commented Dec 28, 2023

junseopark0331 commented Dec 28, 2023

junseopark0331 commented Dec 28, 2023

baekteun Dec 29, 2023 •

edited

Loading

🔀 :: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 #216

🔀 :: [#184] 급식이 없을때 "급식이 없습니다"를 나타나게끔 수정 #216

Conversation

junseopark0331 commented Dec 21, 2023 • edited Loading

💡 개요

📃 작업내용

🔀 변경사항

🙋‍♂️ 질문사항

🍴 사용방법

🎸 기타

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

baekteun commented Dec 21, 2023

uuuunseo Dec 21, 2023

Choose a reason for hiding this comment

junseopark0331 Dec 22, 2023

Choose a reason for hiding this comment

uuuunseo Dec 22, 2023

Choose a reason for hiding this comment

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

junseopark0331 commented Dec 22, 2023

baekteun commented Dec 26, 2023

baekteun commented Dec 26, 2023

baekteun left a comment

Choose a reason for hiding this comment

baekteun Dec 28, 2023

Choose a reason for hiding this comment

junseopark0331 commented Dec 28, 2023

baekteun commented Dec 28, 2023

junseopark0331 commented Dec 28, 2023

junseopark0331 commented Dec 28, 2023

baekteun Dec 29, 2023 • edited Loading

Choose a reason for hiding this comment

junseopark0331 commented Dec 21, 2023 •

edited

Loading

baekteun Dec 29, 2023 •

edited

Loading