0

我的下一个代码可以正常执行，但是需要花费太多时间...有什么方法可以使用 createQueryBuilder 和更新方法来优化它吗？

这是当前功能：

 private function getCsv()
    {

        $file = 'Diva_tarif.csv';
        Message::write("Retriving $file", 3);
        $file = $this->root_data . $file;

        $serializer = new Serializer([new ObjectNormalizer()], [new CsvEncoder(';')]);
        $counter = 0;
        $batch_size = 30;
        if (($handle = fopen($file, 'r')) !== false) {
            $header = fgets($handle);
            while (($line = fgets($handle)) !== false) {

                $line = $serializer->decode($header . $line, 'csv');
                $repo = $this->em->getRepository(Article::class);
                if(isset($line['DOS'])){
                    $counter ++;

                    $arts = $repo->findBy(array('dos' => $line['DOS'], 'ref' => $line['REF']));

                    $used_metas = [
                        'TACOD',
                        'VENUN',
                        'DEV',
                        'PUB',
                        'ALZTXREMMAX',
                        'ALZTXREMMAXLALPHA',                    
                    ];

                    foreach($arts as $art){                        
                        foreach ($used_metas as $metakey ) {

                            $meta_obj = new ArticleMeta();
                            $meta_obj->setName($metakey);
                            $meta_obj->setValue($line[$metakey]);
                            $meta_obj->setArticle($art);
                            $this->em->persist($meta_obj);
                        }
                    }   
                    if (($counter % $batch_size) === 0) {                   
                        $this->em->flush();
                        $this->em->clear(); 
                    }

                    if(($counter % 500) == 0){                   
                        Message::write("$counter lines added", 4);
                    }
                }
            }
            Message::write("$counter lines added", 4);
            Message::write("Done", 3);
            $this->em->flush();
            $this->em->clear(); 
            fclose($handle);           
        }
    }

目标是获取文章并为 ArticleMeta 分配新值。任何的想法？

4

1 回答 1

1

您可以在CsvEncoder没有序列化器的情况下直接使用，您正在使用没有非规范化的解码。

我区分了两种提高性能的方法

无需更改代码

您应该将该对索引ref-dos到您的数据库中，这确实会增加查询时间。

但是执行脚本的时间会随着文件行数的增加而增加。

一个大查询

您可以解析一次文件，提取所有dosand ref，然后使用 pair 构建一个数组ref-dos。

示例： [ ABCD-1234, BCDE-2345, ... ]

之后，您可以ONE对您的数据库进行大查询，并避免一个查询/行，IMO 非常耗时。

这种方式与行数的关系较小，因为您查询数据库一次。

这样IMO是最好的

此外，您可以$used_metas在$repowhile 之外声明。

您已经在使用批处理 + 实体管理器清理，这是使用 Doctrine 插入许多实体的好方法。

如果你真的想快点，你可以使用原始 SQL 并避免学说实体水合。但我不建议你这样

于 2018-09-04T15:40:12.127 回答