Instruments Leaksは存在しないメソッド呼び出しを示します

Question

Instrument Leaksツールをまったく理解していないか、怒っています。私は、私のiPhoneアプリでツールを実行して、それはいくつかのリークを示しています。私が正しく理解していれば、リークの1つとして、メソッド "writeHeading"によって割り当てられたNSDateオブジェクトであると言います。オブジェクトを割り当てるメソッドは、 "dateWithTimeIntervalSinceReferenceDate："です。しかし、私のwriteHeadingメソッドはそのメソッドを使用しません。実際、この方法は私のアプリケーション全体では使用されていません。

誰もが何が起こっているのか考えていますか？ここで

はwriteHeadingのコードです：ここで

- (void) writeHeading:(CLHeading *)heading 
{ 
    if (self.inFlight) { 
     [log writeHeading:heading]; 
    } else { 
     IGC_Event *event = [[IGC_Event alloc] init]; 
     event.code = 'K'; 
     event.timestamp = heading.timestamp;  
     event.heading = heading; 
     [self addEvent:event]; 
     [event release]; 
    } 
} 

は、楽器のスクリーンショットです：

そして、ここではIGC_Event（複数応答で尋ねたとして）の定義である：

@interface IGC_Event : NSObject { 
    int code; 
    CLLocation *location; 
    CLHeading *heading; 
    NSString *other; 
    NSDate *timestamp; 
} 

@property int code; 
@property (nonatomic, retain) CLLocation *location; 
@property (nonatomic, retain) CLHeading *heading; 
@property (nonatomic, retain) NSString *other; 
@property (nonatomic, retain) NSDate *timestamp; 

@end 


@implementation IGC_Event 

@synthesize code; 
@synthesize location; 
@synthesize heading; 
@synthesize other; 
@synthesize timestamp; 

@end 

Answer 1

ARCがないと仮定すると、IGC_Eventオブジェクトがタイムスタンプや保持またはコピーされた可能性のある他の参照を解放するようにする必要があります。あなたはそれをリリースしているべき場所

- (void) dealloc { 

    [timestamp release]; 
    [location release]; 
    [heading release]; 
    [other release]; 


    [super dealloc]; 
} 

リークだけではない、そのタイムスタンプオブジェクトが作成されたあなたを語っている：

だからIGC_Eventにあなたはこのようにdealloc何かを必要としています。

これはもちろん、あなたが漏れている唯一の場所ではないかもしれませんが、そこには4つの潜在的な漏れがあります。

Answer 2

IGC_Eventクラスの実装がありますか？そのtimestampプロパティのセッターがdateWithTimeIntevalSinceReferenceDate:を呼び出す可能性はありますか？ （これは、タイムスタンプがクラスNSDateであり、サブクラスではなく、渡されたタイムスタンプから独立していることを保証します）

（免責事項：私はObjective-Cの-ERの本当にあまりないんだけど、これは愚かな質問のように思えるなら、それはおそらくです。！）

Answer 3

コンパイラがコードを実行すると、方法がありますあなたのスクリーンショットには小さな人物が横たわっているあなた自身が直接呼び出し、次に結果としてコアフレームワークで呼び出されるメソッドを呼び出します。あなたが望んでいた場合は、このプロセスを自分で管理することができ

event.timestamp = heading.timestamp; 

：コードのこの部分からの質問の結果のメソッドちなみに

NSDate *eventTimestamp = heading.timestamp; 
event.timestamp = eventTimestamp; 

は、そのタイムスタンプを格納することは、完全に冗長であり、以来、不要なメモリを使用しています見出しはevent.headingにすべてのプロパティと共に格納するので、いつでもevent.heading.timestampでそのタイムスタンプにアクセスできます。しかし、それを別々に保存する理由があるかもしれません。