Refactor DcpProducer::getNextItem to use helper functions

The function getNextItem is pretty long with multiple levels of indentations and loops and the scopes don't fit on a single screen. It could easily be split into calling multiple helper functions to make the code become more readable.