새로운 카페테리아 University Club을 추가하였습니다

TaekH commented 1 year ago

close #98

University Club이라는 서울 캠퍼스의 카페테리아를 추가하였습니다.

중앙대학교 포털에서 University Club이라는 카페테리아가 운영됨에 따라 추가하였습니다. 크롤러 확인 결과 정상적으로 크롤링 해오는 것을 확인했습니다 따라서 단순히 case 추가만 하였습니다.

UserDefaults와 Cafeteria 모델 등 각 사용되는 부분에 추가하였습니다.
설정창에서도 추가 후에 카페테리아 순서 변경시 적용되는 것을 확인하였습니다.

추가 수정사항

6a382e4

University Club의 각 메뉴가 하나뿐인 관계로 colCount를 1로 수정하여 메뉴가 하나로 들어가고 잘리지 않도록 하였습니다.

if data.cafeteria == .blueMirB { colCount = 1 }

이 부분을

if data.cafeteria == .blueMirB || data.cafeteria == .universityClub { colCount = 1 }

으로 수정하여 생활관B의 케이스 외에도 추가하였습니다.

추가 수정사항

b09dbb7

university club 같이 cafeteria case가 들어가있지 않은 상황인 경우에 default값으로 data가 들어가게 되어서 다른 카페테리아로 반영되는 경우가 있었습니다.

아래와 같이 생활관 식단에 현재 cafeteria case에 추가되어있지 않은 university club의 데이터가 default로 설정된 생활관 식당에 들어가서 발생한 오류였습니다.

    static func getCafeteria(_ cafeteria: String) -> Cafeteria {
        switch cafeteria {
        case "생활관식당(블루미르308관)":
            return .blueMirA
        case "참슬기식당(310관 B4층)":
            return .chamseulgi
        case "생활관식당(블루미르309관)":
            return .blueMirB
        case "학생식당(303관B1층)":
            return .student
        case "교직원식당(303관B1층)":
            return .staff
        case "University Club(102관11층)":
            return .universityClub
        case "카우잇츠(cau eats)":
            return .cauEats
        case "(안성)카우버거":
            return .cauBurger
        case "(안성)라면":
            return .ramen
        default:
            return .blueMirA //cafeteria case가 없는 데이터의 경우 생활관으로 지정됨
        }
    }

따라서 Cafeteria 모델에 .none case를 추가하여 case에 없는 데이터가 들어와도 다른 카페테리아로 반영되지 않도록 수정하였습니다.

.none를 추가하면서 다른 switch문도 변경하였습니다. 확인 부탁드립니다 !

+ 리뷰 요청 사항

View에 표시되는 카페테리아 이름을 우선 University Club 그대로 사용하였습니다. 다르다면 확인부탁드립니다.
위치표기를 R&D센터(102) 11층으로 하였습니다. 오기입일시에 수정부탁드립니다.
메뉴가 하나씩이라 여백이 좀 있어보이는데 이 부분도 혹시 어색하다면 확인 부탁드립니다.

TaekH commented 1 year ago

추가로 메뉴가 길어서 잘리는 부분도 수정이 필요할 것 같습니다 ..!

TaekH commented 1 year ago

추가로 메뉴가 길어서 잘리는 부분도 수정이 필요할 것 같습니다 ..!

6a382e4 여기서 수정하였습니다 확인 부탁드립니다 !

HipstuCAU / nyam_nyam