0

クリックされたときにNSColorPanelを表示する次のコードを持つカスタムNSCellがあります。

-(void)setColorFromPanel:(NSColorPanel*)panel{
 NSLog(@"COLOR is HERE!");
 [self setObjectValue:[panel color]];
}

- (NSUInteger)hitTestForEvent:(NSEvent *)event inRect:(NSRect)cellFrame ofView:(NSView *)controlView{
 if(self.isColor){
  if([event type]==NSLeftMouseDown){
   NSColorPanel *panel=[NSColorPanel sharedColorPanel];
   [panel setColor:[self objectValue]];
   [panel setShowsAlpha:YES];
   [panel setAction:@selector(setColorFromPanel:)];
   [panel setTarget:self];
   [panel makeKeyAndOrderFront:nil];
  }
  return NSCellHitContentArea;
 }
    return NSCellHitNone;
}

このコードはカラーピッカーを表示しますが、色をクリックするとクラッシュします。[panel setTarget：self]行を削除すると、効果はありませんが、正常に機能します（ターゲットがないため、カラー値を受け取らないため）。

これがスタックトレースです。エラーはEXC_BAD_ACCESSです。

#0  0x00007fff8667811c in objc_msgSend ()
#1  0x00007fff87081e9a in -[NSApplication sendAction:to:from:] ()
#2  0x00007fff871fa1cd in -[NSColorPanel _forceSendAction:notification:firstResponder:] ()
#3  0x00007fff871fe384 in -[NSColorPanel setColor:] ()
#4  0x00007fff8721d112 in -[NSColorPickerWheel setColor:] ()
#5  0x00007fff8721d5ae in -[NSColorPickerWheel brightnessSlider:] ()
#6  0x00007fff87081e9a in -[NSApplication sendAction:to:from:] ()
#7  0x00007fff87081df9 in -[NSControl sendAction:to:] ()
#8  0x00007fff8710d400 in -[NSCell trackMouse:inRect:ofView:untilMouseUp:] ()
#9  0x00007fff873eaf01 in -[NSSliderCell trackMouse:inRect:ofView:untilMouseUp:] ()
#10 0x00007fff8710c215 in -[NSControl mouseDown:] ()
#11 0x00007fff8702634f in -[NSWindow sendEvent:] ()
#12 0x00007fff86f5ba86 in -[NSApplication sendEvent:] ()
#13 0x00007fff86ef24da in -[NSApplication run] ()
#14 0x00007fff86eeb1a8 in NSApplicationMain ()
#15 0x00000001000029bb in main (argc=1, argv=0x7fff5fbff6a0)
4

1 に答える 1

0

クラッシュレポート/デバッガー出力の関連部分を投稿する必要があります。そうでなければ、何が悪かったのかを正確に推測するのは難しいです！私たちは超能力者ではありません。クラッシュレポートには、クラッシュした理由に関する貴重な情報が含まれています。

とは言うものの、ここにはオープンソースのカラーウェルセルがあります。これはGC対応アプリ専用ですが、そのコードのライセンスが使用法に適合している場合は、構築を開始するのに適しています。車輪の再発明をしないでください。

いくつか単語を追加させてください。NSCellin anは、各行のエントリを描画するためのスタンプNSTableViewとして使用され、通常は再利用されます。各行のanが生成されるわけではありません。これは、システムに豊富なメモリがある現在の観点からはあまり意味がありませんが、NeXTStepが最初に設計された15年前には意味がありました。NSCell

とにかく、この理由のために、そのセルは持続しない傾向があるので、セルがそれ自体を別のオブジェクトのターゲットとして設定することは通常良い考えではありません。これがこのクラッシュの原因だと思いNSTableColumnます。つまり、特定のセルの使用が終了したため、すでにセルを解放していました。一般に、セル自体ではなく、パネルのターゲットとしてコントローラークラスを使用する方が安全です。

于 2011-01-21T02:26:19.433 に答える