[1주차] Database 설계 및 아키텍처 설계 구현 완료

[dbdb1114]

[1주차] Database 설계 및 아키텍처 설계 구현 완료

1. 데이터베이스 설계 및 아키텍처 설계,
2. 메인 페이지 추가
3. 메인 페이지 상영정보 조회 기능 구현

작업 내용

이번 PR에서 진행된 주요 변경 사항을 기술해 주세요.
코드 구조, 핵심 로직 등에 대해 설명해 주시면 좋습니다. (이미지 첨부 가능)
ex: ConcurrentOrderService에 동시 주문 요청 처리 기능 추가

• module-rds-repo : DB와의 연결과 entity, repository를 활용합니다.
• module-service : repository를 이용하여 Data를 가져와 정렬하고, 정제하는 역할을 합니다.

	@Override
	public List<Map.Entry<MovieDTO, List<ShowingDTO>>> getTodayShowing() {
		LocalDateTime from = LocalDateTime.now().plusMinutes(30);
		LocalDateTime to = LocalDateTime.of(LocalDate.now(), LocalTime.MAX);

		List<Showing> showingsByStTimeBetween = showingRepository.findShowingsByStTimeBetween(from, to);

		// 금일 상영 정보가 아예 없을 경우 다음날 조회
		if(showingsByStTimeBetween.isEmpty() || showingsByStTimeBetween.size() < 10){
			LocalDateTime tomorrowFrom = LocalDateTime.of(LocalDate.now().plusDays(1L), LocalTime.MIN);
			LocalDateTime tomorrowTo = to.plusDays(1L);
			showingsByStTimeBetween.addAll(showingRepository.findShowingsByStTimeBetween(tomorrowFrom, tomorrowTo));
		}

		return showingsByStTimeBetween.stream()
			.map(showingEntity -> modelMapper.map(showingEntity, ShowingDTO.class))
			.collect(Collectors.groupingBy(ShowingDTO::getMovie))
			.entrySet().stream()
			.peek(entry -> entry.getValue().sort(Comparator.comparing(ShowingDTO::getStTime)))
			.sorted(Map.Entry.comparingByKey(Comparator.comparing(MovieDTO::getOpenDay, Comparator.reverseOrder())))
			.toList();
	}

• module-app : module-service에서 조회한 데이터를 응답값에 맞게 수정 보완 처리하여 response를 반환합니다.

	@GetMapping("/today-info")
	public ResponseEntity<Map> getTodayShowing() {
		List<Map.Entry<MovieDTO, List<ShowingDTO>>> todayShowing = showingService.getTodayShowing();

		List<Map<String, Object>> responseData = todayShowing.stream()
			.map(entry -> Map.entry(entry.getKey(), entry.getValue().stream()
				.map(val -> modelMapper.map(val, ShowingResponse.class)).toList()))
			.map(entry -> Map.of("movie", entry.getKey(), "timeTable", entry.getValue()))
			.toList();

		if (responseData.isEmpty()) {
			return ResponseEntity.noContent().build();
		} else {
			return ResponseEntity.ok(Map.of("data", responseData));
		}
	}

• module-core : dto클래스를 작성했습니다. 구성은 아래와 같습니다.

.
├── dto
│   ├── external
│   │   └── ShowingResponse.java
│   └── internal
│       ├── GenreDTO.java
│       ├── MovieDTO.java
│       ├── RatingDTO.java
│       ├── ScreenDTO.java
│       └── ShowingDTO.java
├── exception
└── util

발생했던 문제와 해결 과정을 남겨 주세요.

• 문제 1 - 아키텍처를 설계하면서 이정도 기능의 설계에 있어 멀티모듈 아키텍처가 어떤 긍정적인 효과를 가져올지 고민이 많았습니다.
• 해결 방법 1 - 제가 이해하기로는 결국 멀티모듈 아키텍처를 설계하는 이유도 팀의 불편함, 개인의 불편함에서 시작된 것 같았습니다.
  그래서 제가 어떻게 하면 이전보다 나은 코드와 역할과 책임의 분리를 잘 지킬 수 있을지 고민하여 LayeredArchitecture를 기반으로 multimodule 프로젝트를 설계하게 됐습니다.
• 문제 2 - 레이어간 역할과 책임의 분리 어느정도 까지인지 컨트롤러의 책임과 역할은 뭐고, 서비스의 책임과 역할은 어느정도 까지인지 고민이 많았습니다.
• 해결 방법 2 - 무엇을 위해 존재하는가를 생각했습니다. 그래서 컨트롤러는 입출력을 담당하는 부분이라 생각하여, 데이터의 응답 규격에 맞추는 일을 수행하게 했고, 서비스 레이어는 비즈니스적으로 원하는 데이터를 적당히 깔끔한 규격으로 반환하게 하였습니다.

이번 주차에서 고민되었던 지점이나, 어려웠던 점을 알려 주세요.

과제를 해결하며 특히 어려웠던 점이나 고민되었던 지점이 있다면 남겨주세요.

• 클린 코드에 대한 고민이 많았습니다. 이 메소드명은 적절한지, 뎁스는 적절한지 내 코드만 계속 보고 있으니 가독성이 좋게 느껴지는 건 아닌지 고민이 많았습니다.
• stream api를 사용하면 데이터를 추가적인 지역변수나 제어문 선언 없이 로직을 처리할 수 있습니다. 코드가 간결해 보이긴 하지만 하다보니 중간 연산의 값을 파악하기 힘든 그런 일들이 있었습니다. 어떻게 사용하는게 좋은지 헷갈렸습니다.
• 실무에서 어떻게 해야할지 고민이 많습니다. 저는 취준생 입장이라 현업에 나아가서 일을 하려면 지금 프로젝트를 혼자 진행하는 방식과 제 노력과 같은 부분이 적절한지 계속해서 고민했습니다.
• 커밋 로그를 어떻게 정리해야할지 감이 잘 안 잡힙니다. service레이어를 수정하다보면 Repository 수정이 필요해지고 그러다보면 커밋이 시간흐름을 어기게 되거나 중간중간 service와 repository를 왔다 갔다 하며 난잡해지는 것 같습니다.

리뷰 포인트

• 최대한 비판적으로 봐주셨으면 좋겠습니다. 고칠점을 서슴없이 말씀해주셨으면 좋겠습니다.
• stream을 과하게 사용한 것은 아닌지 리뷰 해주셨으면 좋겠습니다.
• entity설계를 지적해주셨으면 좋겠습니다.
• 전반적으로 보시기에 약간 거슬리는 부분들도 asis와 tobe가 명확하진 않더라도 듣고싶습니다.

기타 질문

• 테스트 코드를 제대로 작성하며 프로젝트를 해본 경험이 없습니다. 참고할만한 책이나 강의나 라이브러리를 알려주시면 감사할 것 같습니다.
• 실무에서 어떻게 일하는지 궁금합니다. 예를 들어 하루에 코드를 작성하는 시간은 몇 시간이나 되는지 코드를 작성하기 위해 설계하는 시간은 얼마나 가지는지 궁금합니다.

추가

늦어서 죄송합니다. 계속 부족한 부분들이 보여 급하게 하다보니,,, 오히려 더 늦어졌습니다. 차주에는 시간을 조금 더 잘 지키도록 하겠습니다.