2014-03-19 3 views
0

컨트롤러에서 5 개의 매개 변수가 필요한 Resque 작업이 있습니다. 나는 많은 매개 변수를 메소드에 전달하는 것이 좋지 않다는 것을 알고 있으므로, 개선 할 수있는 좋은 방법이 무엇인지 궁금해하고있었습니다.많은 매개 변수를 메서드로 전달할 때 코드 개선

Resque.enqueue (ExporterJob, "게임", date_range_array, PARAMS [: game_code, 국가 [: ID, current_user.email)

다음

은 작업 호출이 컨트롤러의 모습입니다

답변

0

이 많은 인수를 여러 번 전달할 많은 작업이 있었다면 이론적으로 JSON 직렬 가능이며 모든 인수를 특성으로 캡슐화 할 수있는 게임 내보내기 개체를 만들 수 있지만 그저 오버 헤드가되는 것 같습니다. 하나의 직업.

인수가 너무 많으면 커플 링이 너무 많이 들릴 수 있지만 디자인이 괜찮다는 것이 확실하다면 어떻게 향상시킬 수 있는지 생각할 수 없습니다. 그대로 두십시오. :)