Bobbyshaw\WatsonVisualRecognition\Message\ClassifyResponse::getImages()

Complexity

Conditions	5
Paths	2

Size

Total Lines	22
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getImages()
    {
        if (!$this->images) {

The expression $this->images of type Bobbyshaw\WatsonVisualRecognition\Image[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $images = [];
            foreach ($this->data->images as $image) {
                $classifiers = [];
                if (isset($image->scores)) {
                    foreach ($image->scores as $classifier) {
                        $classifiers[] = new Classifier(
                            $classifier->classifier_id,
                            $classifier->name,
                            $classifier->score
                        );
                    }
                }
                $images[] = new Image($image->image, $classifiers);
            }
            $this->images = $images;
        }

        return $this->images;
    }
}