thread 1: signal SIGABRT error when using slide to refresh

Question

I'm trying to add the slide to refresh feature to my app's table. I'm currently following the guide on YouTube found here and my app is crashing with a SIGABRT error when initializing the slide to refresh.

My Code:

 // outlet - table view
@IBOutlet weak var myTableView: UITableView!


// outet - activity indicator
@IBOutlet weak var spinner: UIActivityIndicatorView!


// outlet - barbutton
@IBOutlet weak var menuButton: UIBarButtonItem!

// slide to refresh

let refreshControl: UIRefreshControl = UIRefreshControl()

// xml parser
var myParser: NSXMLParser = NSXMLParser()

// rss records
var rssRecordList : [RssRecord] = [RssRecord]()
var rssRecord : RssRecord?
var isTagFound = [ "item": false , "title":false, "pubDate": false ,"link":false]



// MARK - View functions

override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view.
    if self.revealViewController() != nil {
        menuButton.target = self.revealViewController()
        menuButton.action = "revealToggle:"
        self.view.addGestureRecognizer(self.revealViewController().panGestureRecognizer())
    }


    // pull to refresh
    refreshControl.addTarget(self, action: "uiRefreshControlAction:", forControlEvents:  UIControlEvents.ValueChanged)
    self.myTableView.addSubview(refreshControl);


    func uiRefreshControlAction() {
        self.myTableView.reloadData()
    }

    // set tableview delegate
    self.myTableView.dataSource = self
    self.myTableView.delegate = self
}

If anyone could tell me what I'm doing wrong that would be great. I'm new to Xcode and this is my first app, so if I did something totally wrong sorry for that.

https://www.raywenderlich.com/10209/my-app-crashed-now-what-part-1 — dan, Mar 23 '16 at 03:16

HardikDG · Answer 1 · 2016-03-23T03:32:08.717

In your code you have written

 // pull to refresh
refreshControl.addTarget(self, action: "uiRefreshControlAction:", forControlEvents:  UIControlEvents.ValueChanged)
self.myTableView.addSubview(refreshControl);

Change that to

 // pull to refresh
refreshControl.addTarget(self, action: "uiRefreshControlAction", forControlEvents:  UIControlEvents.ValueChanged)
self.myTableView.addSubview(refreshControl);

as you are not passing any parameters in that function but you have added ':' in the action name which indicate you are going to have parameter in the function

So your full new code will be like this

// MARK - View functions

override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view.
    if self.revealViewController() != nil {
        menuButton.target = self.revealViewController()
        menuButton.action = "revealToggle:"
        self.view.addGestureRecognizer(self.revealViewController().panGestureRecognizer())
    }

    // pull to refresh
    refreshControl.addTarget(self, action: "uiRefreshControlAction", forControlEvents:  UIControlEvents.ValueChanged)
    self.myTableView.addSubview(refreshControl);

    // set tableview delegate
    self.myTableView.dataSource = self
    self.myTableView.delegate = self
}  

func uiRefreshControlAction() {
     self.myTableView.reloadData()
}

Good, but you are missing the fact that he has written his `func` as a local instead of a method. — matt, Mar 23 '16 at 03:22
@matt Thanks for update, As your answer cover all the things he can get proper solution from your answer — HardikDG, Mar 23 '16 at 03:25
I upvoted your answer anyway. You contributed a lot to this! — matt, Mar 23 '16 at 03:49

score 0 · Accepted Answer · answered Mar 23 '16 at 03:19

0

You cannot just go putting func uiRefreshControlAction just anywhere you feel like. Positions of things in your code have a meaning. Learn Swift. Learn how it works.

Meanwhile I'll just tell you the answer on this one. Change this structure:

override func viewDidLoad() {
    super.viewDidLoad()
    // ...
    func uiRefreshControlAction() {
        self.myTableView.reloadData()
    }
    // ...
}

To this:

override func viewDidLoad() {
    super.viewDidLoad()
    // ...
}
func uiRefreshControlAction() {
    self.myTableView.reloadData()
}

Then change this:

action: "uiRefreshControlAction:"

to this:

action: "uiRefreshControlAction"

Finally, update to Xcode 7.3 where this sort of mistake will be impossible for you to make!

answered Mar 23 '16 at 03:19

matt

515,959
87
875
1,141

I should most likely learn this before I continue. Thanks for the input, but changing that didn't work, after changing this my RSS feed didn't load and the slide to refresh was gone. – Gabe Zimbric Mar 23 '16 at 03:25
did you make other things same as your code ? just remove the function from 'viewDidLoad' and place it outside – HardikDG Mar 23 '16 at 03:28
@matt After reading your edit there isn't any more errors. When I'm done tonight I'll update for sure. Only thing now is that the spinner won't stop. Is it because there is no response from the RSS feed or do I need to do something within the swift file for it to stop after refreshing the feed? – Gabe Zimbric Mar 23 '16 at 03:35
@GabeZimbric UIRefreshControl has two methods which you can use to start/stop loading indicator ` - beginRefreshing - endRefreshing`, this may help you – HardikDG Mar 23 '16 at 03:39
@Pyro That fixed it by adding `self.refreshControl.endRefreshing()` to `func uiRefreshControlAction()` Thanks! – Gabe Zimbric Mar 23 '16 at 03:41

thread 1: signal SIGABRT error when using slide to refresh

2 Answers2