search

DonutWorks / Ari

0 stars 0 forks source link

SendSMS 제거하기 #340

Closed shaynekang closed 10 years ago

shaynekang commented 10 years ago

lib/send_sms.rb는 사용하지 않는 코드인거죠? 그럼 제거하고 SendSMS2의 이름도 바꾸도록 합시다. ㅎㅎ

그리고 SmsSender와 SendSMS의 이름이 모호한 것 같습니다. 이름만으로는 두 개가 어떤 차이가 있는지 모르겠네요. 이 역시 수정하면 좋겠습니다. ㅎㅎ

angdev commented 10 years ago

SmsSender 는 '공지에서 메시지 보내는 서비스 객체' 로 분리하면 좋을 것 같습니다.