一个名称建议

[zhanfengkuang]

public init(photoLoader: JXPhotoLoader,
                numberOfItems: @escaping () -> Int,
                localImage: @escaping (Int) -> UIImage?,
                autoloadURLString: @escaping (Int) -> String?) {
        self.photoLoader = photoLoader
        self.numberOfItemsCallback = numberOfItems
        self.localImageCallback = localImage
        self.autoloadURLStringCallback = autoloadURLString
    }

localImage 改为placeholder 是不是更合适一些

[JiongXing]

是的，后面优化

[zhanfengkuang]

/// 打开浏览器
    open func show(pageIndex: Int) {
        self.pageIndex = pageIndex
         UIViewController.jx.topMost?.present(self, animated: true, completion: nil)
    }

如果index 越界, 没有做处理 @JiongXing

[JiongXing]

调用方最清楚数据源，越界由调用方判断比较好。Browser内部掩盖越界错误不是很恰当

[zhanfengkuang]

其实这样做也很对, ,用户可以在此基础封装, 对其异常的统一处理, 说一点我自己的看法 直接交给用户自己做处理 感觉和OC通过下标取元素的思想是一致的, 但我想swift本身是一门安全语言, 它的一些可选机制, 类型转换都是为了减少出错的几率(将语言和三方库相比较一点牵强), 不过可以从json解析的库考虑一下, 一些json解析库是很简便了我们的代码, 我觉得减少解析出错的几率也是其很重要的一点. 一点看法, 不一定对😄

[JiongXing]

感谢提出建议，2.1.0已发布，对此作了保护。