From 9c067604aedcead2184c780711d466dd911b16e1 Mon Sep 17 00:00:00 2001
From: Robin Wood <robin@digi.ninja>
Date: Sun, 8 Aug 2021 22:01:23 +0100
Subject: [PATCH] frequency ordering days and months

---
 checkers_available/date_checker.rb | 83 +++++++++++++++++++++++-------
 1 file changed, 65 insertions(+), 18 deletions(-)

diff --git a/checkers_available/date_checker.rb b/checkers_available/date_checker.rb
index 2b521a1..49f8687 100644
--- a/checkers_available/date_checker.rb
+++ b/checkers_available/date_checker.rb
@@ -51,6 +51,17 @@ def process_word (word, extras = nil)
 		@total_words_processed += 1
 	end
 
+	def freq_sort(data)
+		count_ordered = []
+		data.each_pair do |val, count|
+			count_ordered << [val, count] unless count == 0
+		end
+		freqs_sorted = count_ordered.sort do |x,y|
+			(x[1] <=> y[1]) * -1
+		end
+        return freqs_sorted
+	end
+
 	def get_results()
 		ret_str = "Dates\n"
 
@@ -63,16 +74,10 @@ def get_results()
 			end
 		end
 		unless disp
-			ret_str = "None found\n"
-		end
-
-		count_ordered = []
-		@months.each_pair do |month, count|
-			count_ordered << [month, count] unless count == 0
-		end
-		months_sorted = count_ordered.sort do |x,y|
-			(x[1] <=> y[1]) * -1
+			ret_str << "None found\n"
 		end
+		
+        months_sorted = freq_sort(@months)
 
 		ret_str << "\nMonths (Frequency ordered)\n"
 		disp = false
@@ -85,7 +90,7 @@ def get_results()
 			end
 		end
 		unless disp
-			ret_str = "None found\n"
+			ret_str << "None found\n"
 		end
 
 		ret_str << "\nDays\n"
@@ -100,6 +105,22 @@ def get_results()
 			ret_str << "None found\n"
 		end
 
+        days_sorted = freq_sort(@days)
+
+		ret_str << "\nDays (Frequency ordered)\n"
+		disp = false
+		days_sorted.each do |data|
+			day = data[0]
+			count = data[1]
+			unless count == 0
+				disp = true
+				ret_str << "#{day} = #{count.to_s} (#{((count.to_f/@total_words_processed) * 100).round(2).to_s}%)\n" unless count == 0
+			end
+		end
+		unless disp
+			ret_str << "None found\n"
+		end
+
 		ret_str << "\nMonths (Abreviated)\n"
 		disp = false
 		@months_ab.each_pair do |month, count|
@@ -112,6 +133,22 @@ def get_results()
 			ret_str << "None found\n"
 		end
 
+		months_ab_sorted = freq_sort(@months_ab)
+
+		ret_str << "\nMonths (Abreviated) (Frequency ordered)\n"
+		disp = false
+		months_ab_sorted.each do |data|
+			month = data[0]
+			count = data[1]
+			unless count == 0
+				disp = true
+				ret_str << "#{month} = #{count.to_s} (#{((count.to_f/@total_words_processed) * 100).round(2).to_s}%)\n" unless count == 0
+			end
+		end
+		unless disp
+			ret_str << "None found\n"
+		end
+
 		ret_str << "\nDays (Abreviated)\n"
 		disp = false
 		@days_ab.each_pair do |day, count|
@@ -124,6 +161,22 @@ def get_results()
 			ret_str << "None found\n"
 		end
 
+		days_ab_sorted = freq_sort(@days_ab)
+
+		ret_str << "\nDays (Abreviated) (Frequency ordered)\n"
+		disp = false
+		days_ab_sorted.each do |data|
+			day = data[0]
+			count = data[1]
+			unless count == 0
+				disp = true
+				ret_str << "#{day} = #{count.to_s} (#{((count.to_f/@total_words_processed) * 100).round(2).to_s} %)\n" unless count == 0
+			end
+		end
+		unless disp
+			ret_str << "None found\n"
+		end
+
 		ret_str << "\nIncludes years\n"
 		disp = false
 		@years.each_pair do |number, count|
@@ -136,17 +189,11 @@ def get_results()
 			ret_str << "None found\n"
 		end
 
-		count_ordered = []
-		@years.each_pair do |year, count|
-			count_ordered << [year, count] unless count == 0
-		end
-		@years = count_ordered.sort do |x,y|
-			(x[1] <=> y[1]) * -1
-		end
+		years_freq_sort = freq_sort(@years)
 
 		ret_str << "\nYears (Top #{@cap_at.to_s})\n"
 		disp = false
-		@years[0, @cap_at].each do |data|
+		years_freq_sort[0, @cap_at].each do |data|
 			disp = true
 			ret_str << "#{data[0].to_s} = #{data[1].to_s} (#{((data[1].to_f/@total_words_processed) * 100).round(2).to_s}%)\n"
 		end