Skip to content

[운송 I (배송)] 미션 구현 (박호건)#3

Open
moongua404 wants to merge 3 commits intoallrounder-backend:mainfrom
moongua404:feat-5th-week
Open

[운송 I (배송)] 미션 구현 (박호건)#3
moongua404 wants to merge 3 commits intoallrounder-backend:mainfrom
moongua404:feat-5th-week

Conversation

@moongua404
Copy link
Contributor

Allrounder-Study-Mission-PR

[운송 I (배송)] 미션 구현 (박호건)

💡 Summary

구현한 기능

  • 터미널 입출력
  • 소요시간 예측

✅ CheckList

  • 기본 기능 구현
  • 응용 기능 구현
  • 심화 기능 구현
  • 통합테스트 작성
  • 단위테스트 작성

신경써서 구현한 점 / 어려웠던 점

  • 헥사고날 아키텍처 도입
  • 아키텍처에 맞는 역할
  • 다형적인 코드 작성 (CsvLoader)

📚 Retrospective (YWT)

What I did

  • 헥사고날 아키텍처 "진짜" 도입
  • 자바의 다형성 맛보기
  • 리꾸 (README 꾸미기..ㅎ)

What I Learned

  • "진짜" 헥사고날 아키텍처

What I will do next

  • 단위테스트...
  • parsable 로직 수정... (제네릭과 인터페이스는 무엇을 위한 것인가...)

✨ Impression

오랫만에 열심히 코드 짰네요 하하...
확실히 직접 고민해보는 시간이 길수록 배운게 많은거같아요

- 다익스트라 shortest path (feat: GPT... 야무지네;;;)
Place startPoint = program.getStartPoint();
Place endPoint = program.getEndPoint();
LocalTime duration = program.predictDuration(startPoint, endPoint);
System.out.printf("이동 시간은 %d시간 %d분으로 예측됩니다. %n"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 printf문은 왜 adapter에서 안 하나요?

import mission.application.domain.exception.DataLoadException;
import mission.utility.Parsable;

public record Place(int id, String name, String address) implements Parsable<Place> {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확실히 record 쓰니까 간결하고 좋네요

public record Place(int id, String name, String address) implements Parsable<Place> {

@Override
public Place parse(List<String> objects) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

왜 static 메서드하고 인스턴스 메서드 둘 다 만든 건가요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants