iOS代码重构的一些方法

最近项目已到尾声了。对于这个只是换汤不换药的新项目来说，开始接手的时候是有些抵触情绪的。经过熟悉了几个月，出于对职业的责任心，开始一直在公司的iOS讨论组里面分享知识，鼓励大家一起去改变这一切。希望能把共同认为是好的方式带进来。下面是最近进行代码重构的一些心得。

删掉死代码

诸如以下这些:

- (id)initWithNibName:(NSString *)nibNameOrNil bundle:(NSBundle *)nibBundleOrNil
{
    if ((self = [super initWithNibName:nibNameOrNil bundle:nibBundleOrNil])) {
        // Custom initialization
    }
    return self;
}

- (void)didReceiveMemoryWarning {
    [super didReceiveMemoryWarning];
    // Dispose of any resources that can be recreated.
}

使用Category

我们平时写UITableView是这样的:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
    static NSString *cellIdentifier = @"CellIdentifier";
    
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:cellIdentifier];
    
    if(cell == nil) {
        cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifier];
    }
    //do something here
    return cell;
}

重构后:

.h文件

@interface UITableView (dequeue)
- (UITableViewCell *)dequeueCell;
@end

.m文件

@implementation UITableView (dequeue)

- (UITableViewCell *)dequeueCell
{
    static NSString *cellIdentifier = @"CellIdentifier";
    
    UITableViewCell *cell = [self dequeueReusableCellWithIdentifier:cellIdentifier];
    if(cell == nil) {
        cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifier];
    }
    return cell;
}
@end

//使用
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
    UITableViewCell *cell = [tableView dequeueCell];
    //do something here
    return cell;
}

使用NSDictionary映射字符串等

重构前:

    switch (expression) {
        case constant1:
            return @"result1";
        case constant2:
            return @"result2";
        ...
        case constantN:
            return @"resultN";
        default:
            return @"default";
    }

重构后:

    static NSDictionary *resultMap;
    if (!resultMap) {
        resultMap = @{@(constant1) : @"result1",
                      @(constant2) : @"result2",
                      ...
                      @(constantN) : @"resultN"};
    }
    
    return resultMap[expression] ?: @"default";

是不是逻辑上变得更清晰了呢?

Objective-C 动态方法

`NSSelectorFromString` 与 `[NSObject performSelector:]`

一般我们是这样写

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
    if (indexPath.row == 0) {
        [self func0];
    } else if (indexPath.row == 1) {
        [self func1];
    } else if (indexPath.row == N) {
        [self funcN];
    } else {
        [self doSomething];
    }
}

重构后

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
    SEL funcSEL = NSSelectorFromString([NSString stringWithFormat:@"func%d", indexPath.row]);
    if ([self respondsToSelector:funcSEL]) {
        [self performSelector:funcSEL];
    } else {
        [self dosomething];
    }
}

加上 前一点 用NSDictionary 来映射字符串，是不是又多了一分想象力呢？

PS:基于以上两点，如果UITableView展示一些客户端固有的属性，可以将数据与点击的响应函数组织成一个字典，放在dataSource里面。

delegate 与 block

接口使用delegate好还是block好呢?个人选择如下：

对于那些block的生命周期里不会出现循环引用的优先选择block。否则选择delegate。

比如,`[NSArray enumerateObjectsUsingBlock:]`和UITableView的delegate两者。`[NSArray enumerateObjectsUsingBlock:]`中，即使有循环引用存在，也将在block在使用完毕后释放。而UITableView中，如果用block，出现循环引用需要自己打破。

基于以上选择规则，UIAlertView跟UISheetView中使用block也是可以的。重构之。

其它一些规则：

杜绝魔数。

使用NS_ENUM来枚举有规则的数。

尽量把变量的声明、定义放在最接近使用的地方。

使用最新的OC语法。

尽量精简方法。