WeDevBr\Bankly\Validators\BillPaymentValidator

Complexity

Total Complexity

16

Size/Duplication

Total Lines	82
Duplicated Lines	25.61 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	validate()	0	7	1
A	validateBankBranch()	7	7	4
A	validateBankAccount()	7	7	4
A	validateAmount()	0	7	3
A	validateId()	7	7	3

<?php

namespace WeDevBr\Bankly\Validators;

use WeDevBr\Bankly\BillPayment;

/**
 * BillPaymentValidator class
 *
 * PHP version 7.3|7.4|8.0
 *
 * @author    WeDev Brasil Team <contato@wedev.software>
 * @author    Rafael Teixeira <rafaeldemeirateixeira@gmail.com>
 * @copyright 2020 We Dev Tecnologia Ltda
 * @link      https://github.com/wedevBr/bankly-laravel/
 */
class BillPaymentValidator
{
    private $billPayment;

    /**
     * BillPaymentValidator constructor.
     * @param BillPayment $billPayment
     */
    public function __construct(BillPayment $billPayment)
    {
        $this->billPayment = $billPayment;
    }

    /**
     * This validate the bank account
     *
     * @return void
     */
    public function validate(): void
    {
        $this->validateBankBranch();
        $this->validateBankAccount();
        $this->validateAmount();
        $this->validateId();
    }

    /**
     * This validate a bank branch
     *
     * @return void
     * @throws InvalidArgumentException
     */
    private function validateBankBranch()

This method seems to be duplicated in your project.

    {
        $bankBranch = $this->billPayment->bankBranch;
        if (is_null($bankBranch) || !is_string($bankBranch) || !is_numeric($bankBranch)) {
            throw new \InvalidArgumentException('branch should be a numeric string');
        }
    }

    /**
     * This validate a bank account
     *
     * @return void
     * @throws InvalidArgumentException
     */
    private function validateBankAccount()

This method seems to be duplicated in your project.

    {
        $bankAccount = $this->billPayment->bankAccount;
        if (is_null($bankAccount) || !is_string($bankAccount)  || !is_numeric($bankAccount)) {
            throw new \InvalidArgumentException('account should be a numeric string');
        }
    }

    /**
     * This validates a amount
     *
     * @return void
     * @throws InvalidArgumentException
     */
    private function validateAmount()
    {
        $amount = $this->billPayment->amount;
        if (is_null($amount) || !is_numeric($amount)) {
            throw new \InvalidArgumentException('amount should be a numeric');
        }
    }

    /**
     * This validates a given ID
     *
     * @return void
     * @throws InvalidArgumentException
     */
    private function validateId()

This method seems to be duplicated in your project.

    {
        $id = $this->billPayment->id;
        if (is_null($id) || !is_string($id)) {
            throw new \InvalidArgumentException('ID should be a string');
        }
    }
}