배열 누출

Question

원인 TRUE로 설정 됨

그러나 SearchCriteria * tmpSc = [[SearchCriteria alloc] init]; 누수가 발생하고 해결 방법을 모르겠다. 함수의 끝에서 놓으면 포인터가 풀려서 충돌한다.

아이디어가 있으십니까?

Answer 1

검색 필터가 큰 경우 twolfe18에서 코드를 훨씬 느리게 만듭니다. -objectAtIndex:은 대용량 어레이에서 빠른 작업이 아니므로 사용자가해야하는 것보다 더 많이 수행하면 안됩니다. (사실 FE가 objectAtIndex :보다 빠르다는 사실은 문제를 과장하여 파탄 열심히하기의 장점에 대한 저의 다른 의견을 참조하십시오.)

코드에 여러 가지 문제가 있습니다.

• "설정"을 시작하지만 접근자가 아닌 메서드는 생성하지 마십시오. Objective-C가 Key-Value Compliance를 제공하는 방법 때문에 매우 놀라운 버그가 발생할 수 있습니다. 이름이 중요합니다. userFilters이라는 속성에는 -userFilters이라는 게터와 -setUserFilters:이라는 세터가 있어야합니다. setter는 getter가 반환하는 것과 동일한 유형을 사용해야합니다. 따라서이 방법을 사용하면 더 이상 -updateUserFilters이라고 불리는 것이 좋으며 무엇을하는지 더 정확하게 나타낼 수 있습니다.

• 항상 접근자를 사용하십시오. 그들은 모든 종류의 메모리 관리 문제를 저장합니다. -setUserFilters을 두 번 호출하면 현재 코드가 전체 배열을 누설합니다.

• 여기에 임시로 할당 할 필요가없는 두 가지 의견이 모두 맞습니다. 실제로 가장 좋은 해결책은 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]; 
       } 
      } 
  } 
  

  : 모두 함께 이겠지
  는

, 여기 당신이 (다른 많은 좋은 솔루션이있다, 그것을 할 방법을 하나 이상하지만,이 일을 이해하는 것은 매우 간단하다)하고 싶은거야

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; 

희망.