chore: refactor code so we use a hash instead of a switch statement for long-term performance

speedup-admin-data
Nelson Jovel 2 years ago
parent 1e43e288b1
commit e203be056a

@ -6,17 +6,6 @@ class Income < ApplicationRecord
friendly_id :designation, use: [:slugged]
def label
case designation
when "Economically Disadvantaged - Y"
"Economically Disadvantaged"
when "Economically Disadvantaged - N"
"Not Economically Disadvantaged"
when "Unknown"
"Unknown"
end
end
def self.to_designation(income)
case income
in /Free\s*Lunch|Reduced\s*Lunch|Low\s*Income|Reduced\s*price\s*lunch/i
@ -27,4 +16,14 @@ class Income < ApplicationRecord
"Unknown"
end
end
LABELS = {
"Economically Disadvantaged - Y" => "Economically Disadvantaged",
"Economically Disadvantaged - N" => "Not Economically Disadvantaged",
"Unknown" => "Unknown"
}
def label
LABELS[designation]
end
end

Loading…
Cancel
Save