配列からランダムに抽出して、別の配列にまとめる方法

Question

let letters = ["a","b","c"]
var result:[String] = []
for n in 0...7 {
    func pickOne() {
        letters.randomElement()
        result.insert(HERE, at: n)
        //ここでrandomizedLettersにlettersから選んで8こ文字を入れたい
    }
}
let randomLetters = "(result[0])(result[1])(result[2])(result[3])(result[4])(result[5])(result[6])(result[7]))"
print(randomLetters)

6行目に大文字でHEREと書いてある所に5行目で出した値を入れたいのですが、戻り値の扱いが難しく、うまくいきません。

Answer 1

別のやり方として。

ArrayのrandomElement()メソッドは配列が空であったときのみnilを返しますので初めに配列が空でないかを確認しています。
エラー処理に関しては設計に合わせて変更してください。

guard !letters.isEmpty else { fatalError("元の配列が空") }

let result = (1...8).map { _ in letters.randomElement()! }

force unwrapを使っていますが、ここに来る場合にはlettersが空でないことがわかっているので安全です。

Answer 2

戻り値が難しい、というのは、おそらくrandomElement()の戻り値が Optional 型なので、次の行の insert(_:at:)でうまく使えない、ということだと解釈しています。

そうであれば、オプショナル値の扱いや、アンラップ、オプショナルバインディングをキーワードに調べ直していただく必要があるかと思います。

let letters = ["a","b","c"]
var result:[String] = []
for _ in 0...7 {
    // オプショナルバインディングを行う
    if let word = letters.randomElement() {
        //result.insert(word, at: n)
        // 末尾に追加するなら append
        result.append(word)
    }
}
print(result.joined())

ちなみに、ご質問のコードは関数の作成や利用という観点から見ると、for文の中で関数を宣言するなど、実用的な作りになっていませんので、その辺りも見直された方がいいかと思います。