依赖注入与单元测试

不可测的代码

假设我们需要记录一些日志,而日志的容器有一定的容量,如果日志满了就无法记录了,我们可能会写出如下的代码

// MyLogger.m
- (void)writeLog:(NSString *)log {
    MyLoggerService *service = [[MyLoggerService alloc] init];
    if ([service isLogFull]) {
        // Do something else
    }
    else {
        [service writeLog:log];
    }
}

存在的问题

上面这个方法是很难单元测试的,因为:

  1. 这个方法的行为严重依赖service变量的-[MyLoggerService isLogFull]方法的返回,但是我们却不知道什么时候这个方法返回YES,什么时候返回NO;
  2. service变量在方法体内部初始化,我们无法Mock,所以无法控制这个方法的执行路径。
    其实根本原因在于MyLogger对MyLoggerService有一个强依赖。如下图:


    不可测试的实现.png

    强依赖不仅导致了可测性的问题,还导致了扩展性的问题:如果想换一个log的实现方案,需要修改MyLogger的实现,而如果MyLogger已经作为一个组件发布的话,则需要重新发布(开放闭合原则)。

解决方法

对于上面的问题,也许我们已经有了解决方案:将MyLoggerService作为参数传递给MyLogger,这样就可以打破MyLogger对MyLoggerService的依赖(不彻底的打破),类图如下:


依赖注入.png

实现代码

// MyLogger.m
- (void)writeLog:(NSString *)log usingService:(MyLoggerService *)service {
    if ([service isLogFull]) {
        // Do something else
    }
    else {
        [service writeLog:log];
    }
}

测试代码

- (void)testWriteLog {
    id service1 = OCMClassMock([MyLoggerService class]);
    OCMStub([service1 isLogFull]).andReturn(YES);
    
    __block BOOL isCalled = NO;
    OCMStub([service1 writeLog:[OCMArg any]]).andDo(^(NSInvocation *invocation){
        isCalled = YES;
    });
    
    MyLogger *logger = [[MyLogger alloc] init];
    [logger writeLog:@"123" usingService:service1];
    
    XCTAssertFalse(isCalled, @"[MyLoggerService writeLog:] should never be called");
    
    /////////////////////////////////////////////////////////////////////////////////
    
    id service2 = OCMClassMock([MyLoggerService class]);
    OCMStub([service2 isLogFull]).andReturn(NO);
    
    isCalled = NO;
    OCMStub([service2 writeLog:[OCMArg any]]).andDo(^(NSInvocation *invocation){
        isCalled = YES;
    });
    
    [logger writeLog:@"123" usingService:service2];
    
    XCTAssertTrue(isCalled, @"[MyLoggerService writeLog:] should be called");
}

依赖注入

其实上面的这个小重构(没错,这就是重构!)使用了依赖注入的原则,维基百科上将依赖注入定义为:
In software engineering, dependency injection is a technique whereby one object supplies the dependencies of another object. A dependency is an object that can be used (a service). An injection is the passing of a dependency to a dependent object (a client) that would use it. The service is made part of the client's state. Passing the service to the client, rather than allowing a client to build or find the service, is the fundamental requirement of the pattern.
简单的来说就是Client不应该自己构造Service对象,而应该通过外部传入Service对象。

问题依然存在

通过这种方式,虽然可以让MyLogger不那么依赖MyLoggerService,使得MyLogger可以被测试,但是依赖依然存在,因为MyLogger中有一个硬编码的MyLoggerService类。
如果我们需要更换一个log的实现方案,虽然不用修改MyLogger类了,但是却需要修改MyLoggerService类,这样的修改是我们不能容忍的。
我们还需要做一次重构,这次我们使用接口(或者说协议)完全打破MyLogger对MyLoggerService的依赖。

使用接口

首先我们定义LoggerService的接口:

@protocol LoggerServiceProtocol

- (BOOL)isLoggerFull;

- (void)writeLog:(NSString *)log;

@end

然后我们修改MyLogger的实现:

- (void)writeLog:(NSString *)log usingService:(id<LoggerServiceProtocol>)service {
    if ([service isLoggerFull]) {
        // Do something else
    }
    else {
        [service writeLog:log];
    }
}

类图如下:


接口.png

使用接口实现方式,可以做到极大的灵活性,只要是实现了LoggerServiceProtocol的类,调用者都可以自由的选择用来记录log了。
当然,我们也可以通过实现LoggerServiceProtocol,不依赖Mock和Stub完成MyLogger的单元测试,如下是一个示例:

// InsufficientLoggerService.m
- (BOOL)isLoggerFull {
    self.isCalled = NO;
    return YES;
}

- (void)writeLog:(NSString *)log {
    self.isCalled = YES;
}

// MyLoggerTests.m
- (void)testWriteLog {
    MyLogger *logger = [[MyLogger alloc] init];
    InsufficientLoggerService *insufficient = [[InsufficientLoggerService alloc] init];
    [logger writeLog:@"123" usingService:insufficient];
    
    XCTAssertFalse(insufficient.isCalled, @"writeLog should never be called");
}

总结

  1. 代码中的强依赖很有可能导致不可测的问题,依赖注入可以有效的解决强依赖;
  2. 尽量不使用局部变量,因为局部变量难以mock;
  3. 同样的,静态变量也是不受欢迎的,因为静态变量只能初始化一次,也难以mock,后续状态难以跟踪;
  4. 单例模式尽量少用,因为单例也算一个广义的静态变(另外异步更新单例也容易产生诡异的bug)。
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 214,377评论 6 496
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,390评论 3 389
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,967评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,344评论 1 288
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,441评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,492评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,497评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,274评论 0 269
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,732评论 1 307
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,008评论 2 328
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,184评论 1 342
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,837评论 4 337
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,520评论 3 322
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,156评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,407评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,056评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,074评论 2 352

推荐阅读更多精彩内容