質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.77%

初心者です。Swift4でメモリリークします。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 3,062

score 12

プログラム6ヶ月目の初心者です。swift4で取引所のAPIに3秒ごとにアクセスし、買値をJsonにて取得するものを作成したのですが、少しずつメモリリークしているようで、メモリの値が上がっていきます。
xcodeの「Debug Memory Graph」で調査しましたら紫色の「!」が17個発生していること以外わかりませんでした。以下がそのコードです。
メモリリークを止める方法ご教示宜しくお願い致します。

タイマーで呼び出すViewControllerクラス

class TickerViewController: UIViewController {

    @IBOutlet weak var cc_buy: UILabel!
    @IBOutlet weak var cc_sell: UILabel!


    @IBOutlet weak var amount_text: UITextField!

    weak var timer: Timer? // 情報取得タイマー

                 ...
    
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(true)

        //タイマーの設定
        timer = Timer.scheduledTimer(timeInterval: 3.0, target: self, selector: #selector(self.update), userInfo: nil, repeats: true)

        timer?.fire()

    }


    @objc func update(tm: Timer) {


        let ticker = Ticker_master_API()
       _ = ticker.cc_ticker(cc_buy_rate: cc_buy, cc_sell_rate: cc_sell, amount: amount_text)

     }

}

取引所にアクセスして値を取って来るクラス

class Ticker_master_API {  //

    func cc_ticker(cc_buy_rate:UILabel,cc_sell_rate:UILabel,amount:UITextField)-> (Bool){

        var bids_rate_array_cc:[String] = []
        var bids_amount_array_1_cc:[String] = []
        var bids_amount_array_cc:[Double] = []


        var bids_rate_ok_cc:String = String()


        //key
        var key1:Bool = true


        // URLを指定してオブジェクトを作成(コインチェック)
        let stringUrl_cc = "https://coincheck.com/api/order_books"
        let url_cc = URL(string: stringUrl_cc)
        let request_cc = URLRequest(url: url_cc!)

        // コンフィグを指定してHTTPセッションを生成
        let session_cc = URLSession(configuration: URLSessionConfiguration.default, delegate: nil, delegateQueue: OperationQueue.main)

        // HTTP通信を実行する
        // ※dataにJSONデータが入る
        let task_cc:URLSessionDataTask = session_cc.dataTask(with: request_cc, completionHandler: {cc_data, responce, error in
            // エラーがあったら出力
            if error != nil {
                print("読み込みエラー\(error!)")
                key1 = false
                return; //セミコロンで処理を抜ける

            }


            DispatchQueue.main.async {
                // データ取得後の処理


                let cc_json = JSON(cc_data!)


                    //レートの取得と配列代入
                    for i in 0...9{

                        let bid_x_cc:String = String(describing: cc_json["bids"].array![i][0])
                        bids_rate_array_cc.append(bid_x_cc) //買い板 レート

                    }

                   //買いレートの最新板のレートを取得し、JPYの購入金額から基準数量をだす

                    let bit_base_amount:Double = Double(amount.text!)!

                    //数量を取得と仮配列に代入
                    for i in 0...9{

                        let bid_x2_cc:String = cc_json["bids"].array![i][1].string!
                        bids_amount_array_1_cc.append(bid_x2_cc) //買い板 レート

                    }

                    //数量を変換と配列に代入(前の板の数量を加算していく)
                    let bid_amo1_cc = Double(bids_amount_array_1_cc[0])
                    let bid_amo2_cc = Double(bids_amount_array_1_cc[1])
                    let bid_amo3_cc = Double(bids_amount_array_1_cc[2])
                    let bid_amo4_cc = Double(bids_amount_array_1_cc[3])
                    let bid_amo5_cc = Double(bids_amount_array_1_cc[4])
                    let bid_amo6_cc = Double(bids_amount_array_1_cc[5])
                    let bid_amo7_cc = Double(bids_amount_array_1_cc[6])
                    let bid_amo8_cc = Double(bids_amount_array_1_cc[7])
                    let bid_amo9_cc = Double(bids_amount_array_1_cc[8])
                    let bid_amo10_cc = Double(bids_amount_array_1_cc[9])

                    let bid_amo1_last_cc = bid_amo1_cc!
                    bids_amount_array_cc.append(bid_amo1_last_cc)
                    let bid_amo2_last_cc = bid_amo1_cc! + bid_amo2_cc!
                    bids_amount_array_cc.append(bid_amo2_last_cc)
                    let bid_amo3_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc!
                    bids_amount_array_cc.append(bid_amo3_last_cc)
                    let bid_amo4_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc!
                    bids_amount_array_cc.append(bid_amo4_last_cc)
                    let bid_amo5_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc!
                    bids_amount_array_cc.append(bid_amo5_last_cc)
                    let bid_amo6_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! + bid_amo6_cc!
                    bids_amount_array_cc.append(bid_amo6_last_cc)
                    let bid_amo7_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! + bid_amo6_cc! + bid_amo7_cc!
                    bids_amount_array_cc.append(bid_amo7_last_cc)
                    let bid_amo8_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! + bid_amo6_cc! + bid_amo7_cc! + bid_amo8_cc!
                    bids_amount_array_cc.append(bid_amo8_last_cc)
                    let bid_amo9_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! + bid_amo6_cc! + bid_amo7_cc! + bid_amo8_cc! + bid_amo9_cc!
                    bids_amount_array_cc.append(bid_amo9_last_cc)
                    let bid_amo10_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! + bid_amo6_cc! + bid_amo7_cc! + bid_amo8_cc! + bid_amo9_cc! + bid_amo10_cc!
                    bids_amount_array_cc.append(bid_amo10_last_cc)


                    //数量による比較と表示価格の選定

                    for i in 0 ... 9 {

                        if(bit_base_amount <= bids_amount_array_cc[i]){

                            bids_rate_ok_cc = bids_rate_array_cc[i]

                            break

                        }// if

                    }// for

                    cc_buy_rate.text = String(bids_rate_ok_cc)

                    key1 = false

            }// async

        })

        // HTTP通信を実行

        task_cc.resume()

        return key1

    }//func

イメージ説明

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • fuzzball

