तारीखों के साथ रेल मार्ग

Question

तो मेरे पास साप्ताहिक कैलेंडर दृश्य है और मेरे पास प्रारंभ तिथि के लिए/वर्ष /: माह/दिन स्वीकार करने के लिए एक मार्ग स्थापित है।

match "events/(:year/:month/:day)" => "events#index", 
     :constraints => { :year => /\d{4}/, :month => /\d{2}/, :day => /\d{2}/ }, 
     :as => "events_date" 

मेरे पास इस मार्ग के उपयोग के संबंध में दो प्रश्न हैं।

unless params[:year].nil? || params[:month].nil? || params[:day].nil? 
    start_date = Date.new(params[:year].to_i, params[:month].to_i, params[:day].to_i) 
end 
start_date = start_date.nil? ? Date.today : start_date 

यह मैं के रूप में सुंदर वर्बोज़ और एक तरह से बदसूरत हमलों: सबसे पहले, जब पैरामीटर पार्स करने, यह मैं क्या कर रहा है। क्या कोई बेहतर तरीका है?

और जब (पेजिंग सप्ताह के लिए सप्ताह के लिए) कैलेंडर में एक और सप्ताह के लिए एक लिंक बनाने, मैं की तरह

#assume an date object with the desired start date 
link_to events_date_path(date.strftime('%Y'), date.strftime('%m'), date.strftime('%d')) 

भी तरह का अत्यधिक शब्द और बदसूरत लगता है कौन सा कुछ करने के लिए है। मार्गों में तिथियों के साथ काम करने का सबसे अच्छा तरीका क्या है?

Answer 1

मेरा सुझाव तीन अलग-अलग चर का उपयोग नहीं करना होगा। इस तरह आप अपने नियंत्रक में बहुत अधिक नल जांच और सैनिटी जांच के साथ समाप्त नहीं होते हैं। आप अपने मैच में कुछ इस तरह लग रहे करने के लिए, अपने बाधाओं के साथ अभी भी कुशलता में बदल सकता है:

unless params[:date] 
    start_date = params[:date].strftime("%Y-%m-%d').to_date # assuming you want a Date 
end 

और मैं:

match "events/(:date)" => "events#index", 
     :constraints => { :date => /\d{4}-\d{2}-\d{2}/ }, 
     :as => "events_date" 

इस प्रकार आप कुछ नियंत्रक में एक छोटे से अधिक समझदार के साथ खत्म होगा आम तौर पर, उन प्रकार के कर की 'अगर यह सेट किया गया है' अधिक कुछ इस तरह की जाँच करता है, क्योंकि मैं इसे खोजने के अधिक पठनीय एक सा:

start_date = Date.today unless defined? start_date 

तुम भी उन पिछले दो एक साथ रोल कर सकते हैं:

start_date = defined?(params[:date]) ? params[:date].strftime("%Y-%m-%d').to_date : Date.today