Linha do tempo de Refatoração de uma classe java
Licença atual: CC BY-SA 3.0
22 eventos
quando alternar formato | o que | por | licença | comentário | |
---|---|---|---|---|---|
21/02/2015 às 21:36 | histórico | editada | Maniero♦ |
tags editadas
|
|
27/10/2014 às 15:58 | voto | aceitação | jsantos1991 | ||
22/10/2014 às 13:35 | comentário | adicionado | jsantos1991 | @Renan obrigado pelas duas dicas, a do código comentado que vou remover e a do controle de versão, que na verdade não estou a usar(tenho feito a mão), mas este projeto está na fase final logo num próximo sem duvida que vou usar. | |
22/10/2014 às 13:35 | comentário | adicionado | jsantos1991 | @Caffé obrigado pelos Links... | |
22/10/2014 às 13:34 | comentário | adicionado | jsantos1991 | @Math obrigado, no inicio é mesmo complicado saber o que é apropriado ou nao... | |
22/10/2014 às 12:54 | comentário | adicionado | Oralista de Sistemas |
(...)e algum código comentado para o caso de ser necessário no futuro(...) Isso é mais tóxico do que urânio. Livre-se desse código. Se algum dia esse código se tornar necessário, vocês podem recuperá-lo a partir do repositório. Se isso acontecer, jogue na loteria também, porque uma vida de experiência com isso diz que código comentado voltar a ser útil é algo que ocorre uma vez a cada lua azul. E se não tiver controle de versão, parem o projeto até começarem a usar um.
|
|
22/10/2014 às 12:54 | comentário | adicionado | Caffé | @jsantos1991 São conceitos de Orientação a Objetos. Este link que achei agora parece bom: caelum.com.br/apostila-java-orientacao-objetos Esta seção em especial deve dar uma noção da diferença conceitual entre publicar uma variável ou atributo (quem garante as regas de negócio, como validação dos valores setados?): caelum.com.br/apostila-java-orientacao-objetos/… | |
22/10/2014 às 12:53 | comentário | adicionado | Math♦ | @jsantos1991 não existe certo ou errado, existe apropriado ou inapropriado. O código deve ser apropriado para as suas necessidades, é claro que muitas vezes é difícil enxergar a médio ou longo prazo o que é o mais apropriado para você. Se seu software tiver vida longa provavelmente terá que refatorá-lo mais algumas vezes, e com experiência sofrerá menos nos projetos seguintes. Pra começar, tente nunca fugir dos princípios da OO mais básicos, como o da responsabilidade única (alta coesão) e baixo acoplamento. | |
22/10/2014 às 12:33 | comentário | adicionado | jsantos1991 | @Caffé deu para esclarecer um pouco mais, mas ainda não consegui compreender como gostava, se poderes enviar um link|palavras chaves que fale sobre isso [EN ou PT] ficava agradecido. | |
22/10/2014 às 12:25 | comentário | adicionado | Caffé | @jsantos1991 Não se trata do nome certo, o importante é entendermos que uma classe publica os dados que ela quer expôr, suas características e estado, o que pode ser um pouco mais profundo do que simplesmente aumentar a visibilidade de uma variável. Esse assunto é extenso, rende discussões úteis, mas inadequadas para este espaço. | |
22/10/2014 às 12:25 | comentário | adicionado | jsantos1991 | @Math sim como disseste e muito bem só tenho mesmo uma luz, logo nunca tenho bem a certeza se estou a fazer direito ou não, mas quanto a essa parte acho que está certo e felizmente, neste caso como não tem interface gráfica porque acho que é quando consigo destruir todo o mvc [mas esta parte já é [OFF-TOPIC]] | |
22/10/2014 às 12:11 | comentário | adicionado | jsantos1991 | @Caffé Obrigado só uma duvida, quando se refere a "variáveis" e sim "atributos" quer dizer que é errado dizer "variáveis" certo? porque o nome correto é "atributos" só para esclarecer o que disseste. | |
22/10/2014 às 12:09 | comentário | adicionado | Math♦ | @jsantos1991 que bom que já tem uma luz! Mas em opinião minha, eu faria a parte de acesso ao webservices fora do controller, em um DAO específico, eu inclusive tenho um nome para essa camada, mas não sei se é consenso, eu chamo de Resource. Talvez eu esteja falando besteira, mas é assim que eu faço e eu acho que atende aos princípios da OO. | |
22/10/2014 às 12:05 | comentário | adicionado | jsantos1991 |
obrigado @Math quanto ao ServidorService_2 foi só um exemplo estúpido mas não sabia mesmo que outro nome poderia dar agora com a ajuda do @caffé e com a resposta do @Cold já tenho uma ideia de como fazer... Para mim o "Servidor_Service" é o meu Controller e nele sempre que preciso de algum dado, seja ele do webservice ou BD só tenho de chamar o método que está em outras classes: Ex: new ProdutoDao().guardar(produto); ou listProdutos=getProdutosWebService(); é a isto que se está a referir certo?
|
|
22/10/2014 às 11:59 | análise | Votos para fechar | |||
22/10/2014 às 13:34 | |||||
22/10/2014 às 11:49 | resposta | adicionado | Cold | linha do tempo pontuação: 4 | |
22/10/2014 às 11:44 | comentário | adicionado | Math♦ | Independente do que fizer, não crie ServidorService_2!! Acredito que boa parte desse código deveria estar no Controller, outra parte do DAO com o BD local, outra no DAO para webservice (independente do nome que preferir dar para essa camada) | |
22/10/2014 às 11:30 | comentário | adicionado | Caffé |
No momento que você listou 4 responsabilidades desta classe, você mesmo identificou no mínimo 4 novas classes. Para você gostar do seu código, cada classe deve ter uma única responsabilidade. Não faça ServidorService_2 e ServidorService_3 pois isso não vai melhorar em nada o seu código (talvez piore). Para ter acesso às variáveis , sim, é através de get s. Mas você não deve publicar "variáveis" e sim "atributos" da sua classe (a técnica é a mesma mas o conceito é diferente e ajuda a perceber se o código está ok). Não vou colocar isso como resposta porque acho que é off-topic para o SO.
|
|
22/10/2014 às 11:13 | resposta | adicionado | Eder F. Freitas | linha do tempo pontuação: 2 | |
22/10/2014 às 11:01 | histórico | editada | jsantos1991 | CC BY-SA 3.0 |
removeu 7 caracteres do conteúdo
|
22/10/2014 às 10:37 | resposta | adicionado | paulohddias | linha do tempo pontuação: 3 | |
22/10/2014 às 10:14 | histórico | perguntada | jsantos1991 | CC BY-SA 3.0 |