Skip to content

πŸ”¨ [REFACTOR] RoomConditionQueryServiceImpl λ¦¬νŒ©ν† λ§Β #217

@hambining

Description

@hambining

μ„€λͺ… πŸ“ƒ

ν–„λΉ„λ‹ˆ λ‹΄λ‹Ή @hambining

πŸš¨Β μ€‘λ³΅ μ½”λ“œ too much

childRepository.findById(childId).orElseThrow(()->new ChildException(ChildErrorCode.NOT_FOUND));
        if(!childRepository.existsByChildIdAndParentId(childId, parent.getId())) {
            throw new RoomConditionException(RoomConditionErrorCode.UNAUTHORIZED_ACCESS);
        }

πŸš¨Β ν•œμ€„ μ½”λ“œ γ„΄γ„΄

RoomCondition roomCondition = roomConditionRepository.findTopByChildIdOrderByCreatedAtDesc(childId).orElseThrow(()->new RoomConditionException(RoomConditionErrorCode.NOT_FOUND));
RoomCondition roomCondition = roomConditionRepository.
                findTopByChildIdOrderByCreatedAtDesc(childId).
                orElseThrow(() ->new RoomConditionException(RoomConditionErrorCode.NOT_FOUND));

주녕이 λ‹΄λ‹Ή @jh010303

🚨 SRP μœ„λ°˜

μ„œλΉ„μŠ€ λ„€μž„μ— λŒ€ν•œ μ˜λ„κ°€ μ •ν™• ❌

1/3/7일 κ³„μ‚°ν•˜λŠ” ν•¨μˆ˜κ°€ λ”°λ‘œ μžˆμ—ˆλ‹€λ©΄ ?

곡톡 λΆ€λΆ„ 계산을 λ”°λ‘œ 빼야함

쿼리 μ„œλΉ„μŠ€μΈλ° 쿼리λ₯Ό 날리지 μ•ŠλŠ” 계산을 ν•œλ‹€ ? β†’ SRP μœ„λ°˜

calculating β†’ 쿼리 μ„œλΉ„μŠ€μ—μ„œ γ„΄γ„΄γ„΄

πŸš¨Β μ»¨λ²€μ…˜ @!!@!!!!


μ§„ν–‰ 상황 βœ…

ν˜„λΉˆ

  • 쀑볡 μ½”λ“œ μˆ˜μ •
  • ν•œμ€„ μ½”λ“œ μˆ˜μ •

μ€€ν˜•

  • SRP μœ„λ°˜ λΆ€λΆ„
  • μ˜¨λ„/μŠ΅λ„ 뢈일치 λΆ€λΆ„ μˆ˜μ •
  • μ»¨λ²€μ…˜

Metadata

Metadata

Labels

πŸ”¨ Refactorμ½”λ“œ λ¦¬νŒ©ν† λ§

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions