Why is viewDidLoad() getting called twice?

Question

For some reason my data is being save twice to the model. why is viewdidload running twice? this controller purpose is to be a scoreboard. it is populated by data using core data. Ive tried deleting the tablecontroller and adding a new one.

import UIKit
class ScoreboardTableViewController: UITableViewController {

    var model = scoreboardModel.sharedInstance
    var numbers = [9,12,9]// [Int]()
    var timer = "k"
    var date = "thedateToday"



    override func viewDidLoad() {
        super.viewDidLoad()
        var count = numbers.count
        var x = String(count)


        if(timer != "k"){
            model.saveScoreboard(date, numsMemorized: x,time: timer)
            println("saved!")
        }
        model.getScoreboards()

        }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()

    }

      // MARK: - Table view data source


    override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {

        return model.scoreboard.count
    }


    override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {

        let cell = tableView.dequeueReusableCellWithIdentifier("Cellll", forIndexPath: indexPath) as UITableViewCell

        let score = model.getScore(indexPath)
        cell.textLabel!.text = ("\(score.numsMemorized) | \(score.time)")

        return cell
    }

}

The view that segues to the ScoreboardTableViewController is:

import UIKit

class PracticeReciteController: UIViewController, UITextFieldDelegate {

var theTime = ""

@IBOutlet weak var howManyCorrect: UILabel!

var numbers = [9,12,9]
var x = 0

@IBOutlet weak var textField: UITextField!

var howManyCorrectNum = 0

override func viewDidLoad() {
    super.viewDidLoad()



    var outof = numbers.count
    howManyCorrect.text = "\(howManyCorrectNum) / \(outof)"

    textField.delegate = self;
    textField.addTarget(self, action:"edited", forControlEvents:UIControlEvents.EditingChanged)

}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}



func edited() {

    //[todo] if on last number in array & correct. endgame/collectscore
    var userInput = textField.text.toInt()

    if numbers[x] == userInput{

        if x+1 == numbers.count{


            performSegueWithIdentifier("scoreboardIdentifier", sender: self)

        }
        //println("correct")
        x++

        howManyCorrect.text = String(x) + "/" + String(numbers.count)

            //I set here timer so if user enter correct input then it will remove text after some time
        var timer = NSTimer.scheduledTimerWithTimeInterval(0.8, target: self, selector: Selector("resetText"), userInfo: nil, repeats: false)
    }else if userInput?.isDouble() == numbers[x].isDouble(){
        println("incorrect")
    }
    else{
        println("huh?")

    }
}
//This method will call after some time use if user enter correct input
func resetText(){
    textField.text = ""
}




override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject!) {


    var DestViewController = segue.destinationViewController as ScoreboardTableViewController

    DestViewController.numbers = numbers
    DestViewController.timer = theTime


}

}

It's important for us to know how do you use the vc from outside. You'd better add some code in your question. — kas-kad, Sep 26 '15 at 11:01
What do you mean? its a table controller. each cell shows data from the model. — Topaz t, Sep 26 '15 at 11:09
possible duplicate of [viewDidLoad getting called twice on rootViewController at launch](http://stackoverflow.com/questions/1081131/viewdidload-getting-called-twice-on-rootviewcontroller-at-launch) — Abizern, Sep 26 '15 at 12:28

score 3 · Answer 1 · edited May 23 '17 at 12:24

3

The question was being answered many times before : viewDidLoad getting called twice on rootViewController at launch

viewDidLoad is called twice

Anyway, one important advice for you: don't put any code that will broke your app/data in viewDidLoad. The viewDidLoad not assumed to be called only once in ViewController's lifecycle

edited May 23 '17 at 12:24

Community

1
1

answered Sep 26 '15 at 11:57

kas-kad

3,736
1
26
45

Despite the reason, this is a design(architectural) issue – kas-kad Sep 26 '15 at 12:02

score 0 · Answer 2 · answered Mar 18 '20 at 07:15

This is old issue, but what I found is ..

In Swift, if you access the view of uiviewcontroller like

enter @IBOutlet weak var searchTextField: UITextField! {
    didSet {
        self.searchTextField.borderStyle = .none
        self.searchTextField.keyboardType = .default
        self.searchTextField.placeholder = "Search"
        let width = (self.view.frame.width - 114)
        self.searchTextField.frame = CGRect.init(origin: searchTextField.frame.origin, size: CGSize.init(width: width, height: searchTextField.frame.height))
    }
}

Then it will call viewDidLoad in that stage first, then it will call viewDidLoad as lifecycle.

Why is viewDidLoad() getting called twice?

2 Answers2