Question

我的代码存在这个问题，我认为问题出现在我将语法更改为新swift版本之后。

import UIKit

class FirstTableViewController: UITableViewController {

    var FirstTableArray = [String]()
    var passThisArray = [String]()

    override func viewDidLoad() {
        super.viewDidLoad()
        // This array will display on your tableviewcell.
        FirstTableArray = [lib1]

        //You can pass element of this array
        passThisArray = ["1. Fi "]

    }

    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return FirstTableArray.count
    }

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {


        let Cell = self.tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath) 

        Cell.textLabel?.text = passThisArray[(indexPath as NSIndexPath).row]

        return Cell

    }

    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if(segue.identifier == "detailView") {
            let vc = segue.destination as! ViewController

            //Get the Index of selected Cell
            let indexPath : IndexPath = self.tableView.indexPathForSelectedRow!

            //assign string to next view controller instance from selected cell.
            vc.FirstString = FirstTableArray[(indexPath as NSIndexPath).row]
        }
    }
}

在这一行，它给了我2个错误：

let indexPath : IndexPath = self.tableView.indexPathForSelectedRow!

可选类型的值＆＃39; NSIndexPath？＆＃39;没有打开;你的意思是使用＆＃39;！＆＃39;或者＆＃39;？＆＃39;？
无效使用＆＃39;（）＆＃39;调用非函数类型的值＆＃39; NSIndexPath＆＃39;？

三江源。

Answer 1

问题在于无法保证用户选择了一行，因此您对indexPathForSelectedRow的调用是可选的（可能有也可能没有值）。

改进此方法的一种方法是使用guard语句安全地展开此方法中的两个可选值。如果其中一个未设置（nil），该方法将安全退出而不会使您的应用程序崩溃。

使用守卫而不是if let ...方法的一个好处是，你可以避免厄运的金字塔。使用您的示例，它将需要三个缩进来运行您分配字符串的最终命令，从而使您的代码更难阅读。警卫声明明确表示＆＃34;如果此值失败，请防止崩溃＆＃34;。

override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    if(segue.identifier == "detailView") {
        guard let vc = segue.destination as? ViewController else {
            return
        }

        //Get the Index of selected Cell
        guard let indexPath = self.tableView.indexPathForSelectedRow else {
            return
        }

        //assign string to next view controller instance from selected cell.
        vc.FirstString = FirstTableArray[(indexPath as NSIndexPath).row]
    }
}

另外，两个次要的代码风格的思考：

分配给indexPath时，不需要使用：NSIndexPath。编译器可以为您推断出类型
在声明变量时，iOS约定是使用camelCase，因此虽然indexPath变量很好，但您应该将FirstString更改为firstString

可选类型的值＆＃39; NSIndexPath？＆＃39;没有打开;你的意思是使用＆＃39;！＆＃39;或者＆＃39;？＆＃39;？

1 个答案: