Complex classes like ValidateHelper 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 ValidateHelper, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
8 | class ValidateHelper |
||
9 | { |
||
10 | use traits\Errorable; |
||
11 | |||
12 | private $objPartitaIVA; |
||
13 | private $objCFChecker; |
||
14 | |||
15 | /**+ |
||
16 | * ValidateHelper constructor. |
||
17 | * |
||
18 | * @param \fdisotto\PartitaIVA $objPartitaIVA |
||
19 | * @param \CodiceFiscale\Checker $objCFChecker |
||
20 | */ |
||
21 | 54 | public function __construct(\fdisotto\PartitaIVA $objPartitaIVA, \CodiceFiscale\Checker $objCFChecker) |
|
27 | |||
28 | /** |
||
29 | * @param $codiceRegione |
||
30 | */ |
||
31 | 3 | public function checkCodiceRegione($codiceRegione) |
|
37 | |||
38 | /** |
||
39 | * @param $codiceSSA |
||
40 | */ |
||
41 | 3 | public function checkCodiceSSA($codiceSSA) |
|
47 | |||
48 | /** |
||
49 | * @param $dateStr |
||
50 | * |
||
51 | * @return bool |
||
52 | */ |
||
53 | 9 | public function isoDateValidate($dateStr) |
|
63 | |||
64 | /**+ |
||
65 | * @param $arrVociSpesa |
||
66 | */ |
||
67 | public function checkArrVociSpesa($arrVociSpesa) |
||
90 | |||
91 | /** |
||
92 | * @param $arrSpesa |
||
93 | */ |
||
94 | public function checkArrSpesa($arrSpesa) |
||
107 | |||
108 | /** |
||
109 | * @param $pIva |
||
110 | */ |
||
111 | 3 | public function checkPIva($pIva) |
|
122 | |||
123 | /** |
||
124 | * @param $cfProprietario |
||
125 | */ |
||
126 | 3 | public function checkCfProprietario($cfProprietario) |
|
137 | |||
138 | /** |
||
139 | * @param $codiceAsl |
||
140 | */ |
||
141 | 3 | public function checkCodiceAsl($codiceAsl) |
|
147 | |||
148 | /** |
||
149 | * @param $campo |
||
150 | * @param $valore |
||
151 | */ |
||
152 | 6 | private function checkDataValida($campo, $valore) |
|
162 | |||
163 | /** |
||
164 | * @param $valore |
||
165 | */ |
||
166 | 3 | public function checkTipoSpesa($valore) |
|
172 | |||
173 | /** |
||
174 | * @param $valore |
||
175 | */ |
||
176 | 3 | private function checkImporto($valore) |
|
182 | |||
183 | /** |
||
184 | * @param $valore |
||
185 | * @param $campo |
||
186 | */ |
||
187 | 3 | private function checkRequiredField($valore, $campo) |
|
193 | |||
194 | /** |
||
195 | * @param $campo |
||
196 | * @param $valore |
||
197 | */ |
||
198 | 3 | private function checkDataEmissione($campo, $valore) |
|
202 | |||
203 | /** |
||
204 | * @param $valore |
||
205 | */ |
||
206 | 3 | public function checkFlagOperazione($valore) |
|
212 | |||
213 | /** |
||
214 | * @param $valore |
||
215 | */ |
||
216 | 3 | private function checkCfCittadino($valore) |
|
222 | |||
223 | /** |
||
224 | * @param $valore |
||
225 | */ |
||
226 | 3 | public function checkDispositivo($valore) |
|
232 | |||
233 | /** |
||
234 | * @param $valore |
||
235 | */ |
||
236 | 3 | public function checkNumDocumento($valore) |
|
242 | |||
243 | /** |
||
244 | * @param $valore |
||
245 | * @param int $maxLen |
||
246 | * @param bool|false $zeroFilled |
||
247 | * |
||
248 | * @return bool |
||
249 | */ |
||
250 | 9 | public function checkNumericField($valore, $maxLen = 0, $zeroFilled = false) |
|
268 | |||
269 | /** |
||
270 | * @param $rigaSpesa |
||
271 | */ |
||
272 | private function checkRigaSpesa($rigaSpesa) |
||
283 | |||
284 | /** |
||
285 | * @param $campo |
||
286 | * @param $valore |
||
287 | */ |
||
288 | private function checkDatiSpesa($campo, $valore) |
||
320 | } |
||
321 |