    2018/03/22 11:48

    画像が表示されていません。

    キャンセル

  • fuzzball

    2018/03/22 13:28 編集

    タイマーの間隔を長くしても同じでしょうか?(10秒とか30秒とか)あと、エラーは実行直後から発生していますか?しばらくしてからでしょうか?

    キャンセル

  • hazata

    2018/03/22 15:39

    fuzzball様、ありがとうございます。timerの間隔を30秒にして実験してみましたが、タイマーが呼ばれてすぐエラーが発生しています。起動直後にメモリが一定まで上昇し、そこからしばらく平行に推移して、タイマーで関数が呼ばれた段階でピョンと跳ね上がり、上がったぶんの80%くらい下がって(小山状になり)また平行に推移しています。その誤差分の20%くらいが累積していく感じです。わかりづらい返信で申し訳ありません。どうぞよろしくお願いします。

    キャンセル

  • fuzzball

    2018/03/22 16:00

    たぶん関係ないと思いますが、DispatchQueue.main.async { を削除してもらえますか?(対になるカッコも)そこは元々メインスレッドなので必要ないです。

    キャンセル

回答 1

checkベストアンサー

0

NSURLSessionがメモリリークしてしまうのをなんとかした

URLSessionは自分で解放しないといけないようです。
毎回生成する必要もないと思うので、session_ccをインスタンス変数にして、ViewControllerを開くときに生成、閉じるときに解放すればいいと思います。

もしくはURLSession.sharedを使うか。(これ使うとハンドラの中はメインスレッドじゃなくなるので注意)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/03/22 21:43

    fuzzballさん、ありがとうございました!ご回答と記事を元に解放処理を追加したら嘘のようにエラーが消えました。1週間悩んでいたことが、数時間で解決できて本当に感謝いたします。今後ともよろしくお願いいたします!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.77%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る