2016-11-22 5 views
0

У меня есть функция поиска с несколькими условиями и множеством панелей, соединенных вместе, как показано нижеКак изменить стиль проверки вопроса в rubocop?

hospitals = Hospital.order(created_at: :desc).group(:id) 
hospitals = hospitals.joins(medical_subjects: :sicks).where("(medical_subjects.name LIKE ? OR sicks.name LIKE ?) AND hospitals.name LIKE ?" , 
                  "%#{params[:freeword_medical_subject_sick_name].strip}%" , 
                  "%#{params[:freeword_medical_subject_sick_name].strip}%" , 
                  "%#{params[:freeword_hospital_name].strip}%") if params[:freeword_medical_subject_sick_name].present? 
hospitals = hospitals.joins(prefecture: [:cities, :stations]).where("cities.name LIKE ? OR stations.name LIKE ?" , 
                        "%#{params[:freeword_city_station_name]}%" , 
                        "%#{params[:freeword_city_station_name].strip}%") if params[:freeword_city_station_name].present? 
hospitals = hospitals.where(["hospitals.name LIKE ?" , "%#{params[:freeword_hospital_name].strip}%"]) if params[:freeword_hospital_name] 
hospitals = hospitals.where(woman_doctor_existed: params[:woman_doctor_existed]) if params[:woman_doctor_existed].present? 
hospitals = hospitals.where(emergency_enabled: params[:emergency_enabled]) if params[:emergency_enabled].present? 
hospitals = hospitals.where(checkup_enabled: params[:checkup_enabled]) if params[:checkup_enabled].present? 
hospitals = hospitals.where(dpc_enabled: params[:dpc_enabled]) if params[:dpc_enabled].present? 
hospitals = hospitals.where(parking_enabled: params[:parking_enabled]) if params[:parking_enabled].present? 
hospitals = hospitals.where(card_enabled: params[:card_enabled]) if params[:card_enabled].present? 
hospitals = hospitals.where(newest_medicine_enabled: params[:newest_medicine_enabled]) if params[:newest_medicine_enabled].present? 
hospitals = hospitals.page(params[:page]) 
hospitals 

но встречались некоторые ошибки при запуске rubocop этот файл

  • Назначение Branch условие для размера поиска слишком высока ,
  • Cyclomatic CompLexity для поиска слишком высока.
  • Метод Слишком много строк. [14/10]
  • воспринимаемого СЛОЖНОСТИ для поиска слишком высок

Есть ли решение, чтобы решить эту проблему? Спасибо за чтение и извините за мой английский не очень хорошо

ответ

0

Это говорит о том, что вы делаете много вещей одним способом. Вы можете попытаться разбить его на более мелкие методы. Другая идея - удалить часть дублирования. Например, вы часто добавляете предложение WHERE, если присутствует какой-то параметр. Рассмотрим примерно следующее:

%i(woman_doctor_existed emergency_enabled checkup_enabled more...).each do |key| 
    hospitals = hospitals.where(key => params[key]) if params.has? key 
end 
Смежные вопросы