Question

Here’s my code:

$quizId = '';
foreach ($module['QuizListing'] as $quizListing)    {
    if ($quizListing['id']) {
        $quizId = $quizListing['id'];
        break;
    }
}

Is there a better way of doing this?

Answer 1

What you’re doing is reasonable assuming that:

• multiple quiz listings appear; and
• not all of them have an ID.

I assume from your question that one of both of these is not true. If you want the first quiz listing then do this:

$listing = reset($module['quizListing']);
$quizId = $listing['id'];

The reset() function returns the first element in the array (or false if there isn’t one).

This assumes every quiz listing has an ID. If that’s not the case then you can’t get much better than what you’re doing.