Complex classes like OrderEntity 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 OrderEntity, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
5 | class OrderEntity |
||
6 | { |
||
7 | private $customerName; |
||
8 | private $customerEmail; |
||
9 | private $customerPhone; |
||
10 | private $facility; |
||
11 | private $facilityID; |
||
12 | private $warehouseTransactionID; |
||
13 | private $referenceNum; |
||
14 | private $pONum; |
||
15 | private $retailer; |
||
16 | private $shipToCompanyName; |
||
17 | private $shipToName; |
||
18 | private $shipToEmail; |
||
19 | private $shipToPhone; |
||
20 | private $shipToAddress1; |
||
21 | private $shipToAddress2; |
||
22 | private $shipToCity; |
||
23 | private $shipToState; |
||
24 | private $shipToZip; |
||
25 | private $shipToCountry; |
||
26 | private $shipMethod; |
||
27 | private $markForName; |
||
28 | private $batchOrderID; |
||
29 | private $creationDate; |
||
30 | private $earliestShipDate; |
||
31 | private $shipCancelDate; |
||
32 | private $pickupDate; |
||
33 | private $carrier; |
||
34 | private $billingCode; |
||
35 | private $totWeight; |
||
36 | private $totCuFt; |
||
37 | private $totPackages; |
||
38 | private $totOrdQty; |
||
39 | private $totLines; |
||
40 | private $notes; |
||
41 | private $overAllocated; |
||
42 | private $pickTicketPrintDate; |
||
43 | private $processDate; |
||
44 | private $trackingNumber; |
||
45 | private $loadNumber; |
||
46 | private $billOfLading; |
||
47 | private $masterBillOfLading; |
||
48 | private $aSNSentDate; |
||
49 | private $confirmASNSentDate; |
||
50 | private $rememberRowInfo; |
||
51 | |||
52 | public function toArray() |
||
101 | |||
102 | public function getCustomerName() |
||
106 | |||
107 | public function setCustomerName($customerName) |
||
111 | |||
112 | public function getCustomerEmail() |
||
116 | |||
117 | public function setCustomerEmail($customerEmail) |
||
121 | |||
122 | public function getCustomerPhone() |
||
126 | |||
127 | public function setCustomerPhone($customerPhone) |
||
131 | |||
132 | public function getFacility() |
||
136 | |||
137 | public function setFacility($facility) |
||
141 | |||
142 | public function getFacilityID() |
||
146 | |||
147 | public function setFacilityID($facilityID) |
||
151 | |||
152 | public function getWarehouseTransactionID() |
||
156 | |||
157 | public function setWarehouseTransactionID($warehouseTransactionID) |
||
161 | |||
162 | public function getReferenceNum() |
||
166 | |||
167 | public function setReferenceNum($referenceNum) |
||
171 | |||
172 | public function getPONum() |
||
176 | |||
177 | public function setPONum($pONum) |
||
181 | |||
182 | public function getRetailer() |
||
186 | |||
187 | public function setRetailer($retailer) |
||
191 | |||
192 | public function getShipToCompanyName() |
||
196 | |||
197 | public function setShipToCompanyName($shipToCompanyName) |
||
201 | |||
202 | public function getShipToName() |
||
206 | |||
207 | public function setShipToName($shipToName) |
||
211 | |||
212 | public function getShipToEmail() |
||
216 | |||
217 | public function setShipToEmail($shipToEmail) |
||
221 | |||
222 | public function getShipToPhone() |
||
226 | |||
227 | public function setShipToPhone($shipToPhone) |
||
231 | |||
232 | public function getShipToAddress1() |
||
236 | |||
237 | public function setShipToAddress1($shipToAddress1) |
||
241 | |||
242 | public function getShipToAddress2() |
||
246 | |||
247 | public function setShipToAddress2($shipToAddress2) |
||
251 | |||
252 | public function getShipToCity() |
||
256 | |||
257 | public function setShipToCity($shipToCity) |
||
261 | |||
262 | public function getShipToState() |
||
266 | |||
267 | public function setShipToState($shipToState) |
||
271 | |||
272 | public function getShipToZip() |
||
276 | |||
277 | public function setShipToZip($shipToZip) |
||
281 | |||
282 | public function getShipToCountry() |
||
286 | |||
287 | public function setShipToCountry($shipToCountry) |
||
291 | |||
292 | public function getShipMethod() |
||
296 | |||
297 | public function setShipMethod($shipMethod) |
||
301 | |||
302 | public function getMarkForName() |
||
306 | |||
307 | public function setMarkForName($markForName) |
||
311 | |||
312 | public function getBatchOrderID() |
||
316 | |||
317 | public function setBatchOrderID($batchOrderID) |
||
321 | |||
322 | public function getCreationDate() |
||
326 | |||
327 | public function setCreationDate($creationDate) |
||
331 | |||
332 | public function getEarliestShipDate() |
||
336 | |||
337 | public function setEarliestShipDate($earliestShipDate) |
||
341 | |||
342 | public function getShipCancelDate() |
||
346 | |||
347 | public function setShipCancelDate($shipCancelDate) |
||
351 | |||
352 | public function getPickupDate() |
||
356 | |||
357 | public function setPickupDate($pickupDate) |
||
361 | |||
362 | public function getCarrier() |
||
366 | |||
367 | public function setCarrier($carrier) |
||
371 | |||
372 | public function getBillingCode() |
||
376 | |||
377 | public function setBillingCode($billingCode) |
||
381 | |||
382 | public function getTotWeight() |
||
386 | |||
387 | public function setTotWeight($totWeight) |
||
391 | |||
392 | public function getTotCuFt() |
||
396 | |||
397 | public function setTotCuFt($totCuFt) |
||
401 | |||
402 | public function getTotPackages() |
||
406 | |||
407 | public function setTotPackages($totPackages) |
||
411 | |||
412 | public function getTotOrdQty() |
||
416 | |||
417 | public function setTotOrdQty($totOrdQty) |
||
421 | |||
422 | public function getTotLines() |
||
426 | |||
427 | public function setTotLines($totLines) |
||
431 | |||
432 | public function getNotes() |
||
436 | |||
437 | public function setNotes($notes) |
||
441 | |||
442 | public function getOverAllocated() |
||
446 | |||
447 | public function setOverAllocated($overAllocated) |
||
451 | |||
452 | public function getPickTicketPrintDate() |
||
456 | |||
457 | public function setPickTicketPrintDate($pickTicketPrintDate) |
||
461 | |||
462 | public function getProcessDate() |
||
466 | |||
467 | public function setProcessDate($processDate) |
||
471 | |||
472 | public function getTrackingNumber() |
||
476 | |||
477 | public function setTrackingNumber($trackingNumber) |
||
481 | |||
482 | public function getLoadNumber() |
||
486 | |||
487 | public function setLoadNumber($loadNumber) |
||
491 | |||
492 | public function getBillOfLading() |
||
496 | |||
497 | public function setBillOfLading($billOfLading) |
||
501 | |||
502 | public function getMasterBillOfLading() |
||
506 | |||
507 | public function setMasterBillOfLading($masterBillOfLading) |
||
511 | |||
512 | public function getASNSentDate() |
||
516 | |||
517 | public function setASNSentDate($aSNSentDate) |
||
521 | |||
522 | public function getConfirmASNSentDate() |
||
526 | |||
527 | public function setConfirmASNSentDate($confirmASNSentDate) |
||
531 | |||
532 | public function getRememberRowInfo() |
||
536 | |||
537 | public function setRememberRowInfo($rememberRowInfo) |
||
541 | } |
||
542 |