passing tapped cell data to another view via segue in Swift - ios

Question

I have a tableView and I am trying to send the data related to tapped cell to another view via segue.
My code works only after second tap, and always returning nil at first tap. I suppose for some reason prepareForSegue gets executed first, that's why detailToSend is always nil at the first tap, and with the second tap I get the data from the previous tapped cell. How can I fix this ?
Updated:
var detailToSend = SingleRepository()
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
if filteredResult.count > 0 {
detailToSend = filteredResult[indexPath.row]
} else {
detailToSend = finalArrayUnwrapped[indexPath.row]
}
performSegue(withIdentifier: "showDetailSegue", sender: nil)
}
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
if segue.identifier == "showDetailSegue" {
let vc = segue.destination as! DetailViewController
vc.detail = detailToSend
}
}
My DetailViewController
class DetailViewController: UIViewController {
#IBOutlet weak var detailLabel: UILabel!
var detail: SingleRepository?
override func viewDidLoad() {
super.viewDidLoad()
if let detailUnwrapped = detail {
print(detailUnwrapped)
}
}
override func didReceiveMemoryWarning() {
super.didReceiveMemoryWarning()
// Dispose of any resources that can be recreated.
}
}

Answer 1

Make sure that you have created your segue from the view controller object in the storyboard and not the cell; if you create the segue from the cell, then the segue will fire before didSelectRowAt is called.
Once you have set the segue correctly, you can use the sender parameter to avoid the use of the detailToSend property:
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
var detailToSend: SingleRepository
if filteredResult.count > 0 {
detailToSend = filteredResult[indexPath.row]
} else {
detailToSend = finalArrayUnwrapped[indexPath.row]
}
performSegue(withIdentifier: "showDetailSegue", sender: detailToSend)
}
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
if let vc = segue.destination as? DetailViewController, let detailToSend = sender as? SingleRepository {
vc.detail = detailToSend
}
}

Answer 2

You could setup a manual segue in the storyboard and trigger it on tap.
So instead of segueing from a cell to a controller, segue from a controller to another controller.
and then your code will be something like this:
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
if filteredResult.count > 0 {
detailToSend = filteredResult[indexPath.row]
} else {
detailToSend = finalArrayUnwrapped[indexPath.row]
}
performSegue(withIdentifier: "showDetailSegue", sender: nil)
}
prepareForSegue should remain exactly the same.
Also check your didSelectRowAt method. You are ignoring everything that happens in scope of
if filteredResult.count > 0 {
detailToSend = filteredResult[indexPath.row]
}
because after that you assign
detailToSend = finalArrayUnwrapped[indexPath.row]