1

我有工作代码，并希望从中获得更好的性能，因为如果传递更多数据，它会得到很好的加载。但我不知道我还能做些什么。

输入：带有 logUnits 的数组，每个都包含一个日志事件视图，例如“2013-01-16 15:13:00 Hello”和时间数组 - 此数组仅包含我们有日志事件的时间 (2013-01-02,2013 -01-09 等）

输出：视图数组，其中每个视图包含同一天的日志单元。

我还可能需要按事件类型过滤日志单元。

这是我如何做到的。

 NSMutableArray *views = [[NSMutableArray alloc] initWithCapacity:times.count];
    @autoreleasepool {

        for(int x = 0; x != times.count; x++)
        {
            UIView *foo = [[UIView alloc] initWithFrame:CGRectMake(0, 0, 320, 400)];
            foo.backgroundColor = [UIColor clearColor];
            int f = 0;
            int h = 0;

            for(int i = 0; i != objectArray.count; i++)
            {
                @autoreleasepool
                {

                    LogUnit *unit = [objectArray objectAtIndex:i];
                    if([[times objectAtIndex:x] isEqual:[unit realTime]]) 
                    {

                        if(![[foo subviews] containsObject:unit.view]) // Look if unit was already added
                        {
                            NSString *number = [NSString stringWithFormat:@"SortLog%i ",[[NSUserDefaults standardUserDefaults] stringForKey:@"ObjectNumber"].intValue];
                            NSString *comp = [[NSUserDefaults standardUserDefaults] stringForKey:number];

                            if([[unit status] isEqualToString:comp] || [comp isEqualToString:NULL] || comp == NULL)
                            {
                                if([unit getEvent] != NULL)
                                {
                                    unit.view.frame = CGRectMake(unit.view.frame.origin.x, f * unit.view.frame.size.height + 30, unit.view.frame.size.width, unit.view.frame.size.height);
                                    [foo addSubview:unit.view];
                                    f++;
                                    h = unit.view.frame.size.height * f;
                                }
                            }
                        }
                    }
                }
            }

            foo.frame = CGRectMake(0,0, 320, h + 30 );
            h = 0;
            [views addObject:foo];
        }

我猜这最糟糕的事情是循环中的循环，对于每个外部循环，我必须重新运行所有 logUnit。但不知道我还能如何做到这一点。也许有一种方法可以称为“选择时间等于[单位实时]的单位”有什么想法吗？

4

1 回答 1

0

这里有一些危险信号。最容易实现的结果是您正在扫描 foo 的所有子视图以获取 unit.view，这是一个 O(n) 操作，这在最里面的循环中确实很痛苦。

接下来，如果您按时间对两个数组进行排序，您实际上不必执行双循环来查找大小相同的事件。看看它是如何在归并排序中完成的。

如果这还不够好，请考虑在侧线程上进行所有比较工作，并且仅将添加子视图和视图创建过程分派回主线程。它至少会让它看起来快很多。

于 2013-01-18T08:46:36.503 回答