有没有理由我不应该只定义#define NSFM [NSFileManager defaultManager]而不是每次都写出来?

zek*_*kel 4 cocoa coding-style objective-c

有一个内置的快捷方式NSApp,有什么理由我不应该为NSFileManager添加一个?

#define NSFM [NSFileManager defaultManager]
Run Code Online (Sandbox Code Playgroud)

我认为省略这将使我的代码更清洁,我看不到保持它的任何好处.(我计划从现在开始在我的所有项目中执行此操作,因此它不会模糊不清.)

NSFileManager *fm = [NSFileManager defaultManager] 
Run Code Online (Sandbox Code Playgroud)

Bar*_*ark 5

你为什么不只使用局部变量?

NSFileManager *fm = [NSFileManager defaultManager];
// use fm...
Run Code Online (Sandbox Code Playgroud)

或者更好的是,将文件管理器注入方法参数:

- (void)myMethod {
  //using [NSFileManager defaultManager]
}
Run Code Online (Sandbox Code Playgroud)

- (void)myMethodWithFileManager:(NSFileManager*)fm {
 //usin fm
}
Run Code Online (Sandbox Code Playgroud)

由于defaultManager是一个单例(实际上是全局的),因此它使测试变得非常困难.注入依赖项可以节省您在方法中键入(如您所愿)并使单元测试更容易 - 您可以注入测试double而不是defaultManager.

最后,Cocoa和Objective-C通常倾向于使用简短代码来支持显式代码.哲学基本上是使用更详细的名称使代码更容易阅读(从而维护).这种理念一直延伸到Objective-C的选择器样式,并带有交错的命名参数.除非您真的无法处理额外的输入(并且Xcode的代码完成对您不起作用),否则请坚持使用现有名称.在罗马的时候......

  • +1单位测试角度是非常重要的一点.我没有提及它感到羞耻. (2认同)