Làm thế nào để giữ lại setter được triển khai với @synthesize?

Question

Tôi có sau trong tiêu đề:

@property (nonatomic, retain) UIView *overlay; 

Và trong việc thực hiện:

@synthesize overlay; 

Sau đó:

UIView *tempOverlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 
self.overlay = tempOverlay; 
[tempOverlay release]; 

Không phải là biến tempOverlay trên không cần thiết? Tôi không thể chỉ làm:

self.overlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 

Answer 1

Một setter giữ lại tổng hợp trông giống như:

- (void)setValue: (id)newValue 
{ 
    if (value != newValue) 
    { 
     [value release]; 
     value = newValue; 
     [value retain]; 
    } 
} 

Trong trường hợp của bạn, bạn có hai phương pháp hợp lệ:

1) Tạo một temp var, alloc/init (= giữ lại), thiết lập để sở hữu , giải phóng.

IView *tempOverlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 
self.overlay = tempOverlay; 
[tempOverlay release]; 

2) Không có temp var, được đặt trực tiếp thành ivar.

overlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 

UPDATE: Nếu bạn sử dụng phương pháp 2), bạn phải xử lý một cách rõ ràng phần còn lại của quản lý bộ nhớ (không chỉ giữ lại), bằng cách phát hành bất kỳ giá trị trước đó nó có thể có trước nếu cần thiết. Nếu chỉ thực hiện một lần trong init (ví dụ), bạn có thể chỉ cần đặt [overlay release]; trong dealloc.

Answer 2

Sử dụng thuộc tính retain xác định rằng retain nên được gọi trên đối tượng mới, và các giá trị trước đó được gửi một release.

Vì vậy, trong khối mã thứ hai của bạn, số lượng giữ lại của đối tượng sẽ trở thành 2, vì bạn không còn phát hành nó nữa và setter đang giữ lại nó. Đây không phải là điều bạn muốn.

Answer 3

Có, bạn có thể trực tiếp gán đối tượng mới được tạo cho đối tượng overlay. Nếu bạn muốn, bạn có thể chứng minh điều đó cho chính mình bằng cách in ra các số giữ của đối tượng

NSLog(@"count: %d", [overlay retainCount]); 

Answer 4

Nếu bạn gán các đối tượng trực tiếp đến tài sản, bạn vẫn phải phát hành nó:

self.overlay = [[[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)] autorelease]; 

Answer 5

Như tài sản của bạn được định nghĩa với (giữ lại) bất kỳ trường hợp bạn thiết lập bằng cách sử dụng setter tổng hợp (thông qua cú pháp self.overlay) sẽ tự động được gửi một thông điệp duy trì:

// You're alloc'ing and init'ing an object instance, which returns an 
// instance with a retainCount of 1. 
UIView *tempOverlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 

// The overlay property is defined with (retain), so when you assign the new 
// instance to this property, it'll automatically invoke the synthesized setter, 
// which will send it a retain message. Your instance now has a retain count of 2. 
self.overlay = tempOverlay; 

// Send a release message, dropping the retain count to 1. 
[tempOverlay release]; 

Nếu bạn đã làm:

self.overlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 

Lớp phủ của bạn sẽ bị giữ lại, có thể dẫn đến rò rỉ tại một số thời điểm trong đơn đăng ký của bạn.

Answer 6

Cảm ơn tất cả các câu trả lời.Tôi có một số tuyên bố mâu thuẫn nhau vì vậy tôi đã thử nghiệm sau đây trong một UITableViewController:

- (id)initWithStyle:(UITableViewStyle)style { 
    if ((self = [super initWithStyle:style])) { 
     NSLog(@"count: %d", [overlay retainCount]); 
     self.overlay = [[UIView alloc] initWithFrame:CGRectMake(160.0f, 70.0f, 150.0f, 310.0f)]; 
     NSLog(@"count: %d", [overlay retainCount]); 
     [overlay release]; NSLog(@"released once"); 
     NSLog(@"count: %d", [overlay retainCount]);  
     [overlay release]; NSLog(@"released twice"); 
     NSLog(@"count: %d", [overlay retainCount]); 
    } 
    return self; 
} 

tôi đã đầu ra giao diện điều khiển sau:

• Đôi khi nó chạy tốt:

  count: 0 
  count: 2 
  released once 
  count: 1 
  released twice 
  count: 1 
  

• Lần khác nó bị lỗi:

  count: 0 
  count: 2 
  released once 
  count: 1 
  released twice 
  Program received signal: “EXC_BAD_ACCESS”. 
  

Tôi biết phương pháp sử dụng tempOverlay là chính xác. Nó chỉ có vẻ rất cồng kềnh, nhưng tôi thích nó để autorelease bởi vì tôi không hiểu làm thế nào autorelease hoạt động hoặc khi nó được gọi. Có một điều chắc chắn là. Mã ở trên là sai vì tôi không muốn overlay để giữ lại số lượng 2.

Điều kỳ lạ là tôi không thể phát hành hai lần. Ngay cả khi nó không sụp đổ, số lượng giữ lại không bị giảm sút.

Dù sao thì, tôi đoán là tôi sẽ sử dụng số tempOverlay ngay bây giờ.