確認ですが、あなたのコードは現在このようになっていることを想定しています。
呼び出し側のview controller:
class ViewController: UIViewController {
//...
//どこかのアクションメソッドからPopoverを`present`で表示している...ものと想定
@IBAction func someAction(_ sender: Any) {
//...
let storyboard = UIStoryboard(name: "Main", bundle: nil)
let popover = storyboard.instantiateViewController(withIdentifier: "Popover") as! PopoverViewController
//...
self.present(popover, animated: true, completion: nil)
}
var received: String? = nil
var number: Int!
func doAnything(text: String) {
print(text)
switch text {
case "1":
let activityItem: [Int] = [number] //<- `[number]` ?
let activityVC = UIActivityViewController(activityItems: activityItem, applicationActivities: nil)
self.present(activityVC, animated: true, completion: nil)
//print(url) //<- What is `url`?
case "2":
break
case "3":
break
case "4":
break
case "5":
break
default:
break
}
}
}
Popover用のview controller:
class PopoverViewController: UITableViewController {
//...
var text: [String] = []
override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
//セルの選択解除
tableView.deselectRow(at: indexPath, animated: true)
let back = storyboard!.instantiateViewController(withIdentifier: "View") as? ViewController
back?.doAnything(text: text[indexPath.row])
self.dismiss(animated: true, completion: {() in back?.received = self.text[indexPath.row]})
}
//...
}
あなたが示されたコード、一部コンパイルが通らない部分があったので修正してあります。質問用に少し書き換えられたのかもしれませんが、そのせいで根本的に違う結果になってしまう可能性もあるので注意してください。また、一部推定も含まれています。実際と大きく異なっている点があれば、ご質問に情報を追記するなり、コメントするなりしてお知らせください。
と言うわけで、あなたのコードで致命的に誤っているのはここだと思われます。
let back = storyboard!.instantiateViewController(withIdentifier: "View") as? ViewController
Popoverから戻り先のview controllerにアクセスするのに、新しいインスタンスを作成してしまってはいけません。このコードは(同じViewController
クラスではあるけれど)新たにインスタンスを作成するものであり、作成されたインスタンスは「戻り先」のview controllerとは別物です。
いろいろとやり方は考えられるでしょうが、一番確実なのは戻り先のview controllerを直接プロパティとしてPopover側に持たせてやることでしょう。
Popover側はこんな感じ。
protocol PopoverViewControllerDelegate: class {
var received: String? {get set}
func doAnything(text: String)
}
class PopoverViewController: UITableViewController {
//...
weak var delegate: PopoverViewControllerDelegate?
var text: [String] = []
override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
//セルの選択解除
tableView.deselectRow(at: indexPath, animated: true)
delegate?.doAnything(text: text[indexPath.row])
self.dismiss(animated: true) {
self.delegate?.received = self.text[indexPath.row]
}
}
//...
}
典型的なdelegateパターンということになるので、「戻り先」をdelegate
として、呼び出したい機能(メソッドやプロパティ)をprotocolにしてあります。
呼び出し側はこんな風になるでしょう。
//↓`PopoverViewControllerDelegate`への適合を追加
class ViewController: UIViewController, PopoverViewControllerDelegate {
//...
@IBAction func someAction(_ sender: Any) {
//...
let storyboard = UIStoryboard(name: "Main", bundle: nil)
let popover = storyboard.instantiateViewController(withIdentifier: "Popover") as! PopoverViewController
popover.delegate = self //<- Popoverの呼び出し時にdelegateを設定
//...
self.present(popover, animated: true, completion: nil)
}
var received: String? = nil
var number: Int? //<-未初期化の可能性のある変数にImplicitlyUnwrappedOptionalは使わない
func doAnything(text: String) {
//`number`が非nilであることをguardで確認
guard let number = number else {
print("`number` is nil")
return
}
print(text)
//...
}
}
number
のように呼び出しの順序によっては未初期化状態になる可能性のある変数にImplicitlyUnwrappedOptionalを使うべきではありませんので、普通のOptionalに変え、使う前にguard
で非nilのチェックを入れています。
最初に書いたように一部推定を含んでいるので、全く使い物にならないものをお見せしてしまっている可能性もありますが、うまくあなたのコードに適用できそうでしたら、お試しください。
viewWillAppear
がどんな条件でいつ呼ばれることになるかはいろいろあって一概には言えません。このスレッドなんかも参考になるかと思います。ご自身の質問は「編集」から編集できますのでコードの提示をお願いします。ViewController
」と言うことでよろしいでしょうか?