[Swift] UIImageを持つTableViewCellの非同期表示
解決済
回答 1
投稿
- 評価
- クリップ 1
- VIEW 2,385
実現したいこと
画像を持つUITableViewのCellをキャッシュを利用して非同期に表示させたいのですが、キャッシュが働いてくれなくて困っています。
開発環境は XCode(8.2.1)、 Swift3.0 です
発生している問題
DBから取得した画像(UIImage
)を画像名文字列(NSString
)をKeyとして受け取るNSCache
のサブクラス(ImageCache
)を作成したのですが、ImageCache#setObject(AnyObject, forKey: AnyObject)
でセット出来ていないのか、ImageCache#object(forKey: AnyObject)
の返値がnilとなってしまいます(デバッガで確認済み)。
TableViewの表示自体に問題はなく、キャッシュが動いていないだけです。エラー文は出ていません。
該当のソースコード
struct ResultData {
let name : String!
let imgStr : String!
}
//メインスレッド
class ResultTableViewController: UITableViewController {
var cellArray : [ResultData] = []
override func viewDidLoad() {
super.viewDidLoad()
let query : NCMBQuery = NCMBQuery.init(className: "ImageTest")
query.whereKey("Category", equalTo: "TestButton")
query.findObjectsInBackground({(/*NSArray*/ objects, /*NSError*/ error) in
if error != nil {
print(error.debugDescription)
} else {
for object in objects! {
self.cellArray.append(
ResultData(name: (object as! NCMBObject).object(forKey: "Name") as! String,
imgStr: (object as! NCMBObject).object(forKey: "Image") as! String))
}
self.tableView.reloadData()
}
})
}
override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return self.cellArray.count
}
override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCell(withIdentifier: "Results")! as! CustomCell
cell.result = cellArray[indexPath.row]
return cell
}
}
//画像付きカスタムUITableViewCell
class CustomCell: UITableViewCell {
@IBOutlet weak var thumbnail: UIImageView!
@IBOutlet weak var name: UILabel!
//イテレーション
var generation: Int = 0
override func prepareForReuse() {
super.prepareForReuse()
self.generation += 1
self.result = nil
}
var result: ResultData? {
didSet {
self.name.text = result?.name
self.thumbnail.image = nil
self.thumbnail.alpha = 0
if let result = self.result {
let generation = self.generation
weak var wcell = self
let cache = ImageCache()
//画像文字列で検索してImageCache内に画像があったら表示、無かったらDBから取得して表示
if let img = cache.object(forKey: result.imgStr as AnyObject) {
print("got from ImageCache!")
wcell?.thumbnail.image = img as? UIImage
wcell?.thumbnail.alpha = 0
UIView.animate(withDuration: 0.25, animations: {() -> Void in
wcell?.thumbnail.alpha = 1
})
} else {
let file = NCMBFile.file(withName: result.imgStr, data: nil) as! NCMBFile
file.getDataInBackground({(/*NSData*/ data, /*NSError*/ error) in
if error != nil {
print(error.debugDescription)
} else if let cell = wcell, let image = UIImage(data: data!), cell.generation == generation {
cache.setObject(image, forKey: result.imgStr as AnyObject)
cell.thumbnail.image = image
cell.thumbnail.alpha = 0
UIView.animate(withDuration: 0.25, animations: {() -> Void in
cell.thumbnail.alpha = 1
})
}
})
}
}
}
}
}
//画像を保持するNSCacheのサブクラス
class ImageCache : NSCache<AnyObject, AnyObject> {
//シングルトン
static let sharedInstance = ImageCache()
}
試したこと
最初はキャッシュのライブラリについて調べたのですが、どれもURLから画像を取得し、キャッシュする仕様のものばかりでした。
僕のやりたいことはDB(NCMB)から画像を取得し、単純に画像名文字列をKeyとしてUIImageをキャッシュに格納するというものなので、URLは使いません。
一応HanekeCache
の#setImage()
と#fetchImage()
を試してみましたが、思う様に動かなかったため、自分でキャッシュクラスを作成して実装してみることにしました。
知りたいこと
・バグの解決方法(NSCacheの扱い方?tableViewCellの仕様?)
・(あったら)URLを扱わない、このコードの仕様に合うキャッシュライブラリ
どなたかご教授願います。
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
+2
let cache = ImageCache()
じゃなくて、
let cache = ImageCache.sharedInstance
ですね。
今のコードだと毎回違うインスタンスが生成されてしまいますのでキャッシュされません。
おまけ
Singleton側を、
final class ImageCache : NSCache<AnyObject, AnyObject> {
static let sharedInstance = ImageCache()
private init() {
}
}
とすると、
final
で継承禁止。private init()
の追加で、外部からのインスタンス生成禁止。(ImageCache()でエラーになる)
となります。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう!
- ただいまの回答率 88.13%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
2017/01/10 12:58
Singletonの扱い方が間違っていたのですね、勉強になります。
ご回答ありがとうございました!