NeedleProject\LaravelRabbitMq\Processor\AbstractMessageProcessor

Complexity

Total Complexity

9

Size/Duplication

Total Lines	99
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

76.92%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	consume()	0	29	4
A	ack()	0	18	2
A	nack()	0	10	2
A	getProcessedMessages()	0	4	1
	processMessage()	0	1	?

<?php
namespace NeedleProject\LaravelRabbitMq\Processor;

use PhpAmqpLib\Message\AMQPMessage;
use Psr\Log\LoggerAwareInterface;
use Psr\Log\LoggerAwareTrait;

/**
 * Class AbstractMessageProcessor
 *
 * @package NeedleProject\LaravelRabbitMq\Processor
 * @author  Adrian tilita <adrian@tilita.ro>
 */
abstract class AbstractMessageProcessor implements MessageProcessorInterface, LoggerAwareInterface
{
    use LoggerAwareTrait;

    /**
     * @const string Key used on message to identify if we ack/nack via the child
     */
    const HANDLED_PROPERTY = 'handled_property';

    /**
     * @var int
     */
    private $messageCount = 0;

    /**
     * {@inheritdoc}
     * @param AMQPMessage $message
     */
    public function consume(AMQPMessage $message)
    {
        $this->messageCount++;
        try {
            $response = $this->processMessage($message);
            // Already ack/nack from inside the processor using
            // the protected methods ::ack / ::nack
            if (property_exists($message, self::HANDLED_PROPERTY)) {
                $this->logger->debug("Already handled!");
                return;
            }
            if ($response === true) {
                $this->ack($message);
            } else {
                $this->nack($message);
            }
        } catch (\Throwable $e) {
            $this->logger->error(
                sprintf(
                    "Could not process message, got %s from %s in %d for message: %s",
                    get_class($e) . '-' . $e->getMessage(),
                    (string)$e->getFile(),
                    (int)$e->getLine(),
                    (string)$message->getBody()
                )
            );
            $this->nack($message);
        }
    }

    /**
     * @param AMQPMessage $message
     */
    protected function ack(AMQPMessage $message)
    {
        try {
            $this->logger->debug(sprintf("Processed with success message %s", $message->getBody()));
            $message->delivery_info['channel']->basic_ack($message->delivery_info['delivery_tag']);
            $message->{self::HANDLED_PROPERTY} = true;
        } catch (\Throwable $e) {
            $this->logger->error(
                sprintf(
                    "Could not process message, got %s from %s in %d for message: %s",
                    get_class($e) . '-' . $e->getMessage(),
                    (string)$e->getFile(),
                    (int)$e->getLine(),
                    (string)$message->getBody()
                )
            );
        }
    }

    /**
     * @param AMQPMessage $message
     * @param bool $redeliver
     */
    protected function nack(AMQPMessage $message, bool $redeliver = true)
    {
        try {
            $this->logger->debug(sprintf("Did not processed with success message %s", $message->getBody()));
            $message->delivery_info['channel']->basic_nack($message->delivery_info['delivery_tag'], false, $redeliver);
            $message->{self::HANDLED_PROPERTY} = true;
        } catch (\Throwable $e) {
            $this->logger->debug(sprintf("Did not processed with success message %s", $message->getBody()));
        }
    }

    /**
     * @return int
     */
    public function getProcessedMessages(): int
    {
        return $this->messageCount;
    }

    /**
     * @param AMQPMessage $message
     * @return bool
     */
    abstract public function processMessage(AMQPMessage $message): bool;

function processMessage() does not seem to conform to the naming convention (^(?:is|has|should|may|supports)).

}