0

カメラとカメラロールを使って写真をコレクションビューに保存することのできるアプリを作っているのですが、画像を保存する箇所で処理が遅く数秒かかってしまい非常に煩わしいです。

SubViewController.m

- (void)addSelectedPicture:(SubjectViewController *)controller item:       (UIImage *)item
{
    //_records(NSMutableArray)
    Record *record = _records[_imageSelectionIndexPath.section];
    [record.images addObject:item];

    //ここが時間かかる$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$①
    [[LessonManager sharedManager] saveLessons];

    [self.collectionView reloadData];
}

LessonManager.h

#import <Foundation/Foundation.h>
#import "Lesson.h"

@interface LessonManager : NSObject

@property (nonatomic) NSMutableArray *lessons;

+ (instancetype)sharedManager;
- (void)saveLessons;

@end

LessonManager.m

#import "LessonManager.h"

@implementation LessonManager

+ (instancetype)sharedManager {
    static LessonManager *manager = nil;
    static dispatch_once_t onceToken;
    dispatch_once(&onceToken, ^{
        manager = [[self alloc] init];
    });
    NSLog(@"1場所");
    return manager;
}

- (instancetype)init
{
    self = [super init];
    if (self) {
        self.lessons = [NSMutableArray arrayWithArray:[Lesson     fetchLessons]];
    }
    NSLog(@"2場所");
    return self;
}

- (void)saveLessons
{
    NSLog(@"4場所");
    //ここが遅い$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$②
    [Lesson saveLessons:self.lessons];
    NSLog(@"3場所");
}

@end

Lesson.h

#import <Foundation/Foundation.h>
#import "Record.h"

@interface Lesson : NSObject <NSCoding>

@property (nonatomic) NSString *name;
@property (nonatomic) NSString *teacher;
@property (nonatomic) NSString *room;
@property (nonatomic) NSMutableArray *records;

+ (NSArray *)fetchLessons;
+ (void)saveLessons:(NSArray *)lessons;

@end

Lesson.m

#import "Lesson.h"

@implementation Lesson

+ (NSArray *)fetchLessons
{
    NSLog(@"8場所");
    NSUserDefaults *userDefaults = [NSUserDefaults  standardUserDefaults];
    NSData *data = [userDefaults dataForKey:@"lessons"];
    NSArray *array = [NSKeyedUnarchiver unarchiveObjectWithData:data];
    if (!array) {
        array = [NSArray array];
    }
    return array;
}

+ (void)saveLessons:(NSArray *)lessons
{
   //ここが遅い$$$$$$$$$$$$$$$$$$$$$$$$③
    NSData *data = [NSKeyedArchiver archivedDataWithRootObject:lessons];
    NSUserDefaults *userDefaults = [NSUserDefaults  standardUserDefaults];
    [userDefaults setObject:data forKey:@"lessons"];
    [userDefaults synchronize];
}

- (instancetype)init
{
    self = [super init];
    if (self) {
        self.records = [NSMutableArray array];
    }
return self;
}

@end

順番としては①②③と実行されていくと思うのですが、Lesson.mの
+ (void)saveLessons:(NSArray *)lessons{}
の③のうちの
NSData *data = [NSKeyedArchiver archivedDataWithRootObject:lessons];
の部分が遅くアプリが数秒固まったようになってしまいます。

メモリは③の処理の部分で最大になります。メモリの使いすぎなのでしょうか...
画像の説明をここに入力

なにか解決する方法はありませんか、どなたかよろしくお願いします。

2
  • 1
    最終的に保存しているdataのサイズはどれぐらいでしょう? サイズによっては、この辺りが参考になるかと。qiita.com/yuky_az/items/13dfc156db118db04f17
    – See.Ku
    Commented 2015年6月5日 1:35
  • コメントありがとうございます。問題のNSDataのしたにNSLog(@"File size is : %.2f MB",(float)myData.length/1024.0f/1024.0f);で確認したところ、現時点で70MB近くありました。画像を多く保存するアプリなのでもっと増えそうです...
    – h_got
    Commented 2015年6月5日 8:19

1 件の回答 1

1

どうしてもNSUserDefaultsに保存しないといけないのであれば別ですが、そうでなければ、dataを直接ファイルに保存するようにしてはどうでしょうか?

たぶん、こんな感じになるかと思います。

+ (NSArray*)fetchLessons
{
    NSLog(@"8場所");
    NSString* path = [self makeLibraryPath];
    NSData* data = [[NSData alloc] initWithContentsOfFile:path];

    NSArray* array = [NSKeyedUnarchiver unarchiveObjectWithData:data];
    if (!array) {
        array = [NSArray array];
    }
    return array;
}

+ (void)saveLessons:(NSArray*)lessons
{
    //ここが遅い$$$$$$$$$$$$$$$$$$$$$$$$③
    NSData* data = [NSKeyedArchiver archivedDataWithRootObject:lessons];

    NSString* path = [self makeLibraryPath];
    [data writeToFile:path atomically:YES];
}

/// LibraryDirectoryのファイルへのパスを生成
+ (NSString*)makeLibraryPath
{
    NSArray* paths = NSSearchPathForDirectoriesInDomains(NSLibraryDirectory, NSUserDomainMask, YES);
    NSString* dir = [paths objectAtIndex:0];
    return [dir stringByAppendingPathComponent:@"data.dat"];
}

保存するパスやファイル名は、適当なものをいれてあるので自分で確認して下さい。サイズがさらに大きくなるのであれば、配列をまとめて保存するのではなく、それぞれ別に保存したほうがいいかもしれません。

1
  • 質問に答えていただきありがとうございました。 最初はよかったのですが、サイズがさらに大きくなってまた問題が解決できなくなってしまいました。 後半の配列をそれぞれ別に保存する方法も教えていただけないでしょうか。どうかよろしくお願いします。
    – h_got
    Commented 2015年9月28日 16:56

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

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