Toplan\PhpSms\LuosimaoAgent

Complexity

Total Complexity

8

Size/Duplication

Total Lines	47
Duplicated Lines	23.4 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setResult()	11	11	2
B	sendContentSms()	0	17	5
A	sendVoiceCode()	0	11	1

<?php

namespace Toplan\PhpSms;

/**
 * Class LuosimaoAgent
 *
 * @property string $apikey
 * @property string $voiceApikey
 */
class LuosimaoAgent extends Agent implements ContentSms, VoiceCode
{
    protected static $smsUrl = 'http://sms-api.luosimao.com/v1/send.json';
    protected static $voiceCodeUrl = 'http://voice-api.luosimao.com/v1/verify.json';

    public function sendContentSms($to, $content)
    {
        // 签名必须在最后面
        if ($content && preg_match('/(【[\\s\\S]*】)/', $content, $matches)) {
            if (isset($matches[0]) && strlen($matches[0])) {
                $content = str_replace($matches[0], '', $content) . $matches[0];
            }
        }
        $result = $this->curlPost(self::$smsUrl, [
            'mobile'  => $to,
            'message' => $content,
        ], [
            CURLOPT_HTTPAUTH    => CURLAUTH_BASIC,
            CURLOPT_USERPWD     => "api:key-{$this->apikey}",
        ]);
        $this->setResult($result);
    }

    public function sendVoiceCode($to, $code)
    {
        $result = $this->curlPost(self::$voiceCodeUrl, [
            'mobile' => $to,
            'code'   => $code,
        ], [
            CURLOPT_HTTPAUTH    => CURLAUTH_BASIC,
            CURLOPT_USERPWD     => "api:key-{$this->voiceApikey}",
        ]);
        $this->setResult($result);
    }

    protected function setResult($result)

This method seems to be duplicated in your project.

    {
        if ($result['request']) {
            $this->result(Agent::INFO, $result['response']);
            $result = json_decode($result['response'], true);
            $this->result(Agent::SUCCESS, $result['error'] === 0);
            $this->result(Agent::CODE, $result['error']);
        } else {
            $this->result(Agent::INFO, 'request failed');
        }
    }
}