]> git.mxchange.org Git - flightgear.git/blobdiff - src/Airports/simple.cxx
Another clean-up iteration: FGAirportList::search is gone, replaced by two
[flightgear.git] / src / Airports / simple.cxx
index 1dc6b7d76ac875674551419f879413d2898657c8..af1e0de195e04eb42219494a52aaed80ce140e6d 100644 (file)
@@ -52,8 +52,8 @@
 using std::sort;
 using std::random_shuffle;
 
-
-
+// magic import of a helper which uses FGPositioned internals
+extern char** searchAirportNamesAndIdents(const std::string& aFilter);
 
 /***************************************************************************
  * FGAirport
@@ -200,13 +200,8 @@ bool FGAirport::hasHardRunwayOfLengthFt(double aLengthFt) const
     if (rwy->isReciprocal()) {
       continue; // we only care about lengths, so don't do work twice
     }
-    
-    int surface = rwy->surface();
-    if ((surface != 1) && (surface != 2)) {
-      continue; // no hard surface
-    }
-    
-    if (rwy->lengthFt() >= aLengthFt) {
+
+    if (rwy->isHardSurface() && (rwy->lengthFt() >= aLengthFt)) {
       return true; // we're done!
     }
   } // of runways iteration
@@ -284,32 +279,41 @@ bool FGAirport::HardSurfaceFilter::pass(FGPositioned* aPos) const
   return static_cast<FGAirport*>(aPos)->hasHardRunwayOfLengthFt(mMinLengthFt);
 }
 
+FGAirport* FGAirport::findByIdent(const std::string& aIdent)
+{
+  FGPositionedRef r;
+  AirportFilter filter;
+  r = FGPositioned::findNextWithPartialId(r, aIdent, &filter);
+  if (!r) {
+    return NULL; // we don't warn here, let the caller do that
+  }
+  return static_cast<FGAirport*>(r.ptr());
+}
+
+FGAirport* FGAirport::getByIdent(const std::string& aIdent)
+{
+  FGPositionedRef r;
+  AirportFilter filter;
+  r = FGPositioned::findNextWithPartialId(r, aIdent, &filter);
+  if (!r) {
+    throw sg_range_exception("No such airport with ident: " + aIdent);
+  }
+  return static_cast<FGAirport*>(r.ptr());
+}
+
+char** FGAirport::searchNamesAndIdents(const std::string& aFilter)
+{
+  // we delegate all the work to a horrible helper in FGPositioned, which can
+  // access the (private) index data.
+  return searchAirportNamesAndIdents(aFilter);
+}
+
 /******************************************************************************
  * FGAirportList
  *****************************************************************************/
 
-// Populates a list of subdirectories of $FG_ROOT/Airports/AI so that
-// the add() method doesn't have to try opening 2 XML files in each of
-// thousands of non-existent directories.  FIXME: should probably add
-// code to free this list after parsing of apt.dat is finished;
-// non-issue at the moment, however, as there are no AI subdirectories
-// in the base package.
-//
-// Note: 2005/12/23: This is probably not necessary anymore, because I'm
-// Switching to runtime airport dynamics loading (DT).
 FGAirportList::FGAirportList()
 {
-//     ulDir* d;
-//     ulDirEnt* dent;
-//     SGPath aid( globals->get_fg_root() );
-//     aid.append( "/Airports/AI" );
-//     if((d = ulOpenDir(aid.c_str())) == NULL)
-//         return;
-//     while((dent = ulReadDir(d)) != NULL) {
-//         SG_LOG( SG_GENERAL, SG_DEBUG, "Dent: " << dent->d_name );
-//         ai_dirs.insert(dent->d_name);
-//     }
-//     ulCloseDir(d);
 }
 
 
@@ -326,105 +330,11 @@ FGAirport* FGAirportList::add( const string &id, const SGGeod& location, const S
                          const string &name, bool has_metar, FGPositioned::Type aType)
 {
     FGAirport* a = new FGAirport(id, location, tower_location, name, has_metar, aType);
-    airports_by_id[a->getId()] = a;
     // try and read in an auxilary file
-
     airports_array.push_back( a );
-    SG_LOG( SG_GENERAL, SG_BULK, "Adding " << id << " pos = " << location.getLongitudeDeg()
-            << ", " << location.getLatitudeDeg() << " elev = " << location.getElevationFt() );
-            
     return a;
 }
 
-
-// search for the specified id
-FGAirport* FGAirportList::search( const string& id)
-{
-    airport_map_iterator itr = airports_by_id.find(id);
-    return (itr == airports_by_id.end() ? NULL : itr->second);
-}
-
-// wrap an FGIdentOrdering in an STL-compatible functor. not the most
-// efficent / pretty thing in the world, but avoids template nastiness in the 
-// headers, and we're only doing O(log(N)) comparisoms per search
-class orderingFunctor
-{
-public:
-  orderingFunctor(FGIdentOrdering* aOrder) :
-    mOrdering(aOrder)
-  { assert(aOrder); }
-  
-  bool operator()(const airport_map::value_type& aA, const std::string& aB) const
-  {
-    return mOrdering->compare(aA.first,aB);
-  }
-
-  bool operator()(const std::string& aA, const airport_map::value_type& aB) const
-  {
-    return mOrdering->compare(aA, aB.first);
-  }
-
-  bool operator()(const airport_map::value_type& aA, const airport_map::value_type& aB) const
-  {
-    return mOrdering->compare(aA.first, aB.first);
-  }
-  
-private:
-  FGIdentOrdering* mOrdering;
-};
-
-const FGAirport* FGAirportList::findFirstById(const std::string& aIdent, FGIdentOrdering* aOrder)
-{
-  airport_map_iterator itr;
-  if (aOrder) {
-    orderingFunctor func(aOrder);
-    itr = std::lower_bound(airports_by_id.begin(),airports_by_id.end(), aIdent, func);
-  } else {
-    itr = airports_by_id.lower_bound(aIdent);
-  }
-  
-  if (itr == airports_by_id.end()) {
-    return NULL;
-  }
-  
-  return itr->second;
-}
-
-// search for the airport nearest the specified position
-FGAirport* FGAirportList::search(double lon_deg, double lat_deg, double max_range)
-{
-    static FGAirportSearchFilter accept_any;
-    return search(lon_deg, lat_deg, max_range, accept_any);
-}
-
-
-// search for the airport nearest the specified position and
-// passing the filter
-FGAirport* FGAirportList::search(double lon_deg, double lat_deg,
-        double max_range,
-        FGAirportSearchFilter& filter)
-{
-    double min_dist = max_range;
-
-    airport_list_iterator it = airports_array.begin();
-    airport_list_iterator end = airports_array.end();
-    airport_list_iterator closest = end;
-    for (; it != end; ++it) {
-        if (!filter.pass(*it))
-            continue;
-
-        // crude manhatten distance based on lat/lon difference
-        double d = fabs(lon_deg - (*it)->getLongitude())
-                + fabs(lat_deg - (*it)->getLatitude());
-        if (d < min_dist) {
-            closest = it;
-            min_dist = d;
-        }
-    }
-    return closest != end ? *closest : 0;
-}
-
-
 int
 FGAirportList::size () const
 {
@@ -441,52 +351,14 @@ const FGAirport *FGAirportList::getAirport( unsigned int index ) const
     }
 }
 
-
-/**
- * Mark the specified airport record as not having metar
- */
-void FGAirportList::no_metar( const string &id )
-{
-    if(airports_by_id.find(id) != airports_by_id.end()) {
-        airports_by_id[id]->setMetar(false);
-    }
-}
-
-
-/**
- * Mark the specified airport record as (yes) having metar
- */
-void FGAirportList::has_metar( const string &id )
-{
-    if(airports_by_id.find(id) != airports_by_id.end()) {
-        airports_by_id[id]->setMetar(true);
-    }
-}
-
-
 // find basic airport location info from airport database
 const FGAirport *fgFindAirportID( const string& id)
 {
-    const FGAirport* result = NULL;
-    if ( id.length() ) {
-        SG_LOG( SG_GENERAL, SG_BULK, "Searching for airport code = " << id );
-
-        result = globals->get_airports()->search( id );
-
-        if ( result == NULL ) {
-            SG_LOG( SG_GENERAL, SG_ALERT,
-                    "Failed to find " << id << " in apt.dat.gz" );
-            return NULL;
-        }
-    } else {
+    if ( id.empty() ) {
         return NULL;
     }
-    SG_LOG( SG_GENERAL, SG_BULK,
-            "Position for " << id << " is ("
-            << result->getLongitude() << ", "
-            << result->getLatitude() << ")" );
-
-    return result;
+    
+    return FGAirport::findByIdent(id);
 }