Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Swift random number [duplicate]

I'm having problems with this drawRandomCard function.

It works just like it should for some time, but eventually it crashes the application.

Here is the code:

import Foundation


var cardDeck = Array<PlayingCard>()

class Deck {

    func addCard(card : PlayingCard , atTop : Bool = false){

        if atTop {
            cardDeck.insert(card, atIndex: 0);
        }else{
            cardDeck += card
        }
    }

    func drawRandomCard() -> PlayingCard{
        var card = PlayingCard()
        var randomNumber : Int = Int(arc4random()) % (cardDeck.count - 1)
        card = cardDeck[randomNumber]
        cardDeck.removeAtIndex(randomNumber)
        return card
    }

}
like image 986
vyudi Avatar asked Jun 09 '14 11:06

vyudi


1 Answers

Use arc4random_uniform to avoid modulo bias. Like following:

let randomNumber = arc4random_uniform(150)

For your example, it will be:

let randomNumber = Int(arc4random_uniform(UInt32(cardDeck.count)))
like image 108
Adam Avatar answered Sep 21 '22 09:09

Adam