Project

General

Profile

Actions

Enhancement(機能追加・改善) #1197

closed

[optimization] MemberRelationshipTable::getFriendMemberIds() などで MemberTable::getInactiveMemberIds() の結果を in_array() するのをやめる

Added by Yuki Kan over 14 years ago. Updated over 14 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2010-06-23
Due date:
2010-06-23
% Done:

100%

Estimated time:

Description

Overview

MemberRelationshipTable::getFriendMemberIds() などで MemberTable::getInactiveMemberIds() の結果を in_array() するのをやめる

具体的には以下のような変更になる。

Index: MemberTable.class.php
===================================================================
--- MemberTable.class.php    (revision 40763)
+++ MemberTable.class.php    (revision 40764)
@@ -65,7 +65,7 @@
     $memberIds = array();
     foreach ($members as $member)
     {
-      $memberIds[] = $member[0];
+      $memberIds[$member[0]] = $member[0];
     }

     return $memberIds;

Index: MemberRelationshipTable.class.php
===================================================================
--- MemberRelationshipTable.class.php    (revision 40763)
+++ MemberRelationshipTable.class.php    (revision 40764)
@@ -67,7 +67,7 @@

     foreach ($friendMemberIds as $friend)
     {
-      if (!in_array($friend[0], $inactiveMemberIds))
+      if (!isset($inactiveMemberIds[$friend[0]]))
       {
         $result[] = $friend[0];
       }
Actions #1

Updated by Kousuke Ebihara over 14 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 484780ed0b7f4c26b5a0318e79276d6f16e4121e で適用されました。

Actions #2

Updated by Yuki Kan over 14 years ago

更新履歴 482681b600fe4d4348af8c80160796b5a2142457 で適用されました。

Actions #3

Updated by Yuki Kan over 14 years ago

更新履歴 31efafc427cd509d0681b7a65fe1d36f78be9bb3 で適用されました。

Actions #4

Updated by Shogo Kawahara over 14 years ago

  • Subject changed from 【チューニング】MemberRelationshipTable::getFriendMemberIds() などで MemberTable::getInactiveMemberIds() の結果を in_array() するのをやめる to [optimization] MemberRelationshipTable::getFriendMemberIds() などで MemberTable::getInactiveMemberIds() の結果を in_array() するのをやめる
Actions #5

Updated by Shogo Kawahara over 14 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70
Actions #6

Updated by Kousuke Ebihara over 14 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100
Actions

Also available in: Atom PDF