Fill one array using two loop

Question

I have an object where there are all my articles.

I'm currently looping my object to fill an array where I create an associative table for each article.

In my object I also have a Categories object and I would like to add the label of each category at the end of each associative array previously completed, but I don't know how to do that.. In the Categories object there may be multiple labels.

My code :

$articles = $this->entityManager->getRepository('SGBundle:Article')->findBy([], ['id'=>'desc']);

$arrayCollection = [];

foreach($articles as $article) {

    $arrayCollection[] = [
        'id' => $article->getId(),
        'date_publication' => $article->getDatePublication(),
        ...
    ];

    foreach($article->getCategories() as $categorie) {
        $arrayCollection[] = ['categorie' => $categorie->getLibelle()];
    }
}

On my screenshot, for each article there is an array with 36 values and an array with 1 value and I would like this table to be in the table where there are 36 values. It's possible ?

u_mulder · Accepted Answer · 2018-01-09 20:34:15Z

1

First gather categories, then add'em to article item:

foreach($articles as $article) {
    $categories = [];
    foreach($article->getCategories() as $categorie) {
        $categories[] = $categorie->getLibelle();
    }

    $arrayCollection[] = [
        'id' => $article->getId(),
        'date_publication' => $article->getDatePublication(),
        ...
        // 
        'categorie' => $categories,
    ]; 
}

answered Jan 9, 2018 at 20:34

u_mulder

54.8k5 gold badges54 silver badges72 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Scolopendre · Accepted Answer · 2018-01-10 12:06:09Z

0

If the article.categories field is marked as lazy, then it won't be hydrated by default and the $article->getCategories() will perform a new query on each loop round.

Instead of a simple findBy, you might want a custom DQL query in this case to optimize this and get the exact array you want in one single request.

Also note that your current query is fetching all articles of your database. While this is probably your purpose, keep in mind that this could get pretty heavy with the data growing. In most cases, this kind of query should be paginated.

answered Jan 10, 2018 at 12:06

Scolopendre

2121 silver badge4 bronze badges

Collectives™ on Stack Overflow

Fill one array using two loop

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related