BitWasp\Bitcoin\Serializer\Transaction\TransactionInputSerializer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	12

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	serialize()	0	8	1
A	fromParser()	0	8	1
A	parse()	0	4	1

<?php

declare(strict_types=1);

namespace BitWasp\Bitcoin\Serializer\Transaction;

use BitWasp\Bitcoin\Script\Opcodes;
use BitWasp\Bitcoin\Script\Script;
use BitWasp\Bitcoin\Serializer\Types;
use BitWasp\Bitcoin\Transaction\TransactionInput;
use BitWasp\Bitcoin\Transaction\TransactionInputInterface;
use BitWasp\Buffertools\Buffer;
use BitWasp\Buffertools\BufferInterface;
use BitWasp\Buffertools\Parser;

class TransactionInputSerializer
{
    /**
     * @var OutPointSerializer
     */
    private $outpointSerializer;

    /**
     * @var \BitWasp\Buffertools\Types\VarString
     */
    private $varstring;

    /**
     * @var \BitWasp\Buffertools\Types\Uint32
     */
    private $uint32le;

    /**
     * @var Opcodes
     */
    private $opcodes;

    /**
     * TransactionInputSerializer constructor.
     * @param OutPointSerializerInterface $outPointSerializer
     * @param Opcodes|null $opcodes
     */
    public function __construct(OutPointSerializerInterface $outPointSerializer, Opcodes $opcodes = null)
    {
        $this->outpointSerializer = $outPointSerializer;

$outPointSerializer is of type object<BitWasp\Bitcoin\Serializer\Transaction\OutPointSerializerInterface>, but the property $outpointSerializer was declared to be of type object<BitWasp\Bitcoin\Serializer\Transaction\OutPointSerializer>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->varstring = Types::varstring();
        $this->uint32le = Types::uint32le();
        $this->opcodes = $opcodes ?: new Opcodes();
    }

    /**
     * @param TransactionInputInterface $input
     * @return BufferInterface
     */
    public function serialize(TransactionInputInterface $input): BufferInterface
    {
        return new Buffer(
            $this->outpointSerializer->serialize($input->getOutPoint())->getBinary() .
            $this->varstring->write($input->getScript()->getBuffer()) .
            $this->uint32le->write($input->getSequence())
        );
    }

    /**
     * @param Parser $parser
     * @return TransactionInputInterface
     * @throws \BitWasp\Buffertools\Exceptions\ParserOutOfRange
     * @throws \Exception
     */
    public function fromParser(Parser $parser): TransactionInputInterface
    {
        return new TransactionInput(
            $this->outpointSerializer->fromParser($parser),
            new Script($this->varstring->read($parser), $this->opcodes),
            (int) $this->uint32le->read($parser)
        );
    }

    /**
     * @param BufferInterface $string
     * @return TransactionInput
     * @throws \BitWasp\Buffertools\Exceptions\ParserOutOfRange
     * @throws \Exception
     */
    public function parse(BufferInterface $string): TransactionInput
    {
        return $this->fromParser(new Parser($string));
    }
}