inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

스프링 MVC 2편 - 백엔드 웹 개발 활용 기술

셀렉트 박스

DeliveryCode 리팩토링

해결된 질문

474

nove1080

작성한 질문수 8

1

[문제 상황]

강의 자료에서 언급되어 있는 DeliveryCode 객체가 반복 생성되는 것을 개선하도록 리팩토링 해보았습니다.

제가 작성한 코드의 문제점 또는 개선할 점을 알려주신다면 감사드리겠습니다.


[리팩토링 전]


[리팩토링 후]

spring mvc

답변 2

0

David

안녕하세요. nove1080님, 공식 서포터즈 David입니다.

잘 작성하셨습니다:)

감사합니다.

0

인프런 AI 인턴

안녕하세요, 인프런 AI 인턴이에요.

DeliveryCode 객체의 반복 생성을 개선하는 리팩토링을 진행하셨다고 하셨군요. 제가 코드를 직접 확인할 수는 없지만, 전반적인 구조를 기반으로 문제점과 개선할 점을 알려드릴게요.

리팩토링 전의 상황에서 DeliveryCode 클래스는 FormItemController에서 Model에 등록되는 것으로 보입니다. 그리고 리팩토링 후에는 DeliveryCodeRepository 클래스가 추가되었으며, FormItemController에서 Model 등록 부분이 수정되었다고 하셨네요.

저는 리팩토링 전의 코드를 직접 볼 수는 없지만, 반복 생성되는 문제를 해결하기 위해 DeliveryCodeRepository 클래스를 추가한 것은 좋은 접근 방법입니다. DeliveryCode 객체를 캐싱하고 재사용하는 로직을 추가하여 반복 생성을 방지할 수 있을 거예요.

리팩토링 후에 수정한 FormItemController에서 Model 등록 부분을 보셔야 정확한 개선점을 알 수 있을 것 같습니다. DeliveryCodeRepository를 사용하여 DeliveryCode 객체를 가져와서 Model에 등록하는 것으로 이해하고 있습니다.

그 외에도 리팩토링을 진행하면서 변경한 사항이 있다면 더 자세하게 알려주시면, 보다 구체적인 피드백을 드릴 수 있을 거예요.

리팩토링 작업을 진행하셨다니 멋져요! 리팩토링을 통해 코드의 가독성, 유지보수성, 성능 등을 개선할 수 있기 때문에 꾸준한 노력을 추천해드립니다. 혹시 추가적인 질문이나 도움이 필요하시면 언제든지 물어보세요. 좋은 결과 얻기를 기대할게요!

이미지 업로드와 db 트랜잭션 묶는법

0

46

1

Could not resolve org.springframework.boot:spring-boot-starter-validation:2.4.4

0

55

2

MessageSourceTest 코드

0

49

1

인터셉터 에러 설정

0

48

1

resolveArgument()메서드 질문

0

58

1

43강 검증1 에서 실패 로직 관련 질문있습니다.

0

60

2

타임리프 3.X 버전 rendering, serializer 에러 해결 방법

2

133

3

스프링 빈에 등록이 안되는거 같은데 어떻게 하면 좋을까요?ㅠㅠ

0

93

3

pdf 오타 문의

0

57

1

ItemUpdateForm 검증 관련 질문입니다.

0

50

1

22page 링크 주소 변경

0

59

2

특정 데이터와 파일을 함께 저장 시, 테이블 구조 질문

0

53

1

섹션3번 수업에 대한 질문입니다.

0

80

2

@Autowired 보다 더 좋은 방법이 어떤 걸까요?

0

87

2

타입컨버터 가 람다랑 비슷해 보이는데 저의 생각이 맞는지?.

0

66

1

자바스크립트 인라인에서 객체 직렬화 시 오류가 납니다

0

143

3

스프링부트 - 오류페이지2 에서 500.html 에서 쓰인 객체 질문

0

65

1

톰캣 에러 페이지가 안보입니다.

0

105

2

apiEceptionController에서 센드 에러 호출하면 안되는지?

0

81

1

세션 타임아웃시 쿠키 삭제 방법이 없나요?

0

119

2

ApiExceptionController 질문드립니다.

0

64

1

셀렉박스 챕터에서 option value에 ==배송 방식 선택== 이것을 넣은 이유가 궁금함, 이렇게 구상해도 되는지?

0

66

1

MemberRepository 필드의 fianl 선언 유무

0

85

2

혹시 index.html 에서는 fragment 사용이 안되는건가요

0

59

1