Total Complexity | 47 |
Total Lines | 292 |
Duplicated Lines | 0 % |
Changes | 5 | ||
Bugs | 0 | Features | 0 |
Complex classes like Client often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Client, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
14 | class Client |
||
15 | { |
||
16 | use WritesHttpLogs; |
||
17 | |||
18 | const PROVIDER_BANGLALINK = Providers\Banglalink::class; |
||
19 | const PROVIDER_BD_WEB_HOST_24 = Providers\BdWebHost24::class; |
||
20 | const PROVIDER_BOOM_CAST = Providers\BoomCast::class; |
||
21 | const PROVIDER_ELITBUZZ = Providers\Elitbuzz::class; |
||
22 | const PROVIDER_GRAMEENPHONE = Providers\Grameenphone::class; |
||
23 | const PROVIDER_NOVOCOM = Providers\Novocom::class; |
||
24 | const PROVIDER_PAYSTATION = Providers\Paystation::class; |
||
25 | const PROVIDER_ROBI = Providers\Robi::class; |
||
26 | const PROVIDER_SSL = Providers\Ssl::class; |
||
27 | const PROVIDER_SSL_PLUS = Providers\SslPlus::class; |
||
28 | const PROVIDER_VALUE_FIRST = Providers\ValueFirst::class; |
||
29 | |||
30 | private $provider; |
||
31 | |||
32 | public function __construct(ProviderInterface $provider) |
||
33 | { |
||
34 | $this->provider = $provider; |
||
35 | $this->enableLogging = Config::get('sms-service-with-bd-providers::config.enable_api_call_logging', false); |
||
36 | } |
||
37 | |||
38 | /** |
||
39 | * Return a SMS provider according to the given provider name. |
||
40 | * |
||
41 | * @param string $providerName |
||
42 | * @param array $config |
||
43 | * @param string $url |
||
44 | * |
||
45 | * @return ProviderInterface |
||
46 | */ |
||
47 | public static function getProvider($providerName = self::PROVIDER_SSL, array $config = [], $url = null) |
||
48 | { |
||
49 | switch ($providerName) { |
||
50 | case self::PROVIDER_BANGLALINK: |
||
51 | case self::PROVIDER_BD_WEB_HOST_24: |
||
52 | case self::PROVIDER_BOOM_CAST: |
||
53 | case self::PROVIDER_ELITBUZZ: |
||
54 | case self::PROVIDER_GRAMEENPHONE: |
||
55 | case self::PROVIDER_NOVOCOM: |
||
56 | case self::PROVIDER_PAYSTATION: |
||
57 | case self::PROVIDER_ROBI: |
||
58 | case self::PROVIDER_SSL: |
||
59 | case self::PROVIDER_SSL_PLUS: |
||
60 | case self::PROVIDER_VALUE_FIRST: |
||
61 | return new $providerName($config, $url); |
||
62 | |||
63 | default: |
||
64 | throw new Exception('Invalid SMS provider name is given.'); |
||
65 | } |
||
66 | } |
||
67 | |||
68 | public function send($recipients, $message, array $params = []) |
||
108 | } |
||
109 | |||
110 | public function sendWithFallback($recipients, $message, array $params = []) |
||
111 | { |
||
112 | $log = ['sent' => [], 'failed' => []]; |
||
113 | is_array($recipients) || $recipients = [$recipients]; |
||
114 | |||
115 | foreach ($recipients as $recipient) { |
||
116 | $options = ['url' => $this->provider->getUrl()]; |
||
117 | |||
118 | try { |
||
119 | if (!$data = $this->provider->mapParams($recipient, $message, $params)) { |
||
120 | throw new Exception(json_encode('Failed to map the params.'), 422); |
||
121 | } |
||
122 | |||
123 | $data = array_merge($this->provider->getConfig(), $data); |
||
124 | $validator = Validator::make($data, $this->provider->getValidationRules()); |
||
125 | |||
126 | if ($validator->fails()) { |
||
127 | throw new Exception(json_encode($validator->messages()->all()), 422); |
||
128 | } |
||
129 | |||
130 | $options += $this->prepareCurlOptions($data); |
||
131 | |||
132 | try { |
||
133 | $response = $this->executeWithCurl($options); |
||
134 | } catch (Exception $e) { |
||
135 | $log['failed'][$recipient] = (new Response(false, $e->getMessage()))->toArray(); |
||
136 | $response = ''; |
||
137 | } |
||
138 | |||
139 | $response = $this->provider->parseResponse($response); |
||
140 | |||
141 | if (!$response->getStatus()) { |
||
142 | $this->log('POST', $options['url'], $options, new GuzzleResponse(500, [], $response->getResponseString())); |
||
143 | |||
144 | //Resend sms |
||
145 | Log::info('SMS sending failed response!'); |
||
146 | |||
147 | try { |
||
148 | $response = $this->provider->parseResponse($this->executeWithCurl($options)); |
||
149 | Log::info('Second try of sending SMS', $response); |
||
150 | |||
151 | if (!$response->getStatus()) { |
||
152 | throw new Exception($response->getResponseString(), 500); |
||
153 | } |
||
154 | } catch (Exception $e) { |
||
155 | Log::error('Curl error response: '.$e->getMessage()); |
||
156 | |||
157 | throw $e; |
||
158 | } |
||
159 | } |
||
160 | |||
161 | $log['sent'][$recipient] = $response->toArray(); |
||
162 | |||
163 | $this->log('POST', $options['url'], $options, new GuzzleResponse(200, [], $response->getResponseString())); |
||
164 | } catch (Exception $e) { |
||
165 | $errorCode = $e->getCode() >= 100 ? $e->getCode() : 500; |
||
166 | $errorMessage = 422 != $errorCode ? $e->getMessage() : json_decode($e->getMessage(), true); |
||
167 | $log['failed'][$recipient] = (new Response(false, $errorMessage))->toArray(); |
||
168 | |||
169 | $this->log('POST', $options['url'], $options, new GuzzleResponse($errorCode, [], $errorMessage)); |
||
170 | } |
||
171 | } |
||
172 | |||
173 | return $this->getSummaryWithLogs($log); |
||
174 | } |
||
175 | |||
176 | /** |
||
177 | * Prepare the curl options array according to the given data. |
||
178 | * |
||
179 | * @param array $data |
||
180 | * |
||
181 | * @return array |
||
182 | */ |
||
183 | private function prepareCurlOptions(array $data) |
||
238 | } |
||
239 | |||
240 | /** |
||
241 | * Execute the Curl request according to the given curl options. |
||
242 | * |
||
243 | * @param array $options |
||
244 | * |
||
245 | * @return string |
||
246 | */ |
||
247 | private function executeWithCurl(array $options) |
||
292 | } |
||
293 | |||
294 | private function getSummaryWithLogs(array $log) |
||
309 |