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

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

新規登録して質問してみよう
ただいま回答率
85.48%
メモリリーク

メモリリークは、プログラムファイルがメモリの解放に失敗した時に起こります。

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

Q&A

解決済

1回答

809閲覧

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

hazata

総合スコア13

メモリリーク

メモリリークは、プログラムファイルがメモリの解放に失敗した時に起こります。

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

0グッド

2クリップ

投稿2018/03/22 02:41

編集2018/03/22 04:05

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

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

swift

1class TickerViewController: UIViewController { 2 3 @IBOutlet weak var cc_buy: UILabel! 4 @IBOutlet weak var cc_sell: UILabel! 5 6 7 @IBOutlet weak var amount_text: UITextField! 8 9 weak var timer: Timer? // 情報取得タイマー 10 11 ... 12    13 override func viewWillAppear(_ animated: Bool) { 14 super.viewWillAppear(true) 15 16 //タイマーの設定 17 timer = Timer.scheduledTimer(timeInterval: 3.0, target: self, selector: #selector(self.update), userInfo: nil, repeats: true) 18 19 timer?.fire() 20 21 } 22 23 24 @objc func update(tm: Timer) { 25 26 27 let ticker = Ticker_master_API() 28 _ = ticker.cc_ticker(cc_buy_rate: cc_buy, cc_sell_rate: cc_sell, amount: amount_text) 29 30 } 31 32}

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

swift

1class Ticker_master_API { // 2 3 func cc_ticker(cc_buy_rate:UILabel,cc_sell_rate:UILabel,amount:UITextField)-> (Bool){ 4 5 var bids_rate_array_cc:[String] = [] 6 var bids_amount_array_1_cc:[String] = [] 7 var bids_amount_array_cc:[Double] = [] 8 9 10 var bids_rate_ok_cc:String = String() 11 12 13 //key 14 var key1:Bool = true 15 16 17 // URLを指定してオブジェクトを作成(コインチェック) 18 let stringUrl_cc = "https://coincheck.com/api/order_books" 19 let url_cc = URL(string: stringUrl_cc) 20 let request_cc = URLRequest(url: url_cc!) 21 22 // コンフィグを指定してHTTPセッションを生成 23 let session_cc = URLSession(configuration: URLSessionConfiguration.default, delegate: nil, delegateQueue: OperationQueue.main) 24 25 // HTTP通信を実行する 26 // ※dataにJSONデータが入る 27 let task_cc:URLSessionDataTask = session_cc.dataTask(with: request_cc, completionHandler: {cc_data, responce, error in 28 // エラーがあったら出力 29 if error != nil { 30 print("読み込みエラー(error!)") 31 key1 = false 32 return; //セミコロンで処理を抜ける 33 34 } 35 36 37 DispatchQueue.main.async { 38 // データ取得後の処理 39 40 41 let cc_json = JSON(cc_data!) 42 43 44 //レートの取得と配列代入 45 for i in 0...9{ 46 47 let bid_x_cc:String = String(describing: cc_json["bids"].array![i][0]) 48 bids_rate_array_cc.append(bid_x_cc) //買い板 レート 49 50 } 51 52 //買いレートの最新板のレートを取得し、JPYの購入金額から基準数量をだす 53 54 let bit_base_amount:Double = Double(amount.text!)! 55 56 //数量を取得と仮配列に代入 57 for i in 0...9{ 58 59 let bid_x2_cc:String = cc_json["bids"].array![i][1].string! 60 bids_amount_array_1_cc.append(bid_x2_cc) //買い板 レート 61 62 } 63 64 //数量を変換と配列に代入(前の板の数量を加算していく) 65 let bid_amo1_cc = Double(bids_amount_array_1_cc[0]) 66 let bid_amo2_cc = Double(bids_amount_array_1_cc[1]) 67 let bid_amo3_cc = Double(bids_amount_array_1_cc[2]) 68 let bid_amo4_cc = Double(bids_amount_array_1_cc[3]) 69 let bid_amo5_cc = Double(bids_amount_array_1_cc[4]) 70 let bid_amo6_cc = Double(bids_amount_array_1_cc[5]) 71 let bid_amo7_cc = Double(bids_amount_array_1_cc[6]) 72 let bid_amo8_cc = Double(bids_amount_array_1_cc[7]) 73 let bid_amo9_cc = Double(bids_amount_array_1_cc[8]) 74 let bid_amo10_cc = Double(bids_amount_array_1_cc[9]) 75 76 let bid_amo1_last_cc = bid_amo1_cc! 77 bids_amount_array_cc.append(bid_amo1_last_cc) 78 let bid_amo2_last_cc = bid_amo1_cc! + bid_amo2_cc! 79 bids_amount_array_cc.append(bid_amo2_last_cc) 80 let bid_amo3_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! 81 bids_amount_array_cc.append(bid_amo3_last_cc) 82 let bid_amo4_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! 83 bids_amount_array_cc.append(bid_amo4_last_cc) 84 let bid_amo5_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! 85 bids_amount_array_cc.append(bid_amo5_last_cc) 86 let bid_amo6_last_cc = bid_amo1_cc! + bid_amo2_cc! + bid_amo3_cc! + bid_amo4_cc! + bid_amo5_cc! + bid_amo6_cc! 87 bids_amount_array_cc.append(bid_amo6_last_cc) 88 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! 89 bids_amount_array_cc.append(bid_amo7_last_cc) 90 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! 91 bids_amount_array_cc.append(bid_amo8_last_cc) 92 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! 93 bids_amount_array_cc.append(bid_amo9_last_cc) 94 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! 95 bids_amount_array_cc.append(bid_amo10_last_cc) 96 97 98 //数量による比較と表示価格の選定 99 100 for i in 0 ... 9 { 101 102 if(bit_base_amount <= bids_amount_array_cc[i]){ 103 104 bids_rate_ok_cc = bids_rate_array_cc[i] 105 106 break 107 108 }// if 109 110 }// for 111 112 cc_buy_rate.text = String(bids_rate_ok_cc) 113 114 key1 = false 115 116 }// async 117 118 }) 119 120 // HTTP通信を実行 121 122 task_cc.resume() 123 124 return key1 125 126 }//func

イメージ説明

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

fuzzball

2018/03/22 02:48

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

2018/03/22 04:29 編集

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

2018/03/22 06:39

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

2018/03/22 07:00

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

回答1

0

ベストアンサー

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

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

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

投稿2018/03/22 07:15

fuzzball

総合スコア16731

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

hazata

2018/03/22 12:43

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問