Complex classes like BaseInfo 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. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
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 BaseInfo, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
28 | class BaseInfo extends \Eccube\Entity\AbstractEntity |
||
29 | { |
||
30 | /** |
||
31 | * @var int |
||
32 | * |
||
33 | * @ORM\Column(name="id", type="integer", options={"unsigned":true}) |
||
34 | * @ORM\Id |
||
35 | * @ORM\GeneratedValue(strategy="IDENTITY") |
||
36 | */ |
||
37 | private $id; |
||
38 | |||
39 | /** |
||
40 | * @var string|null |
||
41 | * |
||
42 | * @ORM\Column(name="company_name", type="string", length=255, nullable=true) |
||
43 | */ |
||
44 | private $company_name; |
||
45 | |||
46 | /** |
||
47 | * @var string|null |
||
48 | * |
||
49 | * @ORM\Column(name="company_kana", type="string", length=255, nullable=true) |
||
50 | */ |
||
51 | private $company_kana; |
||
52 | |||
53 | /** |
||
54 | * @var string|null |
||
55 | * |
||
56 | * @ORM\Column(name="postal_code", type="string", length=8, nullable=true) |
||
57 | */ |
||
58 | private $postal_code; |
||
59 | |||
60 | /** |
||
61 | * @var string|null |
||
62 | * |
||
63 | * @ORM\Column(name="addr01", type="string", length=255, nullable=true) |
||
64 | */ |
||
65 | private $addr01; |
||
66 | |||
67 | /** |
||
68 | * @var string|null |
||
69 | * |
||
70 | * @ORM\Column(name="addr02", type="string", length=255, nullable=true) |
||
71 | */ |
||
72 | private $addr02; |
||
73 | |||
74 | /** |
||
75 | * @var string|null |
||
76 | * |
||
77 | * @ORM\Column(name="phone_number", type="string", length=14, nullable=true) |
||
78 | */ |
||
79 | private $phone_number; |
||
80 | |||
81 | /** |
||
82 | * @var string|null |
||
83 | * |
||
84 | * @ORM\Column(name="business_hour", type="string", length=255, nullable=true) |
||
85 | */ |
||
86 | private $business_hour; |
||
87 | |||
88 | /** |
||
89 | * @var string|null |
||
90 | * |
||
91 | * @ORM\Column(name="email01", type="string", length=255, nullable=true) |
||
92 | */ |
||
93 | private $email01; |
||
94 | |||
95 | /** |
||
96 | * @var string|null |
||
97 | * |
||
98 | * @ORM\Column(name="email02", type="string", length=255, nullable=true) |
||
99 | */ |
||
100 | private $email02; |
||
101 | |||
102 | /** |
||
103 | * @var string|null |
||
104 | * |
||
105 | * @ORM\Column(name="email03", type="string", length=255, nullable=true) |
||
106 | */ |
||
107 | private $email03; |
||
108 | |||
109 | /** |
||
110 | * @var string|null |
||
111 | * |
||
112 | * @ORM\Column(name="email04", type="string", length=255, nullable=true) |
||
113 | */ |
||
114 | private $email04; |
||
115 | |||
116 | /** |
||
117 | * @var string|null |
||
118 | * |
||
119 | * @ORM\Column(name="shop_name", type="string", length=255, nullable=true) |
||
120 | */ |
||
121 | private $shop_name; |
||
122 | |||
123 | /** |
||
124 | * @var string|null |
||
125 | * |
||
126 | * @ORM\Column(name="shop_kana", type="string", length=255, nullable=true) |
||
127 | */ |
||
128 | private $shop_kana; |
||
129 | |||
130 | /** |
||
131 | * @var string|null |
||
132 | * |
||
133 | * @ORM\Column(name="shop_name_eng", type="string", length=255, nullable=true) |
||
134 | */ |
||
135 | private $shop_name_eng; |
||
136 | |||
137 | /** |
||
138 | * @var \DateTime |
||
139 | * |
||
140 | * @ORM\Column(name="update_date", type="datetimetz") |
||
141 | */ |
||
142 | private $update_date; |
||
143 | |||
144 | /** |
||
145 | * @var string|null |
||
146 | * |
||
147 | * @ORM\Column(name="good_traded", type="string", length=4000, nullable=true) |
||
148 | */ |
||
149 | private $good_traded; |
||
150 | |||
151 | /** |
||
152 | * @var string|null |
||
153 | * |
||
154 | * @ORM\Column(name="message", type="string", length=4000, nullable=true) |
||
155 | */ |
||
156 | private $message; |
||
157 | |||
158 | /** |
||
159 | * @var string|null |
||
160 | * |
||
161 | * @ORM\Column(name="delivery_free_amount", type="decimal", precision=12, scale=2, nullable=true) |
||
162 | */ |
||
163 | private $delivery_free_amount; |
||
164 | |||
165 | /** |
||
166 | * @var int|null |
||
167 | * |
||
168 | * @ORM\Column(name="delivery_free_quantity", type="integer", nullable=true, options={"unsigned":true}) |
||
169 | */ |
||
170 | private $delivery_free_quantity; |
||
171 | |||
172 | /** |
||
173 | * @var boolean |
||
174 | * |
||
175 | * @ORM\Column(name="option_mypage_order_status_display", type="boolean", options={"default":true}) |
||
176 | */ |
||
177 | private $option_mypage_order_status_display = true; |
||
178 | |||
179 | /** |
||
180 | * @var boolean |
||
181 | * |
||
182 | * @ORM\Column(name="option_nostock_hidden", type="boolean", options={"default":false}) |
||
183 | */ |
||
184 | private $option_nostock_hidden = false; |
||
185 | |||
186 | /** |
||
187 | * @var boolean |
||
188 | * |
||
189 | * @ORM\Column(name="option_favorite_product", type="boolean", options={"default":true}) |
||
190 | */ |
||
191 | private $option_favorite_product = true; |
||
192 | |||
193 | /** |
||
194 | * @var boolean |
||
195 | * |
||
196 | * @ORM\Column(name="option_product_delivery_fee", type="boolean", options={"default":false}) |
||
197 | */ |
||
198 | private $option_product_delivery_fee = false; |
||
199 | |||
200 | /** |
||
201 | * @var boolean |
||
202 | * |
||
203 | * @ORM\Column(name="option_product_tax_rule", type="boolean", options={"default":false}) |
||
204 | */ |
||
205 | private $option_product_tax_rule = false; |
||
206 | |||
207 | /** |
||
208 | * @var boolean |
||
209 | * |
||
210 | * @ORM\Column(name="option_customer_activate", type="boolean", options={"default":true}) |
||
211 | */ |
||
212 | private $option_customer_activate = true; |
||
213 | |||
214 | /** |
||
215 | * @var boolean |
||
216 | * |
||
217 | * @ORM\Column(name="option_remember_me", type="boolean", options={"default":true}) |
||
218 | */ |
||
219 | private $option_remember_me = true; |
||
220 | |||
221 | /** |
||
222 | * @var string|null |
||
223 | * |
||
224 | * @ORM\Column(name="authentication_key", type="string", length=255, nullable=true) |
||
225 | */ |
||
226 | private $authentication_key; |
||
227 | |||
228 | /** |
||
229 | * @var boolean |
||
230 | * |
||
231 | * @ORM\Column(name="option_point", type="boolean", options={"default":true}) |
||
232 | */ |
||
233 | private $option_point = true; |
||
234 | |||
235 | /** |
||
236 | * @var string |
||
237 | * |
||
238 | * @ORM\Column(name="basic_point_rate", type="decimal", precision=10, scale=0, options={"unsigned":true, "default":1}, nullable=true) |
||
239 | */ |
||
240 | private $basic_point_rate = '1'; |
||
241 | |||
242 | /** |
||
243 | * @var string |
||
244 | * |
||
245 | * @ORM\Column(name="point_conversion_rate", type="decimal", precision=10, scale=0, options={"unsigned":true, "default":1}, nullable=true) |
||
246 | */ |
||
247 | private $point_conversion_rate = '1'; |
||
248 | |||
249 | /** |
||
250 | * @var \Eccube\Entity\Master\Country |
||
251 | * |
||
252 | * @ORM\ManyToOne(targetEntity="Eccube\Entity\Master\Country") |
||
253 | * @ORM\JoinColumns({ |
||
254 | * @ORM\JoinColumn(name="country_id", referencedColumnName="id") |
||
255 | * }) |
||
256 | * @ORM\Cache(usage="NONSTRICT_READ_WRITE") |
||
257 | */ |
||
258 | private $Country; |
||
259 | |||
260 | /** |
||
261 | * @var \Eccube\Entity\Master\Pref |
||
262 | * |
||
263 | * @ORM\ManyToOne(targetEntity="Eccube\Entity\Master\Pref") |
||
264 | * @ORM\JoinColumns({ |
||
265 | * @ORM\JoinColumn(name="pref_id", referencedColumnName="id") |
||
266 | * }) |
||
267 | * @ORM\Cache(usage="NONSTRICT_READ_WRITE") |
||
268 | */ |
||
269 | private $Pref; |
||
270 | |||
271 | /** |
||
272 | * Get id. |
||
273 | * |
||
274 | * @return int |
||
275 | */ |
||
276 | 2 | public function getId() |
|
280 | |||
281 | /** |
||
282 | * Set companyName. |
||
283 | * |
||
284 | * @param string|null $companyName |
||
285 | * |
||
286 | * @return BaseInfo |
||
287 | */ |
||
288 | 2 | public function setCompanyName($companyName = null) |
|
294 | |||
295 | /** |
||
296 | * Get companyName. |
||
297 | * |
||
298 | * @return string|null |
||
299 | */ |
||
300 | 11 | public function getCompanyName() |
|
304 | |||
305 | /** |
||
306 | * Set companyKana. |
||
307 | * |
||
308 | * @param string|null $companyKana |
||
309 | * |
||
310 | * @return BaseInfo |
||
311 | */ |
||
312 | public function setCompanyKana($companyKana = null) |
||
318 | |||
319 | /** |
||
320 | * Get companyKana. |
||
321 | * |
||
322 | * @return string|null |
||
323 | */ |
||
324 | 8 | public function getCompanyKana() |
|
328 | |||
329 | /** |
||
330 | * Set postal_code. |
||
331 | * |
||
332 | * @param string|null $postal_code |
||
333 | * |
||
334 | * @return BaseInfo |
||
335 | */ |
||
336 | public function setPostalCode($postal_code = null) |
||
342 | |||
343 | /** |
||
344 | * Get postal_code. |
||
345 | * |
||
346 | * @return string|null |
||
347 | */ |
||
348 | 9 | public function getPostalCode() |
|
352 | |||
353 | /** |
||
354 | * Set addr01. |
||
355 | * |
||
356 | * @param string|null $addr01 |
||
357 | * |
||
358 | * @return BaseInfo |
||
359 | */ |
||
360 | 2 | public function setAddr01($addr01 = null) |
|
366 | |||
367 | /** |
||
368 | * Get addr01. |
||
369 | * |
||
370 | * @return string|null |
||
371 | */ |
||
372 | 11 | public function getAddr01() |
|
376 | |||
377 | /** |
||
378 | * Set addr02. |
||
379 | * |
||
380 | * @param string|null $addr02 |
||
381 | * |
||
382 | * @return BaseInfo |
||
383 | */ |
||
384 | 2 | public function setAddr02($addr02 = null) |
|
390 | |||
391 | /** |
||
392 | * Get addr02. |
||
393 | * |
||
394 | * @return string|null |
||
395 | */ |
||
396 | 11 | public function getAddr02() |
|
400 | |||
401 | /** |
||
402 | * Set phone_number. |
||
403 | * |
||
404 | * @param string|null $phone_number |
||
405 | * |
||
406 | * @return BaseInfo |
||
407 | */ |
||
408 | 6 | public function setPhoneNumber($phone_number = null) |
|
414 | |||
415 | /** |
||
416 | * Get phone_number. |
||
417 | * |
||
418 | * @return string|null |
||
419 | */ |
||
420 | 12 | public function getPhoneNumber() |
|
424 | |||
425 | /** |
||
426 | * Set businessHour. |
||
427 | * |
||
428 | * @param string|null $businessHour |
||
429 | * |
||
430 | * @return BaseInfo |
||
431 | */ |
||
432 | public function setBusinessHour($businessHour = null) |
||
438 | |||
439 | /** |
||
440 | * Get businessHour. |
||
441 | * |
||
442 | * @return string|null |
||
443 | */ |
||
444 | 9 | public function getBusinessHour() |
|
448 | |||
449 | /** |
||
450 | * Set email01. |
||
451 | * |
||
452 | * @param string|null $email01 |
||
453 | * |
||
454 | * @return BaseInfo |
||
455 | */ |
||
456 | 9 | public function setEmail01($email01 = null) |
|
462 | |||
463 | /** |
||
464 | * Get email01. |
||
465 | * |
||
466 | * @return string|null |
||
467 | */ |
||
468 | 36 | public function getEmail01() |
|
472 | |||
473 | /** |
||
474 | * Set email02. |
||
475 | * |
||
476 | * @param string|null $email02 |
||
477 | * |
||
478 | * @return BaseInfo |
||
479 | */ |
||
480 | 7 | public function setEmail02($email02 = null) |
|
486 | |||
487 | /** |
||
488 | * Get email02. |
||
489 | * |
||
490 | * @return string|null |
||
491 | */ |
||
492 | 22 | public function getEmail02() |
|
496 | |||
497 | /** |
||
498 | * Set email03. |
||
499 | * |
||
500 | * @param string|null $email03 |
||
501 | * |
||
502 | * @return BaseInfo |
||
503 | */ |
||
504 | 7 | public function setEmail03($email03 = null) |
|
510 | |||
511 | /** |
||
512 | * Get email03. |
||
513 | * |
||
514 | * @return string|null |
||
515 | */ |
||
516 | 33 | public function getEmail03() |
|
520 | |||
521 | /** |
||
522 | * Set email04. |
||
523 | * |
||
524 | * @param string|null $email04 |
||
525 | * |
||
526 | * @return BaseInfo |
||
527 | */ |
||
528 | 7 | public function setEmail04($email04 = null) |
|
534 | |||
535 | /** |
||
536 | * Get email04. |
||
537 | * |
||
538 | * @return string|null |
||
539 | */ |
||
540 | 36 | public function getEmail04() |
|
544 | |||
545 | /** |
||
546 | * Set shopName. |
||
547 | * |
||
548 | * @param string|null $shopName |
||
549 | * |
||
550 | * @return BaseInfo |
||
551 | */ |
||
552 | 9 | public function setShopName($shopName = null) |
|
558 | |||
559 | /** |
||
560 | * Get shopName. |
||
561 | * |
||
562 | * @return string|null |
||
563 | */ |
||
564 | 292 | public function getShopName() |
|
568 | |||
569 | /** |
||
570 | * Set shopKana. |
||
571 | * |
||
572 | * @param string|null $shopKana |
||
573 | * |
||
574 | * @return BaseInfo |
||
575 | */ |
||
576 | public function setShopKana($shopKana = null) |
||
582 | |||
583 | /** |
||
584 | * Get shopKana. |
||
585 | * |
||
586 | * @return string|null |
||
587 | */ |
||
588 | 8 | public function getShopKana() |
|
592 | |||
593 | /** |
||
594 | * Set shopNameEng. |
||
595 | * |
||
596 | * @param string|null $shopNameEng |
||
597 | * |
||
598 | * @return BaseInfo |
||
599 | */ |
||
600 | public function setShopNameEng($shopNameEng = null) |
||
606 | |||
607 | /** |
||
608 | * Get shopNameEng. |
||
609 | * |
||
610 | * @return string|null |
||
611 | */ |
||
612 | 8 | public function getShopNameEng() |
|
616 | |||
617 | /** |
||
618 | * Set updateDate. |
||
619 | * |
||
620 | * @param \DateTime $updateDate |
||
621 | * |
||
622 | * @return BaseInfo |
||
623 | */ |
||
624 | 43 | public function setUpdateDate($updateDate) |
|
630 | |||
631 | /** |
||
632 | * Get updateDate. |
||
633 | * |
||
634 | * @return \DateTime |
||
635 | */ |
||
636 | public function getUpdateDate() |
||
640 | |||
641 | /** |
||
642 | * Set goodTraded. |
||
643 | * |
||
644 | * @param string|null $goodTraded |
||
645 | * |
||
646 | * @return BaseInfo |
||
647 | */ |
||
648 | public function setGoodTraded($goodTraded = null) |
||
654 | |||
655 | /** |
||
656 | * Get goodTraded. |
||
657 | * |
||
658 | * @return string|null |
||
659 | */ |
||
660 | 9 | public function getGoodTraded() |
|
664 | |||
665 | /** |
||
666 | * Set message. |
||
667 | * |
||
668 | * @param string|null $message |
||
669 | * |
||
670 | * @return BaseInfo |
||
671 | */ |
||
672 | public function setMessage($message = null) |
||
678 | |||
679 | /** |
||
680 | * Get message. |
||
681 | * |
||
682 | * @return string|null |
||
683 | */ |
||
684 | 9 | public function getMessage() |
|
688 | |||
689 | /** |
||
690 | * Set deliveryFreeAmount. |
||
691 | * |
||
692 | * @param string|null $deliveryFreeAmount |
||
693 | * |
||
694 | * @return BaseInfo |
||
695 | */ |
||
696 | 18 | public function setDeliveryFreeAmount($deliveryFreeAmount = null) |
|
702 | |||
703 | /** |
||
704 | * Get deliveryFreeAmount. |
||
705 | * |
||
706 | * @return string|null |
||
707 | */ |
||
708 | 99 | public function getDeliveryFreeAmount() |
|
712 | |||
713 | /** |
||
714 | * Set deliveryFreeQuantity. |
||
715 | * |
||
716 | * @param int|null $deliveryFreeQuantity |
||
717 | * |
||
718 | * @return BaseInfo |
||
719 | */ |
||
720 | 18 | public function setDeliveryFreeQuantity($deliveryFreeQuantity = null) |
|
726 | |||
727 | /** |
||
728 | * Get deliveryFreeQuantity. |
||
729 | * |
||
730 | * @return int|null |
||
731 | */ |
||
732 | 99 | public function getDeliveryFreeQuantity() |
|
736 | |||
737 | /** |
||
738 | * Set optionMypageOrderStatusDisplay. |
||
739 | * |
||
740 | * @param boolean $optionMypageOrderStatusDisplay |
||
741 | * |
||
742 | * @return BaseInfo |
||
743 | */ |
||
744 | 7 | public function setOptionMypageOrderStatusDisplay($optionMypageOrderStatusDisplay) |
|
750 | |||
751 | /** |
||
752 | * Get optionMypageOrderStatusDisplay. |
||
753 | * |
||
754 | * @return boolean |
||
755 | */ |
||
756 | 8 | public function isOptionMypageOrderStatusDisplay() |
|
760 | |||
761 | /** |
||
762 | * Set optionNostockHidden. |
||
763 | * |
||
764 | * @param integer $optionNostockHidden |
||
765 | * |
||
766 | * @return BaseInfo |
||
767 | */ |
||
768 | public function setOptionNostockHidden($optionNostockHidden) |
||
774 | |||
775 | /** |
||
776 | * Get optionNostockHidden. |
||
777 | * |
||
778 | * @return boolean |
||
779 | */ |
||
780 | 8 | public function isOptionNostockHidden() |
|
784 | |||
785 | /** |
||
786 | * Set optionFavoriteProduct. |
||
787 | * |
||
788 | * @param boolean $optionFavoriteProduct |
||
789 | * |
||
790 | * @return BaseInfo |
||
791 | */ |
||
792 | 10 | public function setOptionFavoriteProduct($optionFavoriteProduct) |
|
798 | |||
799 | /** |
||
800 | * Get optionFavoriteProduct. |
||
801 | * |
||
802 | * @return boolean |
||
803 | */ |
||
804 | 121 | public function isOptionFavoriteProduct() |
|
808 | |||
809 | /** |
||
810 | * Set optionProductDeliveryFee. |
||
811 | * |
||
812 | * @param boolean $optionProductDeliveryFee |
||
813 | * |
||
814 | * @return BaseInfo |
||
815 | */ |
||
816 | public function setOptionProductDeliveryFee($optionProductDeliveryFee) |
||
822 | |||
823 | /** |
||
824 | * Get optionProductDeliveryFee. |
||
825 | * |
||
826 | * @return boolean |
||
827 | */ |
||
828 | 31 | public function isOptionProductDeliveryFee() |
|
832 | |||
833 | /** |
||
834 | * Set optionProductTaxRule. |
||
835 | * |
||
836 | * @param boolean $optionProductTaxRule |
||
837 | * |
||
838 | * @return BaseInfo |
||
839 | */ |
||
840 | 42 | public function setOptionProductTaxRule($optionProductTaxRule) |
|
846 | |||
847 | /** |
||
848 | * Get optionProductTaxRule. |
||
849 | * |
||
850 | * @return boolean |
||
851 | */ |
||
852 | 432 | public function isOptionProductTaxRule() |
|
856 | |||
857 | /** |
||
858 | * Set optionCustomerActivate. |
||
859 | * |
||
860 | * @param boolean $optionCustomerActivate |
||
861 | * |
||
862 | * @return BaseInfo |
||
863 | */ |
||
864 | 8 | public function setOptionCustomerActivate($optionCustomerActivate) |
|
870 | |||
871 | /** |
||
872 | * Get optionCustomerActivate. |
||
873 | * |
||
874 | * @return boolean |
||
875 | */ |
||
876 | 9 | public function isOptionCustomerActivate() |
|
880 | |||
881 | /** |
||
882 | * Set optionRememberMe. |
||
883 | * |
||
884 | * @param boolean $optionRememberMe |
||
885 | * |
||
886 | * @return BaseInfo |
||
887 | */ |
||
888 | 7 | public function setOptionRememberMe($optionRememberMe) |
|
894 | |||
895 | /** |
||
896 | * Get optionRememberMe. |
||
897 | * |
||
898 | * @return boolean |
||
899 | */ |
||
900 | 11 | public function isOptionRememberMe() |
|
904 | |||
905 | /** |
||
906 | * Set authenticationKey. |
||
907 | * |
||
908 | * @param string|null $authenticationKey |
||
909 | * |
||
910 | * @return BaseInfo |
||
911 | */ |
||
912 | public function setAuthenticationKey($authenticationKey = null) |
||
918 | |||
919 | /** |
||
920 | * Get authenticationKey. |
||
921 | * |
||
922 | * @return string|null |
||
923 | */ |
||
924 | public function getAuthenticationKey() |
||
928 | |||
929 | /** |
||
930 | * Set country. |
||
931 | * |
||
932 | * @param \Eccube\Entity\Master\Country|null $country |
||
933 | * |
||
934 | * @return BaseInfo |
||
935 | */ |
||
936 | public function setCountry(\Eccube\Entity\Master\Country $country = null) |
||
942 | |||
943 | /** |
||
944 | * Get country. |
||
945 | * |
||
946 | * @return \Eccube\Entity\Master\Country|null |
||
947 | */ |
||
948 | public function getCountry() |
||
952 | |||
953 | /** |
||
954 | * Set pref. |
||
955 | * |
||
956 | * @param \Eccube\Entity\Master\Pref|null $pref |
||
957 | * |
||
958 | * @return BaseInfo |
||
959 | */ |
||
960 | public function setPref(\Eccube\Entity\Master\Pref $pref = null) |
||
966 | |||
967 | /** |
||
968 | * Get pref. |
||
969 | * |
||
970 | * @return \Eccube\Entity\Master\Pref|null |
||
971 | */ |
||
972 | 8 | public function getPref() |
|
976 | |||
977 | /** |
||
978 | * Set optionPoint |
||
979 | * |
||
980 | * @param boolean $optionPoint |
||
981 | * |
||
982 | * @return BaseInfo |
||
983 | */ |
||
984 | 7 | public function setOptionPoint($optionPoint) |
|
990 | |||
991 | /** |
||
992 | * Get optionPoint |
||
993 | * |
||
994 | * @return boolean |
||
995 | */ |
||
996 | 232 | public function isOptionPoint() |
|
1000 | |||
1001 | /** |
||
1002 | * Set pointConversionRate |
||
1003 | * |
||
1004 | * @param string $pointConversionRate |
||
1005 | * |
||
1006 | * @return BaseInfo |
||
1007 | */ |
||
1008 | 7 | public function setPointConversionRate($pointConversionRate) |
|
1014 | |||
1015 | /** |
||
1016 | * Get pointConversionRate |
||
1017 | * |
||
1018 | * @return string |
||
1019 | */ |
||
1020 | 19 | public function getPointConversionRate() |
|
1024 | |||
1025 | /** |
||
1026 | * Set basicPointRate |
||
1027 | * |
||
1028 | * @param string $basicPointRate |
||
1029 | * |
||
1030 | * @return BaseInfo |
||
1031 | */ |
||
1032 | 7 | public function setBasicPointRate($basicPointRate) |
|
1038 | |||
1039 | /** |
||
1040 | * Get basicPointRate |
||
1041 | * |
||
1042 | * @return string |
||
1043 | */ |
||
1044 | 202 | public function getBasicPointRate() |
|
1048 | } |
||
1049 |
This check looks for assignments to scalar types that may be of the wrong type.
To ensure the code behaves as expected, it may be a good idea to add an explicit type cast.