DonutWorks / Ari

0 stars 0 forks source link

사용자에게 자동으로 SMS 를 보낸다. #314

Closed MoojinChae closed 9 years ago

MoojinChae commented 9 years ago
shaynekang commented 9 years ago

수고하셨습니다~

  1. 모델의 상태 변경(create/update)후 무언가를 해줘야 할 때, 모델을 직접 건드려야 하는 경우는 Callback을, 아닌 경우는 Service 또는 Observer를 권장합니다. 모델 안에 코드를 작성해도 동작은 하지만, 장기적으로는 모델 코드가 너무 커져서 유지보수가 어려워 지거든요. models/response.rbafter_ 관련 콜백은 가능하면 다른 곳으로 분리해주면 좋겠네요. ㅎㅎ
  2. 자주 이야기하지만, Native SQL보다는 모델의 where를 직접 활용하는 걸 권장합니다. models/notice.rbNotice.where("notice_type = ? AND due_date = ?", "to", Date.today + 3.days)Notice.where(notice_type: 'to').where(due_date: Date.today + 3.days)로 바꿔줘도 동일하게 동작할 것 같습니다.
  3. scheduler.rb에서 바로 모델을 호출해주는 것도 가능하지만, 별도의 Rake Task를 만들어서 이를 호출해주는 것도 좋습니다. 저라면 models/notice.rbdeadline_send_sms 코드를 Rake Task로 빼서 이를 호출해줄 것 같아요. ㅎㅎ
MoojinChae commented 9 years ago

3번은 다른 이슈로 만들어서 처리 하겠습니다!