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

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

ただいまの
回答率

87.35%

【swift2】JSONデータで取得したデータの表示速度が遅い

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,349

score 7

前提・実現したいこと

データベースのデータをJSONで吐き出すPHPにアクセスし
受け取ったデータをTableViewタイムラグを防いでシームレスに表示させたい。

発生している問題・エラーメッセージ

現在、掲示板アプリを作成していますが。
サーバーにあるデータベースからJSONデータで吐き出すPHPがあります。
そのPHPへアクセスし取得したJSONデータをTableViewでデータを表示しようとしてるのですが、
その表示速度があまりにも遅いので速度アップを図りたいです。
データの内容はテキストと画像です。

該当のソースコード

TableViewController.swift

import UIKit

class TableViewController: UITableViewController {

    var bbsData:[BbsData] = [BbsData]()
    var offset = 0
    var userId: String?
    @IBOutlet var filterBtn: UIBarButtonItem!

    func getData(){
        let url = NSURL(string:"データを取得するPHPへのURL")
        let config = NSURLSessionConfiguration.defaultSessionConfiguration()
        let session = NSURLSession(configuration: config)
        let req = NSMutableURLRequest(URL: url!)
        req.HTTPMethod = "POST"
        req.HTTPBody = ("データの絞込み用POSTパラメータ").dataUsingEncoding(NSUTF8StringEncoding)
        let task = session.dataTaskWithRequest(req, completionHandler: {
            (data,resp,err) in
            do{
                let json = try NSJSONSerialization.JSONObjectWithData(data!, options: NSJSONReadingOptions.MutableContainers) as! NSDictionary
                if json.count > 0{
                    if let bbsDatas = json["bbsData"] as? NSArray{
                        self.offset +=  bbsDatas.count
                        for data1 in bbsDatas{
                            let basebaseUrl: String = "DBからの画像を表示するPHPへのURL"
                            let commentId = data1["commentId"] as! String
                            let baseUrl = basebaseUrl + commentId
                            let url = baseUrl as NSString
                            let nameStr = data1["userName"] as? NSString
                            let ageStr = data1["age"] as? NSString
                            let prefStr = data1["pref"] as? NSString
                            let commentStr = data1["comment"] as? NSString
                            let userId = data1["userListId"] as? NSString
                            let gender = data1["gender"] as? NSString
                            self.setupBbsDatas(url, nameStr: nameStr, ageStr: ageStr, prefStr: prefStr, commentStr: commentStr, userId: userId, gender: gender)
                        }
                    }
                    self.tableView.reloadData()
                }
            }catch{
                print("error")
            }
        })
        task.resume()

    }
    override func viewDidLoad() {
        super.viewDidLoad()
        let backBtn = UIBarButtonItem()
        backBtn.title = "戻る"
        self.navigationItem.backBarButtonItem = backBtn
        getData()
    }


    @IBAction func filterClick(sender: UIBarButtonItem){
        let storyboard = UIStoryboard(name:"Main",bundle:nil)
        let nextView:UIViewController = storyboard.instantiateViewControllerWithIdentifier("filterSelect")
        nextView.modalTransitionStyle = UIModalTransitionStyle.PartialCurl
        navigationController?.pushViewController(nextView, animated: true)
    }

    override func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) {
        userId = bbsData[indexPath.row].userId as String
        tableView.deselectRowAtIndexPath(indexPath, animated: true)
        let storyboard = UIStoryboard(name:"DetailProf",bundle:nil)
        let nextView:UIViewController = storyboard.instantiateViewControllerWithIdentifier("detailProfile")
        nextView.modalTransitionStyle = UIModalTransitionStyle.PartialCurl
        let viewController = nextView as! DetailProf
        viewController.userListId = userId
        navigationController?.pushViewController(nextView, animated: true)


    }
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
            }

    override func numberOfSectionsInTableView(tableView: UITableView) -> Int {
                return 1
    }

    override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {

        return bbsData.count
    }


    override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        print(indexPath.row)
        let cell: bbsTableCell = tableView.dequeueReusableCellWithIdentifier("bbsDataCell", forIndexPath: indexPath) as! bbsTableCell    
        cell.setCell(bbsData[indexPath.row])
        if (self.bbsData.count - 1) == indexPath.row {
            getData()
        }
        //self.tableView.reloadData()
        return cell
    }

func setupBbsDatas(bbsImage: NSString, nameStr: NSString?, ageStr: NSString?, prefStr: NSString?, commentStr: NSString?, userId: NSString?, gender: NSString?){
        let data = BbsData(bbsImage: bbsImage, nameStr: nameStr, ageStr: ageStr, prefStr: prefStr, commentStr: commentStr, userId: userId, gender: gender)
        bbsData.append(data)
    }


}

Model.swift

import Foundation

class BbsData: NSObject{
    var bbsImage: NSString
    var nameStr: NSString
    var ageStr: NSString
    var prefStr: NSString
    var commentStr: NSString
    var userId: NSString
    var gender: NSString

    init(bbsImage: NSString, nameStr: NSString?, ageStr: NSString?, prefStr: NSString?, commentStr: NSString?, userId: NSString?, gender: NSString?){
        self.bbsImage = bbsImage

        if nameStr != nil{
            self.nameStr = nameStr!
        }else{
            self.nameStr = ""
        }
        if ageStr != nil{
           self.ageStr = ageStr!
        }else{
            self.ageStr = ""
        }
        if prefStr != nil{
            self.prefStr = prefStr!
        }else{
            self.prefStr = ""
        }
        if commentStr != nil{
            self.commentStr = commentStr!
        }else{
            self.commentStr = ""
        }
        if userId != nil{
            self.userId = userId!
        }else{
            self.userId = ""
        }
        if gender != nil {
            self.gender = gender!
        }else{
            self.gender = "非表示"
        }
    }

}

bbsTableCell.swift

import UIKit

class bbsTableCell: UITableViewCell {

    @IBOutlet var postImage: UIImageView!
    @IBOutlet var nameLabel: UILabel!
    @IBOutlet var ageLabel: UILabel!
    @IBOutlet var prefLabel: UILabel!
    @IBOutlet var commentLabel: UILabel!
    @IBOutlet var genderImage: UIImageView!

    override func awakeFromNib() {
        super.awakeFromNib()
        // Initialization code
    }

    override func setSelected(selected: Bool, animated: Bool) {
        super.setSelected(selected, animated: animated)

        // Configure the view for the selected state
    }

    func setCell(bbsData: BbsData){

        nameLabel.text = bbsData.nameStr as String
        ageLabel.text = bbsData.ageStr as String + "才"
        prefLabel.text = bbsData.prefStr as String
        commentLabel.text = bbsData.commentStr as String
        switch bbsData.gender {
            case "男性":
            genderImage.image = UIImage(named: "genderMale")

            case "女性":
            genderImage.image = UIImage(named: "genderFemale")

        default:
            genderImage.image = UIImage(named: "genderNodisplay")

        }
        let url = NSURL(string: bbsData.bbsImage as String)
        let requestUrl = NSURLRequest(URL: url!)
        NSURLConnection.sendAsynchronousRequest(requestUrl, queue: NSOperationQueue.mainQueue()){(response,data,error) -> Void in
            if error != nil{
                //print(error)
            }else{
                if let dlImage = UIImage(data: data!){
                    self.postImage.image = dlImage
                }
            }
        }

    }

}

調査したこと、試したこと

TableViewController.swiftのソースに以下のコードを追加してみましたが、
改善されませんでした。

override func viewWillAppear(animated: Bool){
    self.tableView.reloadData()
    super.viewWillAppear(animated)
}


データの読み込み時(表示がされていない時)に、フリック、タップを行うと描画がされるので、
描画周りでの問題だとは思っています。

補足情報(言語)

Xcode7.3
Swift2.*

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • 退会済みユーザー

    2016/06/27 14:04

    こちらの質問が他のユーザから「やってほしいことだけを記載した丸投げの質問」という指摘を受けました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

  • m_tsukasa

    2016/06/27 14:22

    説明ベタで申し訳ございません。
    どのように回答したらいいのかご教授願います。

    キャンセル

回答 1

checkベストアンサー

+1

Instrumentsで計測はしてみましたか?
もしまだしていないならTime profilerあたりでどこに時間がかかっているのかを調べることから始めてはどうでしょうか。

使い方がわからない場合は参考サイトをみてください。

参考

InstrumentsのTime Profilerを使って重たいメソッドを特定する

追記

ソース見てひとつ気がつきました
getData()内の完了ハンドラ内でtableView.reloadData()を呼んでいますが、ここはたぶんバックグラウンドスレッドで実行されています。
バックグラウンドスレッドからUIKitを触ったために表示が乱れているのだと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/06/29 10:28 編集

    追記の情報ありがとうございます。
    tableView.reloadData()の部分を
    dispatch_async(dispatch_get_main_queue(),{[tableView.reloadData()]})
    に変更したところ、希望通りの動作になりました。

    キャンセル

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

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

関連した質問

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