Improve performance for NodeTreeBase::set_num_id_name() (#1239)

set_num_id_name()を呼び出すループ処理を修正して効率を高めます。ただし、コンパイラーが最適化を行っている場合は効果は微小です。
JDimproved · Sep 2, 2023 · 72fa66f · 72fa66f
1 parent 3c55283
commit 72fa66f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/src/dbtree/nodetreebase.cpp b/src/dbtree/nodetreebase.cpp
@@ -3905,15 +3905,16 @@ void NodeTreeBase::update_id_name( const int from_number, const int to_number )
     }
 
     //集計したものを元に各ノードの情報を更新
-    for( const auto &a: m_map_id_name_resnumber ){ // ID = a.first, レス番号の一覧 = a.second
-        // a.second の型 std::set はソートされた連想コンテナなので
+    for( const auto& [id_unused, resnumber_set] : m_map_id_name_resnumber ){ // ID, レス番号の一覧
+        const int num_id_name = resnumber_set.size();
+        // resnumber_set の型 std::set はソートされた連想コンテナなので
         // レスの順番計算はインクリメントするだけでできる
         int posting_order = 0;
-        for( const auto &num: a.second ) {
+        for( const auto num : resnumber_set ) {
             NODE* header = res_header( num );
             if( ! header ) continue;
             if( ! header->headinfo->block[ BLOCK_ID_NAME ] ) continue;
-            set_num_id_name( header, a.second.size(), ++posting_order );
+            set_num_id_name( *header->headinfo, num_id_name, ++posting_order );
         }
      }
 }
@@ -3922,19 +3923,18 @@ void NodeTreeBase::update_id_name( const int from_number, const int to_number )
 /** @brief 発言数( num_id_name )と何番目の投稿( posting_order )を更新
  *
  * @details IDノードの色も変更する
- * @param[in] header        レスのヘッダー
+ * @param[in] headinfo      IDのヘッダ拡張情報
  * @param[in] num_id_name   発言数
  * @param[in] posting_order 何番目の投稿
  */
-void NodeTreeBase::set_num_id_name( NODE* header, const int num_id_name, const int posting_order )
+void NodeTreeBase::set_num_id_name( HEADERINFO& headinfo, const int num_id_name, const int posting_order )
 {
-    if( ! header->headinfo->block[ BLOCK_ID_NAME ] ||
-        ! header->headinfo->block[ BLOCK_ID_NAME ]->next_node ) return;
+    NODE* idnode = headinfo.block[ BLOCK_ID_NAME ]->next_node;
+    if( ! idnode ) return;
 
-    header->headinfo->posting_order = posting_order;
-    header->headinfo->num_id_name = num_id_name;
+    headinfo.posting_order = posting_order;
+    headinfo.num_id_name = num_id_name;
 
-    NODE* idnode = header->headinfo->block[ BLOCK_ID_NAME ]->next_node;
     if( num_id_name >= m_num_id[ LINK_HIGH ] ) idnode->color_text = COLOR_CHAR_LINK_ID_HIGH;
     else if( num_id_name >= m_num_id[ LINK_LOW ] ) idnode->color_text = COLOR_CHAR_LINK_ID_LOW;
     else idnode->color_text = COLOR_CHAR;

diff --git a/src/dbtree/nodetreebase.h b/src/dbtree/nodetreebase.h
@@ -400,7 +400,7 @@ namespace DBTREE
 
         // 発言数( num_id_name )と何番目の投稿( posting_order )を更新
         // IDノードの色も変更する
-        void set_num_id_name( NODE* header, const int num_id_name, const int posting_order );
+        void set_num_id_name( HEADERINFO& headinfo, const int num_id_name, const int posting_order );
 
 
         // from_number番から to_number 番までのレスのフォント判定を更新