FRDModuleManager icon indicating copy to clipboard operation
FRDModuleManager copied to clipboard

带有返回参数的方法写法有问题

Open kiss7xin opened this issue 5 years ago • 1 comments

- (BOOL)application:(UIApplication *)app openURL:(NSURL *)url options:(NSDictionary<UIApplicationOpenURLOptionsKey,id> *)options { BOOL result = NO; for (id<SHModule> module in self.modules) { if ([module respondsToSelector:_cmd]) { result = result || [module application:app openURL:url options:options]; } } return result; }

这种带有BOOL返回值的,不能直接使用 result = result || [module application:app openURL:url options:options]; 这会导致如果一个模块实现了这个方法返回YES,那么他后面的模块都无法再调用这个方法。 应改为: BOOL moduleResult = [module application:app openURL:url options:options]; result = result || moduleResult;

kiss7xin avatar May 27 '20 09:05 kiss7xin

@kiss7xin 多谢,你可以发一个 pr 修正这个问题?我可以 merge。

lincode avatar May 28 '20 09:05 lincode