Comments on: Esse código que acabei de escrever está bom?

By: Gabriel Amorim

Gabriel Amorim — Tue, 26 Jan 2010 20:32:28 +0000

Acredito que carece de divisão de responsabilidades, no exemplo acima foge a regra de desenvolvimento orientado a Objeto, esta tudo definito e trabalhado em apenas uma clase, começaria com a criaçao de um metodo para cada opcao do Menu.
Tornaria as classes BufferedReader e PrintStream extensões da classe DVD, e por sua vez DVD se tornaria Super, etc e por ai vai.

Abraços seu Aluno Gabriel Amorim FJ-11 da Caelum.

By: ACMattos

ACMattos — Thu, 15 Oct 2009 21:24:11 +0000

Temos ter em mente o grau de manutenibilidade de nosso sistema. Uma boa organização da classe já é o inicio do refactor para organizar o sistema em classes que colaboram entre si.
Mas vamos aguardar o próximo post. Passei para prestigiar o blog do amigo (que é muito bom)

By: ACMattos

ACMattos — Thu, 15 Oct 2009 21:21:25 +0000

Nosso sistema, como proposto, é um programa procedural. Não há divisão de responsabilidades. Uma única classe é responsável pela gerência da interface, CRUD de dados e é o próprio mecanismo de persistência.
Além destes aspectos existem os mencionados pelo Marcio. Temos q

By: Márcio

Márcio — Thu, 15 Oct 2009 14:10:00 +0000

Corrigindo a primeira frase do comentário anterior… “o método tem QUE SER simples e objetivo…”

By: Márcio

Márcio — Thu, 15 Oct 2009 14:09:19 +0000

Não é para ficar mais “bonito”, o método tem quer simples e objetivo e fazer apenas o que foi designado a ele. Isso auxilia muito na hora de uma manutenção… Imagina esse código aí com 20 opções a trabalheira que iria dar para um desenvolvedor dar uma manutenção (principalmente para que não conhece o código). Acho que essa divisão de métodos já seria um bom começo…

By: Sergio Azevedo Junior

Sergio Azevedo Junior — Thu, 15 Oct 2009 14:01:57 +0000

Mas porque deveriamos dividir as opções em métodos diferentes? Afinal o codigo apresentado funciona. Isso é só pra ficar mais bonito?

By: Márcio

Márcio — Thu, 15 Oct 2009 13:04:08 +0000

Bom, pra começar, poderíamos dividir as opções em métodos diferentes….