매개 변수로 배열의 배열을 보낸 다음 각 배열을 반복하면서

Question

나는 useremail, videocount, imagecount, videoids, imageids 등의 모든 정보를 얻을 필요가있는 모든 users.hence에게 알림 메일을 보내고 있습니다. usermailer에게 mail.i를 보내기 위해 보내십시오. 모든 필수 매개 변수를 메소드에 보내고 배열의 각 배열을 반복하여이 프로세스를 단순화하고 싶습니다.

for example:- 

    ###########the below code is in a loop of users################ 

    User.signed_in_users.find_in_batches(:batch_size => 100) { |users| users.each { |user| 

    vcount=Video.where(:users_notified=>f).count 
    icount=Image.where(:users_notified=>f).count 
    acount=Audio.where(:users_notified=>f).count 

    @count << vcount + icount + acount 

    vcat=###ALL VIDEO CATEGORIES 
    icat=###ALL IMAGE CATEGORIES 
    acat=###ALL AUDIO CATEGORIES 

    @cats << vcat + icat + acat...and many more 


    ###########now sending all these parameter to mailer(array of arrays) 
    ####how i can simplify this call 
UserMailer.notify_user(user.email,@video_cat,@video_count,@image_cat,@image_count,@audio_cat,@audio_count,@place_cat,@place_count,@Lpage_count,@Dpage_count).deliver 


    } } ###loop ends 

Answer 1

왜 동일한 값을 여러 번 가져 오려고하는지 알 수 없습니다. 아래의 코드는 아무런 이유없이 여러 번 실행됩니다.

vcount=Video.where(:users_notified=>f).count 
icount=Image.where(:users_notified=>f).count 
acount=Audio.where(:users_notified=>f).count 

@count << vcount + icount + acount 

vcat=###ALL VIDEO CATEGORIES 
icat=###ALL IMAGE CATEGORIES 
acat=###ALL AUDIO CATEGORIES 

@cats << vcat + icat + acat 

user 개체를 반복하고 있기 때문에. 이 코드 묶음에는 user의 상호 작용이 없습니다. 따라서 각 블록에서 꺼낼 수 있습니다.

vcount=Video.where(:users_notified=>f).count 
icount=Image.where(:users_notified=>f).count 
acount=Audio.where(:users_notified=>f).count 

@count << vcount + icount + acount 

vcat=###ALL VIDEO CATEGORIES 
icat=###ALL IMAGE CATEGORIES 
acat=###ALL AUDIO CATEGORIES 

@cats << vcat + icat + acat 

User.signed_in_users.find_in_batches(:batch_size => 100) { |users| users.each { |user| 

###########now sending all these parameter to mailer(array of arrays) 
    UserMailer.notify_user(user.email,@count,@cats) 

} } ###loop ends