Swift 2 + Parse: Array index out of range

Question

SOMETIMES THE REFRESH WORKS SOMETIMES IT DOESN'T

I have a UITableViewController which is basically a news feed. I have also implemented a pull to refresh feature. However sometimes when I pull to refresh it gives me the error

'Array index out of range'.

I know this means an item it is trying to get does not exist but can you tell me why? Here is my code:

override func viewDidLoad() {
    super.viewDidLoad()

    refresher = UIRefreshControl()
    refresher.attributedTitle = NSAttributedString(string: "Pull to refresh")
    refresher.addTarget(self, action: "refresh", forControlEvents: UIControlEvents.ValueChanged)
    self.tableView.addSubview(refresher)
    refresh()

    tableView.delegate = self
    tableView.dataSource = self
}

and the refresh() function:

func refresh() {

    //disable app while it does stuff
    UIApplication.sharedApplication().beginIgnoringInteractionEvents()

    //get username and match with userId
    let getUser = PFUser.query()
    getUser?.findObjectsInBackgroundWithBlock({ (objects, error) -> Void in

        if let users = objects {

            //clean arrays and dictionaries so we dont get indexing error???
            self.messages.removeAll(keepCapacity: true)
            self.users.removeAll(keepCapacity: true)
            self.usernames.removeAll(keepCapacity: true)

            for object in users {

                if let user = object as? PFUser {

                    //make userId = username
                    self.users[user.objectId!] = user.username!
                }
            }
        }
    })

    let getPost = PFQuery(className: "Posts")
    getPost.findObjectsInBackgroundWithBlock { (objects, error) -> Void in

        if error == nil {
        if let objects = objects {

            self.messages.removeAll(keepCapacity: true)
            self.usernames.removeAll(keepCapacity: true)

            for object in objects {

                self.messages.append(object["message"] as! String)
                self.usernames.append(self.users[object["userId"] as! String]!)

                self.tableView.reloadData()
            }
        }
        }
    }

    self.refresher.endRefreshing()
    UIApplication.sharedApplication().endIgnoringInteractionEvents()
}

and:

func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
    let myCell = tableView.dequeueReusableCellWithIdentifier("SinglePostCell", forIndexPath: indexPath) as! PostCell


    //ERROR GETS REPORTED ON THE LINE BELOW
    myCell.usernamePosted.text = usernames[indexPath.row]
    myCell.messagePosted.text = messages[indexPath.row]

    return myCell
}

Michael · Accepted Answer · 2016-01-19 01:34:36Z

You have a race condition given you are doing two background tasks, where the second depends on values returned from the first. getUser?.findObjectsInBackgroundWithBlockwill return immediately, and getPost.findObjectsInBackgroundWithBlock will start executing. The getPost should be inside the block for getUser, to ensure the sequence is correct. Similarly, the following two lines should be inside the second block:

self.refresher.endRefreshing()
UIApplication.sharedApplication().endIgnoringInteractionEvents()

Given the error line, you probably also have a race condition between the two background tasks and displaying the tableView. I would be inclined to try:

func tableView(tableView:UITableView!, numberOfRowsInSection section:Int) {
   return self.refresher.refreshing ? 0 : self.usernames.count
}

This way you won't touch self.usernames until the background refresh is finished (as long as you remember to put endRefreshing inside the second block, which is also put inside the first block).

Rana · Accepted Answer · 2016-01-15 19:21:43Z

0

I Believe that in self.users[user.objectId!] = user.username! the user.ObjectId is some random value assigned by parse which looks like this: "34xcf4". This is why you might be getting 'Array index out of range'.

answered Jan 15, 2016 at 19:21

Rana

1,7754 gold badges26 silver badges55 bronze badges

Comments

Marco · Accepted Answer · 2016-01-18 13:31:27Z

0

There are two required methods for configuring a UITableView:

tableView(_:cellForRowAtIndexPath:) and
tableView(_:numberOfRowsInSection:)

In your code you are presenting only one required method, if you don't implement the second method then it that may cause errors.

Check the documentation at: https://developer.apple.com/library/ios/documentation/UIKit/Reference/UITableViewDataSource_Protocol/#//apple_ref/occ/intfm/UITableViewDataSource/tableView:cellForRowAtIndexPath:

answered Jan 18, 2016 at 13:31

Marco

3244 silver badges9 bronze badges

Comments

Alain T. · Accepted Answer · 2016-01-19 02:38:13Z

0

You are calling self.tableView.reloadData() on every addition to your array and doing so in a background thread.

As a general rule, you should not do UI updates in a background thread. When you clear self.messages and self.usernames, because you are in background thread, nothing prevents the tableview from trying to get a cell at an index that no longer has any data in the array.

If you want to keep your code in the background thread (risky as it may be), you should at least call .beginUpdates before reloading your arrays and wait until they're all done before calling reload and endUpdates.

answered Jan 19, 2016 at 2:38

Alain T.

42.2k4 gold badges36 silver badges57 bronze badges

Collectives™ on Stack Overflow

Swift 2 + Parse: Array index out of range

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related