How to fix Initializer for conditional binding must have Optional type, not 'String'?

Question

How can I solve this problem? Initializer for conditional binding must have Optional type, not 'String'

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! TableViewCell

    cell.lblNamnEfternamn.text = personsArray[indexPath.row].wholeName
    cell.lblEmail.text = personsArray[indexPath.row].email
    cell.imageAvatar.image = UIImage(named: "avatar")

    if let newProfileImageUrl = personsArray[indexPath.row].profileImageUrl {
        let url = NSURL(string: newProfileImageUrl)
        let request = URLRequest(url: url! as URL)
        let task = URLSession.shared.dataTask(with: request as URLRequest) { (data, response, error) in
            if error != nil {
                print(error!)
                return
            }

            DispatchQueue.main.async {
                cell.imageAvatar.image = UIImage(data: data!)
                tableView.reloadData()
            }

        }
    }
    return cell
}

in your current approach , you may also need `task.resume()` — Shehata Gamal, Mar 29 '19 at 14:35
you can only use if let to unwrap an optional. The error says profileImageUrl is not an optional. So you don't have to use if let. — Teja Nandamuri, Mar 29 '19 at 14:48
You could declare profileImageUrl to be String? instead of String. — Yaming Lin, Mar 29 '19 at 14:59

Shehata Gamal · Accepted Answer · 2019-03-29T14:33:03.587

3

Replace this

if let newProfileImageUrl = personsArray[indexPath.row].profileImageUrl {

with

let newProfileImageUrl = personsArray[indexPath.row].profileImageUrl

Also it's better to use SDWebImage

cell.imageAvatar.sd_setImage(with: URL(string:newProfileImageUrl), placeholderImage: UIImage(named: "placeholder.png"))

instead of repeatedly fetching the image every scroll

edited Mar 29 '19 at 14:33

answered Mar 29 '19 at 14:29

Shehata Gamal

98,760
8
65
87

doesn't help more errors show up – Genti Malm Mar 29 '19 at 14:31
1

remove the closing `}` of the `if let` – Shehata Gamal Mar 29 '19 at 14:31

How to fix Initializer for conditional binding must have Optional type, not 'String'?

1 Answers1

Linked