Little refactoring in YXButtonCell

Classes/YXButtonCell.h

@@ -10,17 +10,21 @@
 #import "YXAbstractCell.h"
 
 @interface YXButtonCell : YXAbstractCell {
-	NSString * _title;
-	id _delegate;
-	SEL _selector;
+@private
+	NSString * title_;
+	id target_;
+	SEL action_;
 }
 
-@property (nonatomic, copy) NSString *title;
-@property (nonatomic, assign) id delegate;
-@property (nonatomic, assign) SEL selector;
+@property (nonatomic, copy, readonly) NSString * title;
+@property (nonatomic, assign, readonly) id target;
+@property (nonatomic, assign, readonly) SEL action;
 
-// selector is a method like - (void)buttonTapped:(YXButtonCell*)cell;
-+ (id)cellWithReuseIdentifier:(NSString*)reuseIdentifier withTitle:(NSString*)title 
-				 withDelegate:(id)delegate selector:(SEL)selector;
+//
+// Target must implement following action selector
+//   - (void)buttonTapped:(YXButtonCell*)cell;
+//
++ (id)cellWithReuseIdentifier:(NSString *)reuseIdentifier title:(NSString *)title 
+					   target:(id)delegate action:(SEL)selector;
 
 @end

Classes/YXButtonCell.m

@@ -8,32 +8,31 @@
 
 #import "YXButtonCell.h"
 
+@interface YXButtonCell ()
 
-@implementation YXButtonCell
+@property (nonatomic, copy, readwrite) NSString * title;
+@property (nonatomic, assign, readwrite) id target;
+@property (nonatomic, assign, readwrite) SEL action;
 
-@synthesize title = _title;
-@synthesize delegate = _delegate;
-@synthesize selector = _selector;
+@end
 
-- (void)dealloc {
-	self.delegate = nil;
-	_selector = NULL;
 
-	self.title = nil;
+@implementation YXButtonCell
 
-	[super dealloc];
-}
 
-+ (id)cellWithReuseIdentifier:(NSString*)reuseIdentifier withTitle:(NSString*)title 
-				 withDelegate:(id)delegate selector:(SEL)selector {
++ (id)cellWithReuseIdentifier:(NSString *)reuseIdentifier title:(NSString *)title 
+					   target:(id)target action:(SEL)action
+{
 	YXButtonCell * cell = [[YXButtonCell alloc] initWithReuseIdentifier:reuseIdentifier];
-	cell.delegate = delegate;
-	cell.selector = selector;
+
+	cell.target = target;
+	cell.action = action;
 	cell.title = title;
+
 	return cell;
 }
 
-- (UITableViewCell*)tableViewCellWithReusableCell:(UITableViewCell*)reusableCell {
+- (UITableViewCell*)tableViewCellWithReusableCell:(UITableViewCell *)reusableCell {
 	UITableViewCell * cell = reusableCell;
 	if (cell == nil) {
 		cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:self.reuseIdentifier] autorelease];
@@ -50,9 +49,27 @@ - (UITableViewCell*)tableViewCellWithReusableCell:(UITableViewCell*)reusableCell
 - (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
 	[tableView deselectRowAtIndexPath:indexPath animated:YES];
 
-	if (_delegate != nil && _selector != NULL) {
-		[_delegate performSelector:_selector withObject:self];
+	if (self.target != nil && self.action != NULL) {
+		[self.target performSelector:self.action withObject:self];
 	}
 }
 
+
+#pragma mark -
+#pragma mark Memory management
+
+
+@synthesize title = title_;
+@synthesize target = target_;
+@synthesize action = action_;
+
+
+- (void)dealloc {
+	target_ = nil;
+	action_ = NULL;
+	[title_ release];
+
+	[super dealloc];
+}
+
 @end

Classes/YXModelTableViewsViewController.m

@@ -58,9 +58,9 @@ - (void)viewDidLoad {
 
 	YXSection * sectionC = [[YXSection alloc] initWithHeader:@"Section C before" footer:@"Section C after"];
 	[sectionC addCell:[YXButtonCell cellWithReuseIdentifier:@"buttonCell" 
-												  withTitle:@"Button"
-											   withDelegate:self 
-												   selector:@selector(buttonCellTapped:)]];
+												  title:@"Button"
+											   target:self 
+												   action:@selector(buttonCellTapped:)]];
 
 	[self setSections:[NSArray arrayWithObjects:sectionA, sectionB, sectionC, nil]];