Skip to content

[운송 III (운송)] 미션 구현 (신윤철)#12

Open
Shin-Yun-Cheol wants to merge 14 commits intoallrounder-backend:mainfrom
Shin-Yun-Cheol:logistic3
Open

[운송 III (운송)] 미션 구현 (신윤철)#12
Shin-Yun-Cheol wants to merge 14 commits intoallrounder-backend:mainfrom
Shin-Yun-Cheol:logistic3

Conversation

@Shin-Yun-Cheol
Copy link

💡 Summary

구현한 기능

  • 주문 내역을 txt로 저장
  • 저장한 txt에서 이름으로 검색

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

  • 스레드 관리

📚 Retrospective (YWT)

What I did

  • txt파일로 저장하기 위해 로직 추가

What I Learned

  • 동시성 제어를 위한 reentrant사용

What I will do next

  • 함수 볼륨 줄이기
  • 역할 나누기

✨ Impression

앞에 잘 나눠둬서라고는 자신있게 말 못하지만 이번 주차 미션에서 조금만 추가하면 됐던 것 같아서 상당히 편했다고 생각한다

private final PlaceService placeService;
private final double speedKmh;

public DeliveryService(PlaceService placeService, double speedKmh) {

Choose a reason for hiding this comment

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

속도를 생성자에 넣게 해서 설정할 수 있게 했네요.


import java.nio.file.Paths;

public class AppConfig {

Choose a reason for hiding this comment

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

AppConfig에 뭐가 들어가야 할 지 몰랐었는데 필요한 인스턴스를 미션에 맞게 생성하는 것이군요.

import api.Console;

public class Application {
public static void main(String[] args) {

Choose a reason for hiding this comment

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

AppConfig를 쓰니 main함수가 간결해지네요.

if (inputName == null) {
return "";
}
String nfcNormalized = Normalizer.normalize(inputName, Normalizer.Form.NFC);

Choose a reason for hiding this comment

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

정규화에 신경 쓴 점이 좋네요

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