一次方法适配实践

前言


回顾笔者的runtime系列文章,发现实践略少,恰好近来一位朋友入职新公司后进行codereview时遇到了一个问题,和他讨论后制定了一个使用runtime的方案来解决问题,正好记录下这个方案。

问题

在朋友的项目中存在一个异步获取沙盒文件的接口,伪实现如下:

#define BLOCK_SAFE_CALLS(_b_, _f_, _e_) if (_b_) { _b_(_f_, _e_); }

- (void)asyncFetchAllFoldersWithCompleteBlock: (void(^)(NSArray *, NSError *))complete {
    BEGIN_OPERATION_DISPATCHER
    XXXFetchFlodersOperation * operation = [self.XXXSession fetchAllFoldersOperation];
    [operation start: ^(NSError * error, NSArray * folders) {
        BLOCK_SAFE_CALLS(completeBlock, folders, error);
    }];
    END_OPERATION_DISPATCHER
}

由于未知原因,在运行期间,这个方法总在前至多3次调用时出现error,为了避免调用该方法时还需要在回调中实现重新尝试的代码,需要把重试代码的业务放到这个方法中。

方案1:不修改原接口的基础上添加递归调用

#define BLOCK_SAFE_CALLS(_b_, _f_, _e_) if (_b_) { _b_(_f_, _e_); }

- (void)asyncFetchAllFoldersWithCompleteBlock: (void(^)(NSArray *, NSError *))completeBlock {
    [self asyncFetchAllFoldersWithCompleteBlock: completeBlock retryTime: 3];
}

- (void)asyncFetchAllFoldersWithCompleteBlock: (void (^)(NSArray *, NSError *))completeBlock retryTime: (int)retryTime {
    BEGIN_OPERATION_DISPATCHER
    XXXFetchFlodersOperation * operation = [self.XXXSession fetchAllFoldersOperation];
    [operation start: ^(NSError * error, NSArray * folders) {
        if (error && retryTime > 0) {
            [self asyncFetchAllFoldersWithCompleteBlock: completeBlock retryTime: retryTime - 1];
        } else {
            BLOCK_SAFE_CALLS(completeBlock, folders, error);
        }
    }];
    END_OPERATION_DISPATCHER
}

借鉴于递归思想,提供一个额外的接口传入一个标记(代码中为retryTime)以此作为是否在调用发生错误后重新尝试。且上面的方案对现有代码的改动是最小的,几乎无侵害。(然而朋友说不允许修改原接口实现,因此方案作废)

方案2:提供额外的接口来完成操作

@interface XXXXX: NSObject

- (void)asyncFetchAllFoldersWithCompleteBlock: (void(^)(NSArray *, NSError *))completeBlock NS_DEPRECATED_IOS(2_0, 5_0);
- (void)asyncFetchAllFoldersWithCompleteBlock: (void (^)(NSArray *, NSError *))completeBlock retryTime: (int)retryTime;

@end

@implementation XXXXX

- (void)asyncFetchAllFoldersWithCompleteBlock: (void (^)(NSArray *, NSError *))completeBlock retryTime: (int)retryTime {
    NSParameterAssert(completeBlock);
    [self asyncFetchAllFoldersWithCompleteBlock: ^(NSArray * folders, NSError * error) {
        if (error && retryTime > 0) {
            NSLog(@"failed error: %@", error);
            [self asyncFetchAllFoldersWithCompleteBlock: completeBlock retryTime: retryTime - 1];
        } else {
            completeBlock(folders, error);
        }
    }];
}

@end

此方案通过宏定义NS_DEPRECATED_IOS标记原接口为摒弃方法,但是这样一来所有调用原接口的代码都要重新进行修改:


不谈工作量,朋友说他只有修改当前类实现文件的权力,其他外界代码不允许修改。因此,方案作废

方案3:method_swizzling

由于原接口代码以及接口调用不允许改动,留给我们选择的余地就不多了,恰好还有AOP的方式可以来解决这个问题。当然相比起其他两个方案代码数量要多得多,通过交换方法实现的方式将方法的调用实际上转到我们新增的接口中:

+ (void)load {
    aop_method_exchange([self class], @selector(AOPAsyncFetchAllFoldersWithCompleteBlock:), @selector(asyncFetchAllFoldersWithCompleteBlock:));
}

- (void)AOPAsyncFetchAllFoldersWithCompleteBlock: (void (^)(NSArray *, NSError *))completeBlock {
    NSParameterAssert(completeBlock);
    [self asyncFetchAllFoldersWithCompleteBlock: ^(NSArray * folders, NSError * error) {
        completeBlock(folders, error);
    } retryTime: 3];
}

- (void)asyncFetchAllFoldersWithCompleteBlock: (void (^)(NSArray *, NSError *))completeBlock retryTime: (int)retryTime {
    NSParameterAssert(completeBlock);
    [self AOPAsyncFetchAllFoldersWithCompleteBlock: ^(NSArray * folders, NSError * error) {
        if (error && retryTime > 0) {
            [self asyncFetchAllFoldersWithCompleteBlock: completeBlock retryTime: retryTime - 1];
        } else {
            completeBlock(folders, error);
        }
    }];
}

实际上方案3是结合了方案1与方案2的优点以及避开了两者的缺点,即使删除新增的代码,原有代码不会受到任何影响。缺点在于如果方法本身已经被hook过了,那么可能会出现意料之外的错误

尾言

离上次写博客过去也有一个多月了,期间经历了忙碌的春节,以及项目赶工,都没什么时间静下来写博客。最近笔者还报了自考本科,目标是当一个会画画的码农,从此就失去了周末的双休了。哎,心疼一下自己。最后放上新手的画画作业,高能预警!!!


最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 159,219评论 4 362
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 67,363评论 1 293
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 108,933评论 0 243
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 44,020评论 0 206
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 52,400评论 3 287
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,640评论 1 219
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,896评论 2 313
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,597评论 0 199
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,327评论 1 244
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,581评论 2 246
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 32,072评论 1 261
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,399评论 2 253
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 33,054评论 3 236
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,083评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,849评论 0 195
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,672评论 2 274
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,585评论 2 270

推荐阅读更多精彩内容