Complex classes like Recibo 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 Recibo, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 34 | class Recibo extends Retorno |
||
| 35 | { |
||
| 36 | |||
| 37 | const INFO_TAGNAME = 'infRec'; |
||
| 38 | |||
| 39 | private $numero; |
||
| 40 | private $tempo_medio; |
||
| 41 | private $codigo; |
||
| 42 | private $mensagem; |
||
| 43 | private $modelo; |
||
| 44 | |||
| 45 | 1 | public function __construct($recibo = array()) |
|
| 49 | |||
| 50 | /** |
||
| 51 | * Número do Recibo |
||
| 52 | */ |
||
| 53 | public function getNumero($normalize = false) |
||
| 54 | { |
||
| 55 | if (!$normalize) { |
||
| 56 | return $this->numero; |
||
| 57 | } |
||
| 58 | return $this->numero; |
||
| 59 | } |
||
| 60 | |||
| 61 | 1 | public function setNumero($numero) |
|
| 66 | |||
| 67 | /** |
||
| 68 | * Tempo médio de resposta do serviço (em segundos) dos últimos 5 minutos |
||
| 69 | */ |
||
| 70 | public function getTempoMedio($normalize = false) |
||
| 71 | { |
||
| 72 | if (!$normalize) { |
||
| 73 | return $this->tempo_medio; |
||
| 74 | } |
||
| 75 | return $this->tempo_medio; |
||
| 76 | } |
||
| 77 | |||
| 78 | 1 | public function setTempoMedio($tempo_medio) |
|
| 83 | |||
| 84 | /** |
||
| 85 | * Código da Mensagem (v2.0) alterado para tamanho variavel 1-4. |
||
| 86 | * (NT2011/004) |
||
| 87 | */ |
||
| 88 | public function getCodigo($normalize = false) |
||
| 89 | { |
||
| 90 | if (!$normalize) { |
||
| 91 | return $this->codigo; |
||
| 92 | } |
||
| 93 | return $this->codigo; |
||
| 94 | } |
||
| 95 | |||
| 96 | 1 | public function setCodigo($codigo) |
|
| 101 | |||
| 102 | /** |
||
| 103 | * Mensagem da SEFAZ para o emissor. (v2.0) |
||
| 104 | */ |
||
| 105 | public function getMensagem($normalize = false) |
||
| 106 | { |
||
| 107 | if (!$normalize) { |
||
| 108 | return $this->mensagem; |
||
| 109 | } |
||
| 110 | return $this->mensagem; |
||
| 111 | } |
||
| 112 | |||
| 113 | 1 | public function setMensagem($mensagem) |
|
| 118 | |||
| 119 | /** |
||
| 120 | * Código do modelo do Documento Fiscal. 55 = NF-e; 65 = NFC-e. |
||
| 121 | * @param boolean $normalize informa se o modelo deve estar no formato do XML |
||
| 122 | * @return mixed modelo do Envio |
||
| 123 | */ |
||
| 124 | public function getModelo($normalize = false) |
||
| 125 | { |
||
| 126 | if (!$normalize) { |
||
| 127 | return $this->modelo; |
||
| 128 | } |
||
| 129 | switch ($this->modelo) { |
||
| 130 | case Nota::MODELO_NFE: |
||
| 131 | return '55'; |
||
| 132 | case Nota::MODELO_NFCE: |
||
| 133 | return '65'; |
||
| 134 | } |
||
| 135 | return $this->modelo; |
||
| 136 | } |
||
| 137 | |||
| 138 | /** |
||
| 139 | * Altera o valor do Modelo para o informado no parâmetro |
||
| 140 | * @param mixed $modelo novo valor para Modelo |
||
| 141 | * @return Envio A própria instância da classe |
||
| 142 | */ |
||
| 143 | 1 | public function setModelo($modelo) |
|
| 156 | |||
| 157 | public function toArray($recursive = false) |
||
| 158 | { |
||
| 159 | $recibo = parent::toArray($recursive); |
||
| 160 | $recibo['numero'] = $this->getNumero(); |
||
| 161 | $recibo['tempo_medio'] = $this->getTempoMedio(); |
||
| 167 | |||
| 168 | 1 | public function fromArray($recibo = array()) |
|
| 203 | |||
| 204 | public function envia($dom) |
||
| 221 | |||
| 222 | public function consulta($nota = null) |
||
| 236 | |||
| 237 | public function getNode($name = null) |
||
| 251 | |||
| 252 | public function loadNode($element, $name = null) |
||
| 276 | |||
| 277 | /** |
||
| 278 | * Valida o documento após assinar |
||
| 279 | */ |
||
| 280 | public function validar($dom) |
||
| 303 | } |
||
| 304 |