Eccube\Service\PurchaseFlow\ValidatableItemHolderProcessor

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	0	10	2
	validate()	0	1	?
A	handle()	0	3	1

<?php

namespace Eccube\Service\PurchaseFlow;

use Eccube\Entity\ItemHolderInterface;

abstract class ValidatableItemHolderProcessor implements ItemHolderProcessor

Abstract class name is not prefixed with "Abstract"

Missing class doc comment

{
    /**
     * @param ItemHolderInterface $itemHolder
     * @param PurchaseContext     $context
     *
     * @return ProcessResult
     */
    final public function process(ItemHolderInterface $itemHolder, PurchaseContext $context)
    {
        try {
            $this->validate($itemHolder, $context);

            return ProcessResult::success();
        } catch (ItemValidateException $e) {
            return ProcessResult::error($e->getMessage());
        }
    }

    abstract protected function validate(ItemHolderInterface $itemHolder, PurchaseContext $context);

    protected function handle(ItemHolderInterface $itemHolder)

The parameter $itemHolder is not used and could be removed.

    {
    }
}