Merge pull request #141 from glangford/getFix

Use associative array rather than linear search to find airport fixes
zlsa · Aug 20, 2015 · a916c90 · a916c90
2 parents 3c8a7b7 + d86bdb0
commit a916c90
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/assets/scripts/airport.js b/assets/scripts/airport.js
@@ -207,7 +207,7 @@ var Airport=Fiber.extend(function() {
       if(data.fixes) {
         for(var i in data.fixes) {
           var coord = new Position(data.fixes[i], this.position);
-          this.fixes[i] = coord.position;
+          this.fixes[i.toUpperCase()] = coord.position;
         }
       }
 
@@ -372,11 +372,7 @@ var Airport=Fiber.extend(function() {
     },
     getFix: function(name) {
       if(!name) return null;
-      name = name.toLowerCase();
-      for(var i in this.fixes) {
-        if(i.toLowerCase() == name) return this.fixes[i];
-      }
-      return null;
+      return this.fixes[name.toUpperCase()] || null;
     },
     getRunway: function(name) {
       if(!name) return null;