Shippinno\YahooShoppingJp\Request\UpdateItemStockInfoRequest

Complexity

Total Complexity

19

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	setSellerId()	0	14	3
B	setItemCode()	0	20	5
A	setQuantity()	0	14	4
A	setAllowOverdraft()	0	10	2
A	getParams()	0	6	1
A	validateRequest()	0	14	4

<?php

namespace Shippinno\YahooShoppingJp\Request;

use LogicException;
use Shippinno\YahooShoppingJp\Exception\InvalidRequestException;

class UpdateItemStockInfoRequest extends AbstractRequest

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: api, response, validateParams

{
    private $params = [];

    /**
     * @param string $sellerId
     * @return self
     * @throws InvalidRequestException
     */
    public function setSellerId(string $sellerId): self
    {
        if (isset($this->params['seller_id'])) {
            throw new LogicException('seller_id is already set.');
        }

        if (! preg_match('/^[a-z0-9\-]{3,20}$/', $sellerId)) {
            throw new InvalidArgumentException;
        }

        $this->params['seller_id'] = $sellerId;

        return $this;
    }

    /**
     * @param string $itemCode
     * @param string $subCode
     * @return self
     * @throws InvalidRequestException
     */
    public function setItemCode(string $itemCode, string $subCode = ''): self
    {
        if (isset($this->params['item_code'])) {
            throw new LogicException('item_code is already set.');
        }

        if (! preg_match('/^[a-zA-Z0-9\-]{1,99}$/', $itemCode)) {
            throw new InvalidRequestException('item_code error.');
        }
        $this->params['item_code'] = $itemCode;

        if (strlen($subCode)) {
            if (! preg_match('/^[a-zA-Z0-9\-]{1,99}$/', $subCode)) {
                throw new InvalidRequestException('sub_code error.');
            }
            $this->params['item_code'] = $this->params['item_code'].':'.$subCode;
        }

        return $this;
    }

    /**
     * @param int|string $quantity
     * @return self
     * @throws InvalidRequestException
     */
    public function setQuantity($quantity): self
    {
        if (isset($this->params['quantity'])) {
            throw new LogicException('quantity is already set.');
        }

        if (! preg_match('/^((\+|-)?[0-9]{1,9}|INI)$/', $quantity)) {
            throw new InvalidRequestException('Only number or INI can be set.');
        }

        $this->params['quantity'] = is_int($quantity) ? strval($quantity) : $quantity;

        return $this;
    }

    /**
     * @param bool $allowOverdraft
     * @return self
     */
    public function setAllowOverdraft(bool $allowOverdraft): self
    {
        if (isset($this->params['allow_overdraft'])) {
            throw new LogicException('allow_overdraft is already set.');
        }

        $this->params['allow_overdraft'] = (int) $allowOverdraft;

        return $this;
    }

    /**
     * @return array
     */
    public function getParams(): array
    {
        $this->validateRequest();

        return $this->params;
    }

    /**
     * @throws InvalidRequestException
     */
    private function validateRequest(): void
    {
        if (! isset($this->params['seller_id'])) {
            throw new InvalidRequestException('seller_id is not set.');
        }

        if (! isset($this->params['item_code'])) {
            throw new InvalidRequestException('item_code is not set.');
        }

        if (! isset($this->params['quantity'])) {
            throw new InvalidRequestException('quantity is not set.');
        }
    }
}