Swiftでforループを1行で書きたい

Question

このようなメソッドがあります。

// 指定された長さのランダムな文字列を返す
func getRandomString(length: Int) -> String {
    var str = ""
    for _ in 0..<length {
        str += getRandomCharacter()
        // getRandomCharacter()はランダムな1文字がString型で返ってきます
    }
    return str
}

これを1行で書きたいと思っているのですが、以下のように書くと contextual type for closure argument list expects 1 argument, which cannot be implicitly ignored というエラーが出ます。

func getRandomString(length: Int) -> String {
    return (0..<length).map {getRandomCharacter()}.joined()
}

そこで、以下のように .map {$0} を途中に挟んでみると期待した結果が得られましたが、expression of type 'Int' is unused という警告が出ます。

func getRandomString(length: Int) -> String {
    return (0..<length).map {$0}.map {getRandomCharacter()}.joined()
}

もっと良い書き方を教えていただければ幸いです。

Answer 1

Xcodeの修正案のように引数を明示して下さい。

return (0..<length).map { _ in getRandomCharacter() }.joined()

後のコードのようにクロージャの中で$0を使っていればエラーにはなりませんが、map {$0}は意味がないので警告が出てしまっています。例えばこうすればエラーも警告も出ません。

return (0..<length).map { print($0); return getRandomCharacter() }.joined()

【追記】
後れを取ってしまったのでreduceいっときます！

return (0..<length).reduce("") { (result, _) in result + getRandomCharacter() }

Comments

[xekdwpsg]

reduceの方も大変参考になりました。ありがとうございます。

Answer 2

クロージャ式では使用しない引数は _ で捨てるようにしないといけません。

func getRandomString(length: Int) -> String {
    return (0..<length).map { _ in getRandomCharacter() }.joined()
}

Answer 3

これはうごきませーん。

ちょっと見方を変えて、これはどうですか？

let str = String(repeating: getRandomCharacter(), count: 10)

なんでこうですかね？

func getRandomString(length: Int) -> String {
    String(repeating: getRandomCharacter(), count: length)
}

return 省略するのがお作法なんですかね？

Comments

[xekdwpsg]

回答ありがとうございます。
その方法だと、 `getRandomCharacter()` で取得した文字を10回繰り返す、ということになり `"bbbbbbbbbb"` のようにランダムな文字列ではなくなってしまいますね。
`return` が省略できるのは忘れていました。ありがとうございます。

[MasakiHori]

String(repeating:, count: )は１つのインスタンスをcount数複製するものですので質問者さんの希望通りには動きません

[退会済みユーザー]

あーリピートの内部は再取得しなかったですか、ごめんなさい。

[TsukubaDepot]

あー、既に指摘済みでしたね。失礼しました。
---
String(repeating:count:)だと同じインスタンスをcount回繰り返して生成するので、全て同じ文字になりそうですが、いかがでしょうか。