код занимает больше времени, чем ожидалось

Question

Мой код занимает больше времени, чем ожидалось.Как я могу сократить время загрузки?Раньше это не занимало столько времени.Я не изменил код, и все же он замедлился.Мой код выглядит следующим образом:

for (i =1 ; i< [productList count]; i++) {
    UIImage *image;
    products *productItem = [productList objectAtIndex:i-1];
    if(![productItem.productItemPhoto isEqualToString:@""]){
        NSString *productItemPhoto = productItem.productItemPhoto;          
        NSData* data = [NSData dataWithContentsOfFile:productItemPhoto];
        image = [[UIImage alloc] initWithData:data];
    }
    else{
        if(numberOfProductsPerRow == 1)
            image = [UIImage imageNamed:@"no-image-2.png"];
        else
            image = [UIImage imageNamed:@"no-image-1.png"];
    }
    UIImageView *bg1;
    if(numberOfProductsPerRow == 1)
        bg1 = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"image-box-s7.png"]];
    else            
        bg1 = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"image-box-s4.png"]];
        bg1.frame = CGRectMake(x, y,width, height);
        [productScrollView addSubview:bg1];

        UIButton *pro1 = [[UIButton alloc] initWithFrame:CGRectMake(x+spacingX, y+spacingY-15, btnWidth, btnHeight)];

    [pro1 setImage:image forState:UIControlStateNormal];

        [pro1 setTag:i];
        [pro1 addTarget:self action:@selector(selectProduct:) forControlEvents:UIControlEventTouchUpInside];
        [productScrollView addSubview:pro1 ];

        UILabel *lblProductModel = [[UILabel alloc] initWithFrame:CGRectMake(0, height - 40, width, 30)];
        lblProductModel.backgroundColor = [UIColor clearColor];
        lblProductModel.textAlignment = UITextAlignmentCenter;
        lblProductModel.textColor = [UIColor colorWithRed:1.0 green:0.8 blue:0.0 alpha:1];
        NSString *price;

        if([userSettings.priceToShow isEqualToString:@"WholesalePrice"])
            price = [NSString stringWithFormat:@"%@%.0f",productItem.productCurrencySymbol, productItem.productWholesalePrice];
        else if([userSettings.priceToShow isEqualToString:@"RetailsalePrice"])
            price = [NSString stringWithFormat:@"%@%.0f",productItem.productCurrencySymbol, productItem.productRetailSalesValue];
        else
            price = @"";
        lblProductModel.tag = [productList count] + i;
        lblProductModel.text = [NSString stringWithFormat:@"%@ %@", productItem.productModelCode,  price];
        [bg1 addSubview:lblProductModel];
        x = x + width + 10;
        if(i%numberOfProductsPerRow == 0){
            x = 20;
            y=y+height+10;
        }
        [pro1 release];
        [image release];
        [bg1 release];
        [lblProductModel release];

}
if((i-1)%numberOfProductsPerRow!=0)
    scrollViewParent.contentSize = CGSizeMake(0, y+height+spacingY);
else    
    scrollViewParent.contentSize = CGSizeMake(0, y + spacingY);
productScrollView.contentSize = scrollViewParent.contentSize;
[scrollViewParent addSubview:productScrollView];
}

В productList поступает около 380 записейЯ не думаю, что это займет столько времени.

Answer 1

Учитывая количество фьюзинга с UI* API в этом коде, этот код должен выполняться в главном потоке. Вы загружаете тонну изображений в основной поток, и, сравнительно говоря, загрузка изображений является очень медленной и дорогой операцией.

Во-первых, вам действительно нужно загрузить все 380 изображений при запуске? Что происходит, когда это увеличивается до 500 или 1500 или 15000? Держу пари, что вашему приложению не хватает памяти где-то между 380 и 15000 ...

Ваш код должен загружать только те изображения, которые ему нужны, а затем делать это вне этого цикла. Все, что выполняется в главном потоке, будет блокировать взаимодействие с пользователем до тех пор, пока оно выполняется; идеальным является поддержание этих периодов времени как можно короче (или, в идеале, вовсе не перемещая вычисления в фоновую очередь / поток).