swiftで配列を関数の引数にしたときのmutable、immutable

Question

以下のような構造体と関数を作りました。

struct Data: Identifiable {
    var id: Int
    var string:String
}

func makeArray(array:[Data], str:String)->[Data]{
    array.append(Data(id:array.count, string:str))
    
    return array
}

関数内で、次のような警告が出ます。
Cannot use mutating member on immutable value: 'array' is a 'let' constant

可変長の配列を作る関数を作りたいのですが、どうしてもimmutableになってしまいます。
どうした良いでしょうか。

Answer 1

関数の引数はimmutableです。
varにコピーしてあげれば変更できるようになります。

func makeArray(array: [Data], str: String) -> [Data] {
    var result = array

    result.append(Data(id: array.count, string: str))

    return result
}

Swiftの配列は値型なので、最初の代入で、arrayからresultにコピーされます。
実際には、必要になるまでコピーしない最適化が行われるので、パフォーマンスを気にする必要はありません。

Answer 2

Swift の関数・メソッドの引数はデフォルトで immutable であり,
よほどの理由がない限りはそのまま immutable なものとして使うべきだと思います.

なので基本的には, array への追加は array を作った側ですべきかなと.

あとは「配列と, それを変更するメソッドを持つ構造体 or クラス」という形なら,
自身のプロパティを弄る分には問題ありませんのでそうして下さい.

なのでどちらかと言えば「どうして, そうしたいのか」が重要になってきます.
基本的には, 引数に渡した配列を弄らなければならない理由はそれほど多くないはずです.

...それでも, どうしても, 引数に配列を渡し, それを弄る必要がある, それが本質である, ということであれば, 手段はあります.
inout というキーワードです.
詳しくは Swift inout とかで調べたら使い方が出てくると思います.

Comments

[kagilinn]

あとこの設計のままであれば, `append` を使わずに `return array + Data(中略)` ですかね.
append で変更する理由はどちらにせよあんまり無いです.

Answer 3

仮引数はletの扱いなので、仮引数の内容を変更するのであれば、一度代入しなおす必要があります。

func makeArray(array:[Data], str:String)->[Data]{
    var array = array
    array.append(Data(id:array.count, string:str))

    return array
}