[SVR-165] [부엌] 부엌 화면 & 식재료와 조리도구 조회 #150
Conversation
kth1888
requested review from
Atralupus and
upa-r-upa
and removed request for
a team
|
엔드 포인트는 Constant로 따로 저장하고, 참조해서 사용하는 방식이 더 좋을 것 같아요. |
There was a problem hiding this comment.
아무래도 이름이 client2는 좀 그렇죠..
용도에 따라 이름을 바꾸면 좋겠네요
| #slot 1 | ||
| var slot1 = largeslots["firstApplianceSpace"] | ||
| if (slot1.installedKitchenEquipment == null): # not installed | ||
| var bigslot = SLOT_EMPTY.instantiate() | ||
| slot.add_child(bigslot) | ||
| else: # installed but not used | ||
| if (!slot1["installedKitchenEquipment"]["isCooking"]): | ||
| var bigslot = SLOT_NOT_USED.instantiate() | ||
| bigslot.set_data(slot1) | ||
| slot.add_child(bigslot) | ||
| else: # cooking | ||
| var bigslot = SLOT_USED.instantiate() | ||
| bigslot.set_data(slot1) | ||
| slot.add_child(bigslot) | ||
| #slot 2 | ||
| var slot2 = largeslots["secondApplianceSpace"] | ||
| if (slot2.installedKitchenEquipment == null): | ||
| var bigslot = SLOT_EMPTY.instantiate() | ||
| slot.add_child(bigslot) | ||
| else: | ||
| if (!slot2["installedKitchenEquipment"]["isCooking"]): | ||
| var bigslot = SLOT_NOT_USED.instantiate() | ||
| bigslot.set_data(slot2) | ||
| slot.add_child(bigslot) | ||
| else: | ||
| var bigslot = SLOT_USED.instantiate() | ||
| bigslot.set_data(slot2) | ||
| slot.add_child(bigslot) | ||
| #slot 3 | ||
| var slot3 = largeslots["thirdApplianceSpace"] | ||
| if (slot3.installedKitchenEquipment == null): | ||
| var bigslot = SLOT_EMPTY.instantiate() | ||
| slot.add_child(bigslot) | ||
| else: | ||
| if (!slot3["installedKitchenEquipment"]["isCooking"]): | ||
| var bigslot = SLOT_NOT_USED.instantiate() | ||
| bigslot.set_data(slot3) | ||
| slot.add_child(bigslot) | ||
| else: | ||
| var bigslot = SLOT_USED.instantiate() | ||
| bigslot.set_data(slot3) | ||
| slot.add_child(bigslot) |
There was a problem hiding this comment.
이 부분은 사실 값만 다르고 행위는 다 똑같은 것 같은데
함수로 묶거나, 아니면 반복문으로 해도 되고.. 중복 코드를 없애면 좋을 것 같아요
There was a problem hiding this comment.
확실히 그렇군요 한번 함수로 묶어서 정리해보겠습니다 !
|
브랜치 이름이 |
|
그리고 커밋 말머리? 헤드는 물론 자유이긴 한데 https://gist.github.com/joshbuchea/6f47e86d2510bce28f8e7f42ae84c716 |
아무래도 나름 따른다고 비슷하게 적었는데 작업을 잘 못 구분짓나 봅니다.. |
이것도 서버 오락가락 할 때 버튼 형식으로 로컬서버 여부를 선택할 수 있도록 찾아봤는데, 맨 처음 인트로에서 모든 쿼리가 진행되어야 하는 구조 같아서 개별로 처음 진입 전 설정 씬을 한번 거치게 할까 등 한번 다시 고민해보겠습니다. |
네에. 표현은 자유롭게 하셔도 될 것 같아요. |
TL;DR
집 화면에서 작은/큰 조리도구에 따른 조리중인 요리를 볼 수 있습니다.
배경 및 목표
조리중인 음식을 보여주되, 큰 조리도구를 설치하여 사용한 요리는 별개로 보이도록 합니다.
작업 사항
리뷰어가 중점적으로 볼 사항
현재 블록 조회를 위하여 클라이언트 코드를 하나 더 만들었습니다.
(나중에 빌드 시에 endpoint를 둘 다 수정해야 합니다)