Parse all items with PHP Simple HTML DOM Parser

Question

I'm trying to parse DOM with Simple HTML DOM Parser in PHP. Parsed content are movies, so I want to get every genre there is, but when i run my code I only get the last genre, not all. My code looks like this:

if ($obj) {
    foreach($obj as $key => $data) {
        $item['url'] = 'http://geo.saitebi.ge/movie/' . $page;
        $item['poster'] = 'http://geo.saitebi.ge/web/ka/img/movies/' . $page . '/240x340/' . $page . '.jpg';
        $item['geotitle'] = $data->find('div.movie-item-title', 0)->plaintext;
        $item['englishtitle'] = $data->find('div.movie-item-title-en', 0)->plaintext;
        $item['year'] = $data->find('div.movie-item-year', 0)->plaintext;
        foreach($data->find('a.movie-genre-item') as $genre) {
            $item['genres'] = $genre->plaintext . ', ';
        }
        $item['description'] = $data->find('div.movie-desctiption-more', 0)->plaintext;
        $item['imdb_rating'] = $data->find('a.imdb_vote', 0)->plaintext;
        $item['imdb_id'] = trim(substr($data->find('a.imdb_vote',0)->href, strrpos($data->find('a.imdb_vote',0)->href, '/') + 1));
    }
}

As you see I'm getting content as array. Then inside it I run another foreach loop to get all genre items but it only gets last genre item. What is wrong in my code?

Nigel Ren · Accepted Answer · 2018-05-26 08:31:26Z

1

You are just overwriting the last set of data each time. You need to set it to blank and then append it using .= each time, like...

$item['genres'] = '';
foreach($data->find('a.movie-genre-item') as $genre) {
   $item['genres'] .= $genre->plaintext . ', ';
}

answered May 26, 2018 at 8:31

Nigel Ren

57.3k11 gold badges49 silver badges59 bronze badges

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

Comments

Joel Garcia Nuño · Accepted Answer · 2018-05-26 08:29:51Z

0

The code are saving the same '$genre->plaintext' in $item with key 'genres' so the same key replace the values every loop.

 foreach($data->find('a.movie-genre-item') as $genre) {
            $item['genres'] = $genre->plaintext . ', ';
 }

May $item['genres] can be an associative array.. i mean:

foreach($data->find('a.movie-genre-item') as $genre) {
             if ( !isset($item[$genre]) ) {
                      $item[$genre] = array();
              }
          array_push($item[$genre],$genre->plaintext);
     }

answered May 26, 2018 at 8:29

Joel Garcia Nuño

1,04111 silver badges20 bronze badges

1 Comment

DaftPlug Over a year ago

No need to do this. I was just overwriting the last set of data each time, so I needed to set it to blank and then append it using .= each time. Above answer solved the problem :) Thank you!

pspatel · Accepted Answer · 2018-05-26 08:34:55Z

0

Here is genre code which you have to update

    // Here is you genre code 
    $movie_genre='';
    foreach($data->find('a.movie-genre-item') as $genre) {
        $movie_genre .= $genre->plaintext . ',';
    }

    // Here you can use rtrim for removing last comma from genre
    $item['genres'] = rtrim($movie_genre,',');

answered May 26, 2018 at 8:34

pspatel

5282 gold badges7 silver badges19 bronze badges

1 Comment

DaftPlug Over a year ago

Thank you a lot for providing fix for removing last comma from genre. Thank you!

Collectives™ on Stack Overflow

Parse all items with PHP Simple HTML DOM Parser

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related