Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
1 | <?php |
||
12 | class Client extends \GuzzleHttp\Client |
||
13 | { |
||
14 | /** |
||
15 | * @const string Version number |
||
16 | */ |
||
17 | const VERSION = '0.0.1'; |
||
18 | |||
19 | /** |
||
20 | * Defaults to expecting that Apache Tomcat runs on port 8080 on localhost |
||
21 | * (127.0.0.1). |
||
22 | * |
||
23 | * @var array[] |
||
24 | */ |
||
25 | protected $options = [ |
||
26 | 'scheme' => 'http', |
||
27 | 'hostname' => 'localhost', |
||
28 | 'port' => '8080', |
||
29 | ]; |
||
30 | |||
31 | /** |
||
32 | * @param $options array |
||
33 | * |
||
34 | * @return void |
||
|
|||
35 | */ |
||
36 | 2 | public function __construct($options = []) |
|
60 | |||
61 | /** |
||
62 | * Facility provided to cancel recharges. This does not work if the recharge |
||
63 | * has been processed by the MNO. |
||
64 | * |
||
65 | * @param int $reference Order Reference from SmartCall |
||
66 | * |
||
67 | * @throws Exception |
||
68 | * |
||
69 | * @return array |
||
70 | */ |
||
71 | 2 | public function cancelRecharge($reference) |
|
72 | { |
||
73 | try { |
||
74 | 2 | $response = $this->post( |
|
75 | 2 | '/SmartcallRestfulProxy/cancel_recharge_js', |
|
76 | [ |
||
77 | 'json' => [ |
||
78 | 2 | 'orderReferenceId' => $reference, |
|
79 | 2 | ], |
|
80 | ] |
||
81 | 2 | ); |
|
82 | |||
83 | return [ |
||
84 | 2 | 'status' => 'ok', |
|
85 | 2 | 'http_code' => $response->getStatusCode(), |
|
86 | 2 | 'body' => (string) $response->getBody(), |
|
87 | 2 | ]; |
|
88 | } catch (\GuzzleHttp\Exception\ServerException $e) { |
||
89 | return $this->parseError($e); |
||
90 | } |
||
91 | } |
||
92 | |||
93 | /** |
||
94 | * Transfer funds to another SmartCall Dealer Account. |
||
95 | * |
||
96 | * @param int $amount Amount in rands (ZAR) to transfer to the recipients SmartCall Dealer Account |
||
97 | * @param int $msisdn MSISDN of the account to receive the funds being transfered |
||
98 | * @param bool $sendSms true = send sms | false do not send a sms |
||
99 | * |
||
100 | * @throws Exception |
||
101 | * |
||
102 | * @return array |
||
103 | */ |
||
104 | public function fundsTransfer($amount, $msisdn, $sendSms) |
||
105 | { |
||
106 | try { |
||
107 | $response = $this->post( |
||
108 | '/SmartcallRestfulProxy/fund_transfer_js', |
||
109 | [ |
||
110 | 'json' => [ |
||
111 | 'amount' => $amount, |
||
112 | 'recipientMsisdn' => $msisdn, |
||
113 | 'sendSms' => (bool) $sendSms, |
||
114 | ], |
||
115 | ] |
||
116 | ); |
||
117 | |||
118 | return [ |
||
119 | 'status' => 'ok', |
||
120 | 'http_code' => $response->getStatusCode(), |
||
121 | 'body' => (string) $response->getBody(), |
||
122 | ]; |
||
123 | } catch (\GuzzleHttp\Exception\ServerException $e) { |
||
124 | return $this->parseError($e); |
||
125 | } |
||
126 | } |
||
127 | |||
128 | /** |
||
129 | * Fetches the Dealer Balance from SmartCall. |
||
130 | * |
||
131 | * @throws Exception |
||
132 | * |
||
133 | * @return array |
||
134 | */ |
||
135 | 2 | View Code Duplication | public function getDealerBalance() |
136 | { |
||
137 | try { |
||
138 | 2 | $response = $this->get('/SmartcallRestfulProxy/balance'); |
|
139 | |||
140 | return [ |
||
141 | 1 | 'status' => 'ok', |
|
142 | 1 | 'http_code' => $response->getStatusCode(), |
|
143 | 1 | 'body' => (string) $response->getBody(), |
|
144 | 1 | ]; |
|
145 | 1 | } catch (\GuzzleHttp\Exception\ServerException $e) { |
|
146 | 1 | return $this->parseError($e); |
|
147 | } |
||
148 | } |
||
149 | |||
150 | /** |
||
151 | * Fetches the Dealer Balance from SmartCall. |
||
152 | * |
||
153 | * @throws Exception |
||
154 | * |
||
155 | * @return array |
||
156 | */ |
||
157 | 1 | View Code Duplication | public function isDealerRegistered($msisdn) |
176 | |||
177 | /** |
||
178 | * Fetches the Product from SmartCall. |
||
179 | * |
||
180 | * @param int $productId Product Identifier |
||
181 | * |
||
182 | * @throws Exception |
||
183 | * |
||
184 | * @return array |
||
185 | */ |
||
186 | 2 | View Code Duplication | public function getProduct($productId) |
205 | |||
206 | /** |
||
207 | * Fetches the Product List from SmartCall. |
||
208 | * |
||
209 | * @throws Exception |
||
210 | * |
||
211 | * @return array |
||
212 | */ |
||
213 | 1 | View Code Duplication | public function getProducts() |
227 | |||
228 | /** |
||
229 | * Fetches the details of the last transaction processed from SmartCall. |
||
230 | * |
||
231 | * @throws Exception |
||
232 | * |
||
233 | * @return array |
||
234 | */ |
||
235 | 1 | View Code Duplication | public function getLastTransaction() |
249 | |||
250 | /** |
||
251 | * Fetches the Product List by the specified network identifier from SmartCall. |
||
252 | * |
||
253 | * @param int $networkId identifier for the network |
||
254 | * |
||
255 | * @throws Exception |
||
256 | * |
||
257 | * @return array |
||
258 | */ |
||
259 | 1 | View Code Duplication | public function getProductsByNetwork($networkId) |
278 | |||
279 | /** |
||
280 | * Purchase a voucher or do a pinless recharge on SmartCall. |
||
281 | * |
||
282 | * @param int $productId identifier for the product |
||
283 | * @param int $amount amount in rands of the product |
||
284 | * @param int $msisdn mobile number of the recipient of the product |
||
285 | * @param int $deviceId mobile number or meter number of the device being recharged |
||
286 | * @param string $clientRef Client Reference (use a UUID) |
||
287 | * @param bool $pinless true = device will be recharged via the network's IN platform | false = pinbased virtual voucher |
||
288 | * @param bool $sendSms true = SmartCall will send the voucher via SMS | false don't send the voucher via SMS |
||
289 | * |
||
290 | * @throws Exception |
||
291 | * |
||
292 | * @return array |
||
293 | */ |
||
294 | 4 | public function purchaseProduct($productId, $amount, $msisdn, $deviceId, $clientRef, $pinless, $sendSms) |
|
323 | |||
324 | /** |
||
325 | * Searches SmartCall for a specified transaction using a specified key and string to search |
||
326 | * against at SmartCall. |
||
327 | * |
||
328 | * @param string $field client_ref | msisdn | order_reference |
||
329 | * @param string $search Client Reference when client_ref or a users MSISDN when msisdn |
||
330 | * |
||
331 | * @throws Exception |
||
332 | * |
||
333 | * @return array |
||
334 | */ |
||
335 | 6 | public function searchTransaction($field, $search) |
|
364 | |||
365 | 1 | private function parseError(\GuzzleHttp\Exception\ServerException $exception) |
|
377 | } |
||
378 |
Adding a
@return
annotation to a constructor is not recommended, since a constructor does not have a meaningful return value.Please refer to the PHP core documentation on constructors.