Question

我是iOS编程的新手。

通过使用下面的代码，我从数据库中检索图像并将其存储在数组中，然后以缩略图显示这些图像。

通过使用以下代码，一切正常。但我有2个问题

从数据库中检索图像花费的时间太长了。
我存储超过8个图像意味着它不显示图像，它正在终止应用程序。如果我拍摄下面的8个图像，则以缩略图显示图像。

任何机构都可以告诉我，这段代码中有什么错误？

NSString *docsDir;
NSArray *dirPaths;

// Get the documents directory
dirPaths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);

docsDir = [dirPaths objectAtIndex:0];
array=[[NSMutableArray alloc]init];
array1=[[NSMutableArray alloc]init];

// Build the path to the database file
databasePath =  [docsDir stringByAppendingPathComponent: @"Taukydataaa.db"];

NSFileManager *fn=[NSFileManager defaultManager];
NSError *error;
BOOL success=[fn fileExistsAtPath:databasePath];

if(!success)
{    
    NSString *defaultDBPath = [[[NSBundle mainBundle] resourcePath] stringByAppendingPathComponent:@"Taukydataaa.db"];
    success = [fn copyItemAtPath:defaultDBPath toPath:databasePath error:&error];
}

const char *dbpath = [databasePath UTF8String];

sqlite3_stmt    *statement;

if (sqlite3_open(dbpath, &contactDB) == SQLITE_OK)
{
    NSString *querySQL = [NSString stringWithFormat: @"select * from tauky"];

    const char *query_stmt = [querySQL UTF8String];

    if (sqlite3_prepare_v2(contactDB, query_stmt, -1, &statement, NULL) == SQLITE_OK)
    {
        while(sqlite3_step(statement) == SQLITE_ROW)
        {

            NSString* email_idField = [[NSString alloc] initWithUTF8String:(const char *) sqlite3_column_text(statement,1)];
            NSString* email_idField1 = [[NSString alloc] initWithUTF8String:(const char *) sqlite3_column_text(statement,0)];
            [array addObject:email_idField];
            [array1 addObject:email_idField1];

            blaukypath =[[NSMutableArray alloc]init];

            for (NSString* path in array)
            {
                [blaukypath addObject:[UIImage imageWithContentsOfFile:path]];
            }

            myScrollView = [[UIScrollView alloc]initWithFrame:CGRectMake(0.0, 0.0, 320.0, 840.0)];
            myScrollView.delegate = self;
            myScrollView.contentSize = CGSizeMake(320.0, 840.0);
            myScrollView.backgroundColor = [UIColor whiteColor];

            [self.view addSubview:myScrollView];

            float horizontal = 8.0;
            float vertical = 8.0;

            for(int i=0; i<[blaukypath count]; i++)
            {
                if((i%4) == 0 && i!=0)
                {
                    horizontal = 8.0;
                    vertical = vertical + 70.0 + 8.0;
                }

                buttonImage = [UIButton buttonWithType:UIButtonTypeCustom];

                [buttonImage setFrame:CGRectMake(horizontal, vertical, 70.0, 70.0)];
                [buttonImage setTag:i];

                [buttonImage setImage:[blaukypath objectAtIndex:i] forState:UIControlStateNormal];
                [buttonImage addTarget:self action:@selector(buttonImagePressed:) forControlEvents:UIControlEventTouchUpInside];
                [buttonImage setImage:[UIImage imageNamed:@"play.png"] forState:UIControlStateSelected];

                [myScrollView addSubview:buttonImage];

                horizontal = horizontal + 70.0 + 8.0;
            }

            [myScrollView setContentSize:CGSizeMake(320.0, vertical + 78.0)];

            // Do any additional setup after loading the view, typically from a nib.
            self.navigationItem.leftBarButtonItem = self.editButtonItem;

            UIBarButtonItem *done = [[UIBarButtonItem alloc] initWithBarButtonSystemItem:UIBarButtonSystemItemDone target:self action:@selector(insertNewObject:)];

            self.navigationItem.rightBarButtonItem = done;

            [self.myScrollView addSubview:image];
        }

        sqlite3_finalize(statement);
    }

    sqlite3_close(contactDB);
}

Answer 1

有几点意见：

您已将滚动视图的构建放在循环中，您正在从表中读取数据。例如，如果数据库中有9行，则会有9个滚动视图，第一个包含一个图像，第二个包含两个，第三个包含三个，等等，总共有45个图像。我真的怀疑你的意思是什么。
你应该有一个循环从数据库中读取字符串（此时不创建图像...只存储图像路径）并填充数组。然后，您可以拥有一个单独的循环（理想情况下，一个完全独立的例程）来构建您的UI。你真的应该将UI与数据库交互隔离开来。任何由图像对象数组组成的内容都会出现问题：只维护图像路径数组。

这个创建大量不需要的图像的问题肯定会减慢应用程序的速度并消耗内存。根据图像的大小，您甚至可能会耗尽内存并崩溃。

至少，你应该解决这个问题。

还有一些其他问题：

崩溃的另一个潜在原因是，如果阵列中的某个图像路径未将自身解析为图像路径。因此，imageWithContentsOfFile将返回nil，并且任何将nil添加到数组的尝试都将崩溃。在尝试使用之前，请确保测试是否成功找到/加载了图像。
你没有说出图像有多大，但是如果它们大于140x140，你真的想要创建它们的屏幕分辨率再现。如果图像非常大，虽然它可以在70x70渲染它们，但你会耗尽内存来获得完整的图像。当同时显示这么多图像时，如果你不小心使用屏幕分辨率图像，你会很快消耗你的记忆。
根据您在数组中引用的图片数量，您甚至可以考虑一个处理UIScrollViewDelegate方法scrollViewDidScroll的模型，并只创建UIImageView个对象并填充它们滚动到视图中的相应image属性（并删除那些已滚出视图的属性）。如果定位到iOS 6，您可以使用UICollectionView而不是手动生成的滚动视图，并且您将自动获得某些功能（只要您的数组是图像路径数组而不是图像对象数组）。

如果你还在崩溃，你应该与我们分享崩溃的细节（如果你没有告诉我们你得到了什么样的异常/错误，我们只是在猜测）。此外，如果还没有，请实施一个didReceiveMemoryWarning，如果没有其他内容，则告诉您何时有内存警告，以便您可以识别问题并解决问题。并确保在设备上测试这样的内存饥饿的应用程序，因为有许多内存相关的问题不会在模拟器上显示出来，但会在设备上显示出来。

虽然上述内容侧重于崩溃问题，但您还询问了性能问题。特别是如果您没有使用UICollectionView，而是手动构建滚动视图，那么您可能希望异步执行此操作。基本的想法是你将有一个后台操作来创建UIImage对象，然后调度UI任务将它添加到主队列中的滚动视图（因为你从不在后台队列中执行UI内容）。但是，这样，用户可以在图像弹出到位时开始使用应用程序。

您可能需要考虑一些微妙的问题（在多个线程上执行数据库交互需要一些仔细的实现;确保在主队列上发生UI更新等等），但是如果您仍然遇到性能问题，这是典型的解决方案。

尽管如此，我还是首先关注崩溃，然后再解决性能问题。

从iphone中检索图像形式数据库需要太长时间

1 个答案: