swift 無限ループしてしまう

Question

class FizzBuzz{
    
     let value = Decide()
    
    var arrayResult: [String]{
        get{
            return self.arrayResult
        }
        set(newValue){
            self.arrayResult = newValue
        }
    }
    
   
    
    func processed(number:Int) -> [String]{
        for number in 1..<number{
            arrayResult[number] = value.decide(by: number)
        }
        return arrayResult
    }
    

}
class Decide {
    let value = IsFizzBuzz()
    func decide(by number:Int) -> String{
        if (value.isFizz(a: number) && value.isBuzz(a: number)){return "FizzBuzz"}
        if (value.isFizz(a: number))                           {return "Fizz"}
        if (value.isBuzz(a: number))                           {return "Buzz"}
        return ""
    }
}

class IsFizzBuzz{
    func isFizz(a number:Int) -> Bool{
        if(number%3 == 0 ){
            return true
        }
        return false
    }
    
    func isBuzz(a number:Int) -> Bool{
        if(number%5 == 0){
            return true
        }
        return false
    }
}

processedを呼ぶとarrayResult[number] = value.decide(by: number)で無限にゲッターを読んでしまうようです。
どのようにしたらいいですか？

具体できにしたいこと
arryResultをjavaで言うフィールドのように使用したい
value.decide(by: number)の結果を配列それぞれの配列に保持
ループが終わり次第配列全てをリターン

Answer 1

arrayResultに定義しているgetter/setterは自分自身の実装を参照しているため無限に再帰呼び出ししてしまうのでしょうね。swiftコンパイラーが文句を言ってくれると思いきや何もいわずにコンパイルが成功してしまいますね・・・なんとなくコンパイラーの「さぼり」に見えます。

対処：

getter/setter内で自分自身を参照しないようにすれば動くようにはなると思います。そのためにはarrayResultとは別のインスタンス変数を用意しておきそちらを参照するように変更する必要がありますが・・・arrayResultになぜgetter/setterを定義したいのでしょう？ご質問のコードを拝見する限り意味がない気がします。

そもそもインスタンス変数にする必要もない気がします。例えばいつ長さが決まるかと言えばprocessedメソッドが呼ばたときですね。単にprocessedメソッドのローカル変数として宣言すれば済むと思います。

func processed(number:Int) -> [String]{
    var arrayResult = Array(repeating: "", count: number)
    for number in 1..<number {
        arrayResult[number] = value.decide(by: number)
    }
    return arrayResult
}

Comments

[L85A2]

ご指摘ありがとうございます

今swiftを勉強中でどのような記述の仕方ができるのかや、動作はどうなっているのかを学ぶ為にgetter/setterを使用していました。

無駄なことを記述してしまっていることに関しては、技術不足な点もしくは、試し書きをしている最中だと思ってください。まだコードをうまく記述できなく学ぶことが多いいと自覚しています。

わかりやすい解説でしたありがとうございます。

[KSwordOfHaste]

なるほど、納得しました。
いろいろ試しながらというのは理解しやすくなるのでよい方法と思います！

Answer 2

arrayResult関係の処理が何をしているのかよく分からないのですが、

var arrayResult: [String]{
    get{
        return self.arrayResult
    }
    set(newValue){
        self.arrayResult = newValue
    }
}

を

var arrayResult = [String]()

にして、

func processed(number:Int) -> [String]{
    for number in 1..<number{
        arrayResult[number] = value.decide(by: number)
    }
    return arrayResult
}

を、

func processed(number:Int) -> [String]{
    arrayResult = []
    for number in 1..<number{
        arrayResult += [value.decide(by: number)]
    }
    return arrayResult
}

にするのではダメでしょうか？