Erdemkeren\JetSms\ShortMessage

Complexity

Total Complexity

9

Size/Duplication

Total Lines	79
Duplicated Lines	17.72 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	body()	0	4	1
A	hasManyReceivers()	0	4	1
A	receivers()	0	4	1
A	receiversString()	0	4	1
A	toArray()	0	7	1
A	toSingleMessageXml()	7	7	1
A	toMultipleMessagesXml()	7	7	1

<?php

namespace Erdemkeren\JetSms;

/**
 * Class ShortMessage.
 */
class ShortMessage
{
    /**
     * The short message body.
     *
     * @var string
     */
    protected $body;

    /**
     * The receivers.
     *
     * @var array
     */
    protected $receivers;

    /**
     * ShortMessage constructor.
     *
     * @param string       $body
     * @param string|array $receivers
     */
    public function __construct($receivers, $body)
    {
        $this->body = $body;
        $this->receivers = is_array($receivers)
            ? array_map('trim', $receivers)
            : [trim($receivers)];
    }

    public function body()
    {
        return $this->body;
    }

    public function hasManyReceivers()
    {
        return count($this->receivers()) > 1;
    }

    public function receivers()
    {
        return $this->receivers;
    }

    public function receiversString($glue = null)
    {
        return implode($glue, $this->receivers);
    }

    public function toArray()
    {
        return array_filter([
            'Msisdns'        => $this->receiversString('|'),
            'Messages'       => $this->body(),
        ]);
    }

    public function toSingleMessageXml()

This method seems to be duplicated in your project.

    {
        $text = str_replace("'", "'", htmlentities($this->body()));
        $gsmNo = $this->receiversString(',');

        return "<text>{$text}</text><message><gsmnos>{$gsmNo}</gsmnos></message>";
    }

    /**
     * Get the xml representation of the short message.
     *
     * @return string
     */
    public function toMultipleMessagesXml()

This method seems to be duplicated in your project.

    {
        $text = str_replace("'", "'", htmlentities($this->body()));
        $gsmNo = $this->receiversString(',');

        return "<message><gsmno>{$gsmNo}</gsmno><text>{$text}</text></message>";
    }
}