2

我是 iOS 编程的新手。

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

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

  1. 从数据库中检索图像花费的时间太长。
  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);
}
4

1 回答 1

1

几点观察：

  • 您已将滚动视图的构建放在从表中读取数据的循环中。例如，如果您的数据库中有 9 行，您将有 9 个滚动视图，第一个有一个图像，第二个有两个，第三个有三个，等等，总共 45 个图像。我真的怀疑这就是你的意思。

  • 您应该有一个循环从数据库中读取字符串（此时不要创建图像......只需存储图像路径）并填充您的数组。然后，您可以有一个单独的循环（理想情况下，一个完全独立的例程）来构建您的 UI。您确实应该将 UI 与数据库交互隔离开来。任何由图像对象数组组成的东西都会有问题：只维护图像路径数组。

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

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

还有几个额外的问题：

  • 另一个潜在的崩溃来源是，如果您的数组中的一个图像路径没有自行解析为图像的路径。因此，imageWithContentsOfFile将返回，并且任何添加到数组的nil尝试都会崩溃。nil在尝试使用它之前，请确保您测试以查看您是否成功找到/加载了图像。

  • 你没有说图像有多大，但如果它们大于 140x140，你真的想考虑创建它们的屏幕分辨率再现。如果图像非常大，虽然它可以以 70x70 渲染它们，但您将用完整个图像的内存。同时显示这么多图像时，如果您不小心使用屏幕分辨率图像，您将很快消耗您的内存。

  • 根据您在数组中引用的图像数量，您甚至可以考虑一个模型，您可以在其中处理该UIScrollViewDelegate方法scrollViewDidScroll并仅创建UIImageView对象并在它们滚动到视图中时填充它们各自的image属性（并删除那些已滚动到视图之外的对象）。如果针对 iOS 6，您可以使用 aUICollectionView而不是手动生成的滚动视图，并且您将自动获得其中的一些功能（只要您的数组是图像路径数组而不是图像对象数组）。

如果您仍然崩溃，您应该与我们分享崩溃的详细信息（如果您不告诉我们您遇到了哪种异常/错误，我们只是在猜测）。此外，如果您还没有，请实施一个didReceiveMemoryWarning，如果没有别的，当您有内存警告时会告诉您，以便您可以识别问题并解决它。并确保在设备上测试这样的内存消耗应用程序，因为有许多与内存相关的问题不会在模拟器上表现出来，但会在设备上引起注意。

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

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

尽管如此，我还是会先专注于修复崩溃，然后再处理性能问题。

于 2013-04-08T18:25:58.463 回答