Replies: 5 comments 4 replies
-
|
저도 두 번째 방법을 더 선호합니다. 도메인 지식을 다루는 도메인 객체와 DB 에 직접 질의를 하는 ORM 이 분리가 되어있다면 첫 번째 방법을 선호할 것 같지만, |
Beta Was this translation helpful? Give feedback.
-
|
저도 optione2에 동의합니다. |
Beta Was this translation helpful? Give feedback.
-
|
Option 2에 동의합니다. |
Beta Was this translation helpful? Give feedback.
-
|
저도 Option 2 에 동의합니다. 다만 만약 def publish(self, commit=True):
self.published_at = timezone.now()
if commit is True:
self.save()와 같이 만드는 것을 선호합니다. |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
데이터를 변경하는 인스턴스 메서드의 save() 포함 여부에 관한 의견을 들어보고 싶습니다.
아래의
Post모델과 인스턴스 메서드publish가 있다고 가정하겠습니다.인스턴스 메서드의 save() 함수의 포함 여부에 따라 다음과 같이 두 가지 방식으로 함수를 사용할 수 있습니다.
Option 1
Option 2
제 생각에는 Option 2 의 방식이 대부분의 경우 합리적인것 같습니다.
일반적으로
publish라는 행위에서 실제로 포스팅이 공개되는 것 까지를 기대하게 될 것 같기 때문입니다.만약 Option 1 형태로 작성할 경우가 종종 필요할 수 있는데요.
이 경우 명시적으로 함수의 이름을
set_published_at과 같은 형태로 만들어줄 수 있을것 같습니다.다른 분들의 의견이 궁금해요.
Beta Was this translation helpful? Give feedback.
All reactions