配列は、リーク

Question

原因 - （無効）setUserFilters {

//init the user filters array 
    userFilters = [[NSMutableArray alloc] init]; 
    SearchCriteria *tmpSc= [[SearchCriteria alloc] init]; 
    for(int i=0;i<[searchFilters count];i++) 
    { 
    tmpSc=[self.searchFilters objectAtIndex:i]; 
    if(tmpSc.enabled==TRUE) 
     [userFilters addObject:tmpSc]; 
    } 
    } 

searchFiltersをtrueまたはfalseにsettedと私だけなフィルタとテーブルビューを移入するためにuserFiltersを使用することができるフィルタのリストです。 TRUEに設定されています

しかし、行SearchCriteria * tmpSc = [[SearchCriteria alloc] init];私は関数の終わりに私のポインタを緩めるとそれがクラッシュするので、リークを引き起こすと私は解決する方法がわからない

すべてのアイデア？

Answer 1

twolfe18は、searchFiltersが大きくなる可能性がある場合は、コードを>非常に遅くしました。 -objectAtIndex:は大容量アレイでは高速動作ではありませんので、必要以上に行うべきではありません。 （FEはobjectAtIndex：よりも高速ですが、これは問題を誇張していますので、私はそれを打ち破っています; Fast Enumerationの利点に関する他のコメントを参照してください）

コードにはいくつかの問題があります。

• 「設定」を開始するがアクセサーではないメソッドを作成しないでください。 Objective-CがKey-Value準拠を提供する方法により、これは非常に驚くべきバグにつながります。名前は重要です。 userFiltersという名前のプロパティは、-userFiltersというゲッターと-setUserFilters:というセッターを持つ必要があります。セッターはゲッターが返すのと同じ型を取るべきです。だから、この方法は、この問題を避けるため（そしてそれが何をするかをより正確に示すために）、-updateUserFiltersといいます。

• 常にアクセサを使用してください。彼らはあなたにあらゆる種類のメモリ管理の問題を救います。 -setUserFiltersが2回呼び出されると、現在のコードが配列全体をリークします。

• 両方のコメントは、ここで一時的に割り当てる必要はありません。実際には、Fast Enumerationを使用することをお勧めします。Fast Enumerationは非常に速く、非常にメモリ効率が良い（コード作成が簡単です）。

  @interface MyObject() 
  @property (nonatomic, readwrite, retain) NSMutableArray *userFilters; 
  @property (nonatomic, readwrite, retain) NSMutableArray *searchFilters; 
  @end 
  
  @implementation MyObject 
  @synthesize userFilters; 
  @synthesize searchFilters; 
  
  - (void)dealloc 
  { 
      [searchFilters release]; 
      serachFilters = nil; 
      [userFilters release]; 
      userFilters = nil; 
      [super dealloc]; 
  } 
  
  
  - (void)updateUserFilters 
  { 
      //init the user filters array 
      // The accessor will retain for us and will release the old value if 
      // we're called a second time 
      self.userFilters = [NSMutableArray array]; 
  
      // This is Fast Enumeration 
      for (SearchCriteria *sc in self.searchFilters) 
      { 
       if(sc.enabled) 
       { 
        [self.userFilters addObject:sc]; 
       } 
      } 
  } 
  

  ：すべて一緒にそれを引っ張る
  は

、ここであなたは（他の多くの優れたソリューションがありますが、それを行うには、少なくとも一つの方法を、しかし、この1つは理解することは非常に簡単です）やって欲しいものです

Answer 2

まず、書くことができる最悪のn00bコードはif(condition==true) do_something()です。ちょうどif(condition) do_something()と書いてください。

-(void)setUserFilters{ 

//init the user filters array 
userFilters = [[NSMutableArray alloc] init]; 
for(int i=0;i<[searchFilters count];i++) 
{ 
    if([self.searchFilters objectAtIndex:i].enabled) 
      [userFilters addObject:[self.searchFilters objectAtIndex:i]]; 
} 
} 

Answer 3

あなたが最初にあなたの前にSearchCriteriaオブジェクトを作成しているようだ：

二つ目は、あなたが次の操作を行うだけでできる、（それのためのallocメモリを気にしない）まったくtempScを持っている理由はありませんそれを使用するか、self.searchFiltersの別のオブジェクトに変数を再割り当てします。したがって、最初のオブジェクトを作成する必要はなく、なぜそれが漏れていて、解放されていないのですか。

試してみてください。役立ちます

SearchCriteria *tmpSc = nil; 

希望。