Changeset 669 in dev

Timestamp:

Jun 28, 2008, 8:17:17 PM (16 years ago)

Author:

dai_9181

Message:

NamespaceScopes::IsEqualを高速化した（他、リファクタリング）。

Location:

trunk/ab5.0/abdev/ab_common

Files:

• include/Lexical/Namespace.h (modified) (2 diffs)
• include/Lexical/NamespaceSupporter.h (modified) (1 diff)
• src/Lexical/Namespace.cpp (modified) (1 diff)
• src/Lexical/NamespaceSupporter.cpp (modified) (1 diff)

trunk/ab5.0/abdev/ab_common/include/Lexical/Namespace.h

@@ -20 +20 @@
     NamespaceScopes( const std::string &namespaceStr );
     ~NamespaceScopes(){}
+
+    NamespaceScopes operator+ ( const NamespaceScopes &namespaceScopes ) const;
 
     std::string ToString() const
@@ -50 +52 @@
 
     // 等しいかをチェック
-    bool IsEqual( const NamespaceScopes &namespaceScopes ) const
-    {
-        if( ToString() == namespaceScopes.ToString() ){
-            return true;
-        }
-        return false;
-    }
+    bool IsEqual( const NamespaceScopes &namespaceScopes ) const;
 
     // 所属しているかをチェック

trunk/ab5.0/abdev/ab_common/include/Lexical/NamespaceSupporter.h

@@ -90 +90 @@
 
     // 指定された名前空間が同一エリアと見なされるかどうかをチェック
-    bool IsSameAreaNamespace( const NamespaceScopes &baseNamespaceScopes, const NamespaceScopes &entryNamespaceScopes ) const
-    {
-        if( entryNamespaceScopes.size() )
-        {
-            if( IsCoverd( baseNamespaceScopes, entryNamespaceScopes ) )
-            {
-                // 包括しているときは同一と見なす
-                return true;
-            }
-        }
-        else{
-            if( baseNamespaceScopes.size() )
-            {
-                // 名前空間の判断が必要なとき
-                if( this->importedNamespaces.IsImported( baseNamespaceScopes )
-                    || IsLiving( baseNamespaceScopes ) )
-                {
-                    // Using指定があるとき
-                    // または
-                    // 指定された名前空間が現在の名前空間スコープと同一のとき
-                    return true;
-                }
-            }
-            else{
-                return true;
-            }
-        }
-
-        return false;
-    }
+    bool IsSameAreaNamespace( const NamespaceScopes &baseNamespaceScopes, const NamespaceScopes &entryNamespaceScopes ) const;
 };
 

trunk/ab5.0/abdev/ab_common/src/Lexical/Namespace.cpp

@@ -24 +24 @@
         i = i2 + 1;
     }
+}
+
+NamespaceScopes NamespaceScopes::operator+ ( const NamespaceScopes &namespaceScopes ) const
+{
+    NamespaceScopes result = *this;
+    BOOST_FOREACH( const std::string &name, namespaceScopes )
+    {
+        result.push_back( name );
+    }
+    return result;
+}
+
+bool NamespaceScopes::IsEqual( const NamespaceScopes &namespaceScopes ) const
+{
+    if( this->size() != namespaceScopes.size() )
+    {
+        return false;
+    }
+
+    for( int i=static_cast<int>(this->size()-1); i>=0; i-- )
+    {
+        if( (*this)[i] != namespaceScopes[i] )
+        {
+            return false;
+        }
+    }
+
+    return true;
 }
 

trunk/ab5.0/abdev/ab_common/src/Lexical/NamespaceSupporter.cpp

@@ -21 +21 @@
     return ImportsNamespace( namespaceScopes );
 }
+
+bool NamespaceSupporter::IsSameAreaNamespace( const NamespaceScopes &baseNamespaceScopes, const NamespaceScopes &entryNamespaceScopes ) const
+{
+    if( entryNamespaceScopes.size() )
+    {
+        if( IsCoverd( baseNamespaceScopes, entryNamespaceScopes ) )
+        {
+            // 包括しているときは同一と見なす
+            return true;
+        }
+    }
+    else{
+        if( baseNamespaceScopes.size() )
+        {
+            // 名前空間の判断が必要なとき
+            if( this->importedNamespaces.IsImported( baseNamespaceScopes )
+                || IsLiving( baseNamespaceScopes ) )
+            {
+                // Using指定があるとき
+                // または
+                // 指定された名前空間が現在の名前空間スコープと同一のとき
+                return true;
+            }
+        }
+        else{
+            return true;
+        }
+    }
+
+    return false;
+}