Swiftでhit&blowのアプリを作ろうと思っているのですが...

Swift
1/*４桁のランダムな値を生成*/
2func RandomNumber()->UInt32{
3let min = 1000
4let range = 8999
5let num = arc4random_uniform(UInt32(range)) + UInt32(min)
6return num
7}
8
9
10//４桁の数字を分解
11func breakNumber()->Bool{
12    let n4 = RandomNumber() / 1000
13    var a = RandomNumber() % 1000
14    let n3 = a / 100
15    a = a % 100
16    let n2 = a / 10
17    a = a % 10
18    let n1 = a
19    
20    if n4 == n3 {return false} else if n4 == n2 {return false} else if n4 == n1 {return false} else if n3 == n2 {return false} else if n3 == n1 {return false} else if n2 == n1 {return fal`se}
21    else {return true}
22}

最後のところでfalseならもう一度RandomNumber()からやり直しで、trueならこれを抜けるというようなやり方はございませんか？
それか、もし他にいいやり方がありましたら教えていただきたいです。

行動規範の内容に同意します

回答1件

ベストアンサー

hit&blowの出題値を生成するために、breakNumberという関数を作って同じ数字を含まない4桁の数字を生成したいが、このコーディングをどう修正するとうまく処理できるかという質問なのですよね?

まず、4桁の数字というのが1000-9999の範囲でいいんでしょうか?
0123とかを出題値とする必要はないんですか?

次に、4桁の数字といっても別に足し算/引き算するわけではありませんから、UInt32で扱う必要はありません。
今回扱う4桁の数字は、桁ごとに一致しているかしていないかを調べるのが主な処理になると思いますので、UInt32で扱うのではなく、0-9の数字4個で構成される配列として定義した方が後でいろいろ処理しやすいと思います。

そして、同じ数字を持たない4個の数字を生成する方法ですが、以下のように処理するのが、無駄な重複リトライをしなくてすむのでスマートだと思います。
0. まず、0-9までの数字が順番に入っている10個の要素を持つ配列(numbers)を作成します。
0. 0-9の範囲のランダム値(r)を取得します。
0. numbersのr番目の要素を1個目の数字とし、その要素をnumbersから削除します。(残る要素数は9個です)
0. 0-8の範囲のランダム値(r)を取得します。
0. numbersのr番目の要素を2個目の数字とし、その要素をnumbersから削除します。(残る要素数は8個です)
0. 0-7の範囲のランダム値(r)を取得します。
0. numbersのr番目の要素を3個目の数字とし、その要素をnumbersから削除します。(残る要素数は7個です)
0. 0-6の範囲のランダム値(r)を取得します。
0. numbersのr番目の要素を4個目の数字とします。
数字を4個取り出せばいいだけなので、わかりやすいよう1個ずつ順番に全部書きましたが、やりたいことがわかれば4回ループで処理すれば簡潔に記述できます。