mazinsw /
nfe-api
Complex classes like Inutilizacao 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 Inutilizacao, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 37 | class Inutilizacao extends Retorno |
||
|
|
|||
| 38 | { |
||
| 39 | |||
| 40 | private $id; |
||
| 41 | private $ano; |
||
| 42 | private $cnpj; |
||
| 43 | private $modelo; |
||
| 44 | private $serie; |
||
| 45 | private $inicio; |
||
| 46 | private $final; |
||
| 47 | private $justificativa; |
||
| 48 | private $numero; |
||
| 49 | |||
| 50 | 1 | public function __construct($inutilizacao = array()) |
|
| 54 | |||
| 55 | /** |
||
| 56 | * Formado por: |
||
| 57 | * ID = Literal |
||
| 58 | * 43 = Código Estado |
||
| 59 | * 15 = Ano |
||
| 60 | * |
||
| 61 | * 00000000000000 = CNPJ |
||
| 62 | * 55 = Modelo |
||
| 63 | * 001 = Série |
||
| 64 | * 000000411 = |
||
| 65 | * Número Inicial |
||
| 66 | * 000000411 = Número Final |
||
| 67 | */ |
||
| 68 | public function getID($normalize = false) |
||
| 69 | { |
||
| 70 | if (!$normalize) { |
||
| 71 | return $this->id; |
||
| 72 | } |
||
| 73 | return 'ID'.$this->id; |
||
| 74 | } |
||
| 75 | |||
| 76 | 1 | public function setID($id) |
|
| 81 | |||
| 82 | public function getAno($normalize = false) |
||
| 83 | { |
||
| 84 | if (!$normalize) { |
||
| 85 | return $this->ano; |
||
| 86 | } |
||
| 87 | return $this->ano % 100; |
||
| 88 | } |
||
| 89 | |||
| 90 | 1 | public function setAno($ano) |
|
| 95 | |||
| 96 | public function getCNPJ($normalize = false) |
||
| 97 | { |
||
| 98 | if (!$normalize) { |
||
| 99 | return $this->cnpj; |
||
| 100 | } |
||
| 101 | return $this->cnpj; |
||
| 102 | } |
||
| 103 | |||
| 104 | 1 | public function setCNPJ($cnpj) |
|
| 109 | |||
| 110 | /** |
||
| 111 | * Código do modelo do Documento Fiscal. 55 = NF-e; 65 = NFC-e. |
||
| 112 | * @param boolean $normalize informa se o modelo deve estar no formato do XML |
||
| 113 | * @return mixed modelo do Envio |
||
| 114 | */ |
||
| 115 | public function getModelo($normalize = false) |
||
| 116 | { |
||
| 117 | if (!$normalize) { |
||
| 118 | return $this->modelo; |
||
| 119 | } |
||
| 120 | switch ($this->modelo) { |
||
| 121 | case Nota::MODELO_NFE: |
||
| 122 | return '55'; |
||
| 123 | case Nota::MODELO_NFCE: |
||
| 124 | return '65'; |
||
| 125 | } |
||
| 126 | return $this->modelo; |
||
| 127 | } |
||
| 128 | |||
| 129 | /** |
||
| 130 | * Altera o valor do Modelo para o informado no parâmetro |
||
| 131 | * @param mixed $modelo novo valor para Modelo |
||
| 132 | * @return Envio A própria instância da classe |
||
| 133 | */ |
||
| 134 | 1 | public function setModelo($modelo) |
|
| 147 | |||
| 148 | public function getSerie($normalize = false) |
||
| 149 | { |
||
| 150 | if (!$normalize) { |
||
| 151 | return $this->serie; |
||
| 152 | } |
||
| 153 | return $this->serie; |
||
| 154 | } |
||
| 155 | |||
| 156 | 1 | public function setSerie($serie) |
|
| 161 | |||
| 162 | public function getInicio($normalize = false) |
||
| 163 | { |
||
| 164 | if (!$normalize) { |
||
| 165 | return $this->inicio; |
||
| 166 | } |
||
| 167 | return $this->inicio; |
||
| 168 | } |
||
| 169 | |||
| 170 | 1 | public function setInicio($inicio) |
|
| 175 | |||
| 176 | public function getFinal($normalize = false) |
||
| 177 | { |
||
| 178 | if (!$normalize) { |
||
| 179 | return $this->final; |
||
| 180 | } |
||
| 181 | return $this->final; |
||
| 182 | } |
||
| 183 | |||
| 184 | 1 | public function setFinal($final) |
|
| 189 | |||
| 190 | public function getJustificativa($normalize = false) |
||
| 191 | { |
||
| 192 | if (!$normalize) { |
||
| 193 | return $this->justificativa; |
||
| 194 | } |
||
| 195 | return $this->justificativa; |
||
| 196 | } |
||
| 197 | |||
| 198 | 1 | public function setJustificativa($justificativa) |
|
| 203 | |||
| 204 | public function getNumero($normalize = false) |
||
| 205 | { |
||
| 206 | if (!$normalize) { |
||
| 207 | return $this->numero; |
||
| 208 | } |
||
| 209 | return $this->numero; |
||
| 210 | } |
||
| 211 | |||
| 212 | 1 | public function setNumero($numero) |
|
| 217 | |||
| 218 | /** |
||
| 219 | * Informa se os números foram inutilizados |
||
| 220 | */ |
||
| 221 | public function isInutilizado() |
||
| 222 | { |
||
| 223 | return $this->getStatus() == '102'; |
||
| 224 | } |
||
| 225 | |||
| 226 | public function toArray($recursive = false) |
||
| 227 | { |
||
| 240 | |||
| 241 | 1 | public function fromArray($inutilizacao = array()) |
|
| 296 | |||
| 297 | public function gerarID() |
||
| 311 | |||
| 312 | public function getNode($name = null) |
||
| 342 | |||
| 343 | public function getReturnNode() |
||
| 373 | |||
| 374 | public function loadNode($element, $name = null) |
||
| 432 | |||
| 433 | public function envia($dom) |
||
| 448 | |||
| 449 | /** |
||
| 450 | * Assina o XML com a assinatura eletrônica do tipo A1 |
||
| 451 | */ |
||
| 452 | public function assinar($dom = null) |
||
| 468 | |||
| 469 | /** |
||
| 470 | * Valida o documento após assinar |
||
| 471 | */ |
||
| 472 | public function validar($dom) |
||
| 495 | } |
||
| 496 |
Loading history...
The class complexity is the sum of the complexity of all methods. A very high value is usually an indication that your class does not follow the single reponsibility principle and does more than one job.
Some resources for further reading:
You can also find more detailed suggestions for refactoring in the “Code” section of your repository.