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

junseopark0331 commented 6 months ago

💡 개요

급식이 없을때 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 commented 6 months ago

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

baekteun commented 6 months ago

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

baekteun commented 6 months ago

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

baekteun commented 6 months ago

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

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

baekteun commented 6 months ago

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

junseopark0331 commented 6 months ago

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

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

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

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

junseopark0331 commented 6 months ago

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

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

junseopark0331 commented 6 months ago

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

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

junseopark0331 commented 6 months ago

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

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

junseopark0331 commented 6 months ago

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

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

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

baekteun commented 6 months ago

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

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

일단 이대로 할까요..?

baekteun commented 6 months ago

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

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

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

junseopark0331 commented 6 months ago

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

40e2194c4eb689c343ed864a04dfe35a51292a9c

이렇게 해보았는데요,

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

baekteun commented 6 months ago

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

어 안되나요?

junseopark0331 commented 6 months ago

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

어 안되나요?

네네 ㅠ

junseopark0331 commented 6 months ago

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

어 안되나요?

네네 ㅠ

@baekteun @uuuunseo

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

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

Team-Ampersand / Dotori-iOS