必要がないと書かれているインターネットの情報は多いと思いますが実際は observer (この場合、ViewController) の deinit() で unregister するほうが良いです。

これ、Swift Concurrencyの世界だとdeinitでは @MainActor なクラスであるUIViewController自体(self)を触れないので今後できなくなる気がします。 SE-0371が入ればdeinitでMainActorなクラスを触れるようになるっぽい？ https://github.com/apple/swift-evolution/blob/main/proposals/0371-isolated-synchronous-deinit.md 今のところはViewに関係ないNotificationはViewControllerではないクラスで受け取るのがいいのかも (MainActorでなければdeinitでselfに触れるので)

swift-evolution/proposals/0371-isolated-synchronous-deinit.md at ma...

This maintains proposals for changes and user-visible enhancements to the Swift Programming Language. - apple/swift-evolution