Crash error in Xcode8 swift3

[skywalkerlw]

it crashes if you directly use a local variable "let dropDown = DropDown()", rather than assigning dropDown as a property. However it works in Xcode7.3.

[leizard]

Seems like I have the same issue. Could you please tell more about "assigning dropDown as a property". I don't quite get that.

[leizard]

This is the error that I had. Is it the same to yours ?

[skywalkerlw]

yes, exactly the same

[skywalkerlw]

by the way, if you have codes in extension via objc_setAssociatedObject, you will not be able to dropdown list ,but no crash happens

[leizard]

I finally make it work. Instead of use let dropDown = DropDown() in my button's IBAction scope. I create local variable let dropDown = DropDown() in my View Controller. then in my button IBAction function, I just init the data source, anchor view and finally show it. Now it works like a charm :D

[kevin-hirsch]

Hello @leizard and @wangbourne,

this is not new to Xcode 8 and Swift 3, every DropDown instance has no direct superview and is not retained by its anchorView (note that anchorView can be nil, for the DropDown to be centered on the screen). Thus, it has to be retained by something else, usually the class instance.

[yesh0907]

@leizard I tried your solution however it didn't work. I keep getting the same error. Here is my code

let dropDown = DropDown()
@IBOutlet weak var breadButton: UIButton!
let breads = ["Multigrain", "Honey Oat", "Italian", "Hearty Italian", "Paramesan Oregano", "Wrap", "Flatbread"]

@IBAction func showBread(_ sender:AnyObject) {
    dropDown.anchorView = view

    dropDown.dataSource = breads

    dropDown.show()
}

override func viewDidLoad() {
    super.viewDidLoad()

    dropDown.bottomOffset = CGPoint(x: 0, y: breadButton.bounds.height)
    dropDown.selectionAction = { [weak self] (index: Int, item: String) in
        self?.breadButton.setTitle(item, for: .normal)
    }

    dropDown.backgroundColor = UIColor(red: 32/255, green: 152/255, blue: 93/255, alpha: 1.0)
    dropDown.textColor = UIColor.white

    dropDown.selectionBackgroundColor = UIColor(red: 24/255, green: 138/255, blue: 82/255, alpha: 1.0)
}

Can anyone help? Ty.

[Luccifer]

Bump, same problem

UPD:

Tried to change 484L in DropDown.swift to
DispatchQueue.main.async { void in
self.tableView.flashScrollIndicators()
}

Looks like it works for now

[AdamLenda]

Same issue

[AdamLenda]

I've been coding for 20 years and I just spent 4 hours struggling with your code. Either your documentation or your code sucks.

[ChandraNaveen]

make your dropdown instance global.It works.

[Luccifer]

you mean public static?

[ChandraNaveen]

just make it global to your class.

[jigs611989]

Make it optional
Example:
In ViewController File
let dropDown: DropDown?

then in viewDidLoad function

override func viewDidLoad() {
  super.viewDidLoad()
  dropDown = DropDown()
}

[NonAtomicFunk]

Hello, @kevin-hirsch

I'm having the very same problem while flipping TableViewControllers using following method.
None of solutions listed above helps me out.
Please advise.

public lazy var orderedViewControllers: [UIViewController] = {
return [self.controller(cls: FirstPageTableViewControllerStatic.self),
self.controller(cls: SecondPageTableViewController.self)]
}() as! [UITableViewController]

[HemantRMali]

Make it optional, this will work.
let dropDown: DropDown?

[kevin-hirsch]

Hello everyone!
Sorry for the late response.
I've pushed a new commit that should fix this problem (80a125b on master)