Make split by gender faster

c-busqueda/handler.cr

@@ -42,27 +42,6 @@ class Handler
     html.gsub("gnuplot_canvas", canvas_name)
   end
 
-  def query(sql)
-    # Runs a SQL query against the database.
-    #
-    # Returns an array of values [[Year,Count]...]
-    # Or nil if there are no results
-
-    DB.open("postgres://#{USER}:#{PASS}@10.61.0.1:5432/nombres") do |cursor|
-      cursor.query sql do |result_set|
-        result = [] of Tuple(Int32, Int32)
-        result_set.each do
-          year = result_set.read(Int32)
-          contador = result_set.read(Int32)
-          result.push({year, contador})
-        end
-        return result
-      end
-    end
-    # No result, return nil
-    nil
-  end
-
   def normalize_name(s)
     # Remove diacritics, turn lowercase
     normalized = s.unicode_normalize(:nfkd).chars
@@ -71,13 +50,12 @@ class Handler
     }.join("").downcase
   end
 
-  def feminidad(nombre)
+  def feminidad(cursor, nombre)
     # Yes this database is upper case
     nombre = nombre.to_s.upcase
     sql1 = %(
       SELECT COALESCE(frecuencia,0)
       FROM mujeres WHERE nombre='#{nombre}'
-
     )
     sql2 = %(
     SELECT COALESCE(frecuencia,0)
@@ -85,13 +63,11 @@ class Handler
     )
 
     hombres = mujeres = 0
-    DB.open("postgres://#{USER}:#{PASS}@10.61.0.1:5432/nombres") do |cursor|
+    cursor.query sql1 do |result|
-      cursor.query sql1 do |result|
+      mujeres = result.read(Int32)
-        mujeres = result.read(Int32)
+    end
-      end
+    cursor.query sql2 do |result|
-      cursor.query sql2 do |result|
+      hombres = result.read(Int32)
-        hombres = result.read(Int32)
-      end
     end
     if hombres == mujeres == 0
       return 0.5
@@ -99,24 +75,24 @@ class Handler
     mujeres / (hombres + mujeres)
   end
 
-  def split_por_genero(nombres)
+  # def split_por_genero(cursor, nombres)
-    femeninos = Array(Tuple(Int32, String)).new
+  #   femeninos = Array(Tuple(Int32, String)).new
-    masculinos = Array(Tuple(Int32, String)).new
+  #   masculinos = Array(Tuple(Int32, String)).new
-    nombres.map { |nombre|
+  #   nombres.map { |nombre|
-      fem = feminidad(nombre[1])
+  #     fem = feminidad(cursor, nombre[1])
-      # El overlap en 0.5 es intencional!
+  #     # El overlap en 0.5 es intencional!
-      if fem >= 0.5
+  #     if fem >= 0.5
-        femeninos << nombre
+  #       femeninos << nombre
-      end
+  #     end
-      if fem <= 0.5
+  #     if fem <= 0.5
-        masculinos << nombre
+  #       masculinos << nombre
-      end
+  #     end
-    }
+  #   }
-    {
+  #   {
-      "f": femeninos,
+  #     "f": femeninos,
-      "m": masculinos,
+  #     "m": masculinos,
-    }
+  #   }
-  end
+  # end
 
   def run(request : HTTP::Request)
     # Try to find most popular names based on a prefix, year and gender.
@@ -198,9 +174,8 @@ class Handler
     puts "QUERY: #{sql}"
 
     datos = [] of Tuple(Int32, String)
-    DB.open("postgres://#{USER}:#{PASS}@10.61.0.1:5432/nombres") do |cursor|
+    DB.open(DB_URL) do |cursor|
       cursor.query sql do |result_set|
-        puts "loop"
         result_set.each do
           valor = result_set.read(Int32)
           nombre = result_set.read(String)
@@ -225,12 +200,38 @@ class Handler
       row[1].to_s.includes? " "
     }
 
-    
     if genero
-      datos = split_por_genero(datos)[genero]
+      DB.open(DB_URL) do |cursor|
-      puts "Data split by gender"
+        datos = datos.find do |item|
+          # How feminine is this name?
+          # Yes this database is upper case
+          nombre = item[1].upcase
+          puts "Checking #{nombre}"
+          feminidad = 0
+          sql = %(
+            SELECT COALESCE((SELECT frecuencia FROM mujeres WHERE nombre='#{nombre}'), 0) AS mujeres,
+                 COALESCE((SELECT frecuencia FROM hombres WHERE nombre='#{nombre}'), 0) AS hombres
+          )
+          cursor.query sql do |result|
+            mujeres = result.read(Int32)
+            hombres = result.read(Int32)
+            if hombres == mujeres == 0
+              feminidad = 0.5
+            else
+              feminidad = mujeres / (hombres + mujeres)
+            end
+          end
+          # El overlap en 0.5 es intencional!
+          if (feminidad >= 0.5 && genero == "f") ||
+             (feminidad <= 0.5 && genero == "m")
+            true
+          else
+            false
+          end
+        end
+        puts "Data split by gender"
+      end
     end
-
     datos = datos[..10]
 
     if datos.size > 1