From c56252810a9d4931b29695d6cce2dd73ec8bfba6 Mon Sep 17 00:00:00 2001 From: Andy Allan Date: Mon, 9 Jul 2012 16:30:39 +0100 Subject: [PATCH] Refactor region detection to avoid duplicate regions in first loop. --- run_regions.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/run_regions.rb b/run_regions.rb index 3a7166a..401bc82 100755 --- a/run_regions.rb +++ b/run_regions.rb @@ -24,8 +24,10 @@ @conn.exec("truncate table regions") def add_region(lat, lon) - @conn.exec("insert into regions (lat, lon) values ($1, $2)", [lat, lon]) - @regions[[lat, lon]] = true + unless @regions.key?([lat, lon]) + @conn.exec("insert into regions (lat, lon) values ($1, $2)", [lat, lon]) + @regions[[lat, lon]] = true + end end @conn.transaction do @@ -48,9 +50,7 @@ def add_region(lat, lon) # Add remaining regions that weren't covered by the bounds (-180..179).each do |lon| (-90..89).each do |lat| - unless @regions.key?([lat, lon]) - add_region(lat, lon) - end + add_region(lat, lon) end end end