Virgiliopc / Crud-Java-MySQL-JDBC

Crud Java com MySQL JDBC
1 stars 0 forks source link

Gostei da unificacao das operacoes #1

Open darkzera opened 1 year ago

darkzera commented 1 year ago

https://github.com/Virgiliopc/Crud-Java-MySQL-JDBC/blob/7ff196f853285e49fc81fe05692b1fdd9e42e3a7/src/main/br/com/agenda/dao/ContatoDao.java#L24

Vi seu post no linkedin e tomei a liberdade de comentar aqui. Gostei que vc deixou uma classe resp. pelas operacoes com o banco. O que acha de refatorar e deixar a conexao com o banco de dados separado? Vamos imaginar que vc tenha outra classe Dao, nao precisaria repetir esse codigo de conexao em cada operacao.

Virgiliopc commented 1 year ago

Boa tarde, nossa gostei muito das sugestões. E é sempre importante receber sugestões de melhoria no código. Agradeço não somente pelas sugestões de código mas pelo tempo utilizado para analisar e apresentar as sugestões de melhoria. Contribuições assim nos ajudam a crescer muito em todos os sentidos. Quando possível vou refatorar esse código seguindo as orientações. Muito obrigado pela nobre contribuição! Grande abraço e conte comigo no que precisar. Att, Virgílio Pires da Costa (44) 99918-6469

Em seg., 29 de mai. de 2023 às 16:12, Lucas Cruz @.***> escreveu:

https://github.com/Virgiliopc/Crud-Java-MySQL-JDBC/blob/7ff196f853285e49fc81fe05692b1fdd9e42e3a7/src/main/br/com/agenda/dao/ContatoDao.java#L24

Vi seu post no linkedin e tomei a liberdade de comentar aqui. Gostei que vc deixou uma classe resp. pelas operacoes com o banco. O que acha de refatorar e deixar a conexao com o banco de dados separado? Vamos imaginar que vc tenha outra classe Dao, nao precisaria repetir esse codigo de conexao em cada operacao.

— Reply to this email directly, view it on GitHub https://github.com/Virgiliopc/Crud-Java-MySQL-JDBC/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/APHWVFTWJ45WAQEV4EDRIKTXITYKFANCNFSM6AAAAAAYTC5NKY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Virgílio Pires da Costa