0

以下のコードが正しいかも含めての質問になります。
XCode9.0でSwift4のコードになります。

下記コードで実行しようとしたところ
error: Execution was interrupted, reason: EXC_BAD_ACCESS (code=2, address=0x7fff5c02cff8).という例外が発生しました。
1.理由をご教示いただきたいです
2.そもそも下のコードは誤りでしょうか?
クラス内に自身のインスタンスを持ったり、自身のインスタンスの宣言
 自身のインスタンスを格納する配列定義等は可能でしょうか?

class Test{
    var test:Double{
        get{
            return self.test+100
        }
        set(a){
            self.test=a
        }
    }

    var testList:Array<Test> = Array()

    init(a:Double){
        self.test = a
        addTest(a:a)
    }

    func addTest(a:Double){
        let t:Test = Test(a:a)
        testList.append(t)
    }
}

var test1:Test = Test(a:1)  ※←例外表示場所
print(test1.testList[0].test)
2
  • 1
    Testクラスのイニシャライザの中でaddTest(a:)を呼んでいる、addTest(a:)の中でTestクラスのイニシャライザを呼んでいる、と言うことで無限再帰になっています。あなたのコードはイニシャライザを呼ぶだけで必ず無限再起を引き起こすのですから「誤り」と言えるでしょう。testListはインスタンス変数でないといけないのでしょうか?「クラス内に自身のインスタンスを持ったり」することは可能ですが、あなたのtestListはどのような用途を想定しているのか不明なので、どう対処すべきかはなんとも言えません。
    – OOPer
    Commented 2017年10月16日 11:58
  • 3
    addTest()もそうなのですが、testのgetterとsetterで自身のself.testを呼び出しているところも無限‌​再帰を引き起こしています。クラス内に自身のインスタンスを持ったり、自身のインスタンスを格納する配列を持つことは可能ですが、現在のコードは無限‌​に同じ呼び出しを繰り返して終わらないので誤りです。 Commented 2017年10月16日 12:06

1 件の回答 1

1

1の回答:

  1. test1 を定義して、Test型の初期化子(init())が呼び出されます。ここまでは問題ありません。
  2. self.test = a test変数にaを代入していますね。これも問題ありません。
  3. addTest()関数を呼び出します。

addTest() 関数 {

  1. let t: Test = Test(a: a) ここで問題が発生します。まだまだ見ていきましょう。
  2. t を定義して、Test型の初期化子(init())が呼び出されます。

初期化子 {

  1. self.test = a test変数にaを代入
  2. addTest()関数を呼び出します。ここでもう一度addTest()関数に入ります。

addTest() 関数 {

  1. 分かりましたでしょうか。初期化または関数の実行を止めない限り永遠にループしてしまいます。この後も永遠に処理は続きます…。ループしすぎて「もう我慢できない!」と、システムが自動的にプログラムを落とした結果、あのエラーが現れます。

}

}

}

2の回答: YES。上の理由で誤りです。
一定回数繰り返すと止まるように、

init(a: Double, count: Int = 0){
    self.test = a
    addTest(a: a, count + 1)
}

func addTest(a: Double, count: Int){
    if count < 5 {
        let t: Test = Test(a: a, count: count)
        testList.append(t)
    }
}

こういったやり方でループした数をカウントすればOKだと思います(もし無限に続かないといけない場合は別ですが)。

メモ: var testList:Array<Test> = Array()var testList: [Test] = []

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。