这种方法将来会导致内存泄漏吗?

现在不行。
您的内存图将如下所示:

所以这里没有内存循环,也没有泄漏内存的原因。

重要的 是的。如果在viewcontroller上添加来自datasource/delegate的引用,请确保它是弱引用,否则将创建内存循环。

注意 是的。您可以在collectionview上从datasource/delegate添加强引用,因为collectionview在datasource和delegate上有弱引用。所以也没有周期

旁注

最好在数据源中注册单元格,因为“只有”数据源知道将使用什么类型的单元格。

你的问题相当含糊,但总的来说,这是一种很常见的做法。我们公司经常采用这种模式:

class MainCollectionViewController: UIViewController {
    lazy var dataSource: UICollectionViewDataSource = self
    lazy var delegate: UICollectionViewDelegate = self

    static func with(dataSource: UICollectionViewDataSource, delegate: UICollectionViewDelegate) -> MainCollectionViewController {
        let vc = MainCollectionViewController()
        vc.dataSource = dataSource
        vc.delegate = delegate

        return vc
    }
}

extension MainCollectionViewController: UICollectionViewDataSource {
    // code
}

extension MainCollectionViewController: UICollectionViewDelegate {
    // code
}

两个主要用途是单元测试和将数据传递给视图控制器。测试人员可以在测试时插入自定义数据源并委派:

let testVC = MainCollectionViewController.with(dataSource: ..., delegate: ...)
// do test

或向其传递数据:

// In another view controller
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    if let destinationVC = segue.destination as? MainCollectionViewController {
        destinationVC.dataSource = ...
        destinationVC.delegate = ...
    }
}

至于内存泄漏,这种模式通常是安全的,但很明显偶尔会有人遇到内存问题。你的里程可能不同。