leandro-bsf / Comex

0 stars 0 forks source link

Seguir padrão de codificação #2

Open cacio-costa opened 2 years ago

cacio-costa commented 2 years ago

Leandrão, uma coisa que a gente precisa ter muito carinho é em seguir o padrão de codificação do Java. Isso traz clareza no código, e facilita a comunicação entre desenvolvedores.

Então, lembrar que variáveis e métodos sempre devem começar com letras minúsculas. Porque não confunde com classes. Na linha 56, parece que está invocando um método estático na classe ProdV.

https://github.com/leandro-bsf/Comex/blob/28d7bc339330a58b11df1e3b7c8042d6554ef160/comex/src/br/com/comex/csv/LeitorPedidosCsv.java#L56

Outra coisa é evitar abreviações. Elas deixam o código confuso. Se um desenvolvedor novo chegar na equipe e olhar a classe PedMaxMin, ele vai ficar imaginando o que pode ser. ;)

https://github.com/leandro-bsf/Comex/blob/28d7bc339330a58b11df1e3b7c8042d6554ef160/comex/src/br/com/comex/csv/LeitorPedidosCsv.java#L53

leandro-bsf commented 2 years ago

Entendi Cacio. Muito Obrigado pelas dicas. vou seguir.

Em ter., 18 de out. de 2022 às 11:52, Cácio Costa @.***> escreveu:

Leandrão, uma coisa que a gente precisa ter muito carinho é em seguir o padrão de codificação do Java. Isso traz clareza no código, e facilita a comunicação entre desenvolvedores.

Então, lembrar que variáveis e métodos sempre devem começar com letras minúsculas. Porque não confunde com classes. Na linha 56, parece que está invocando um método estático na classe ProdV.

https://github.com/leandro-bsf/Comex/blob/28d7bc339330a58b11df1e3b7c8042d6554ef160/comex/src/br/com/comex/csv/LeitorPedidosCsv.java#L56

Outra coisa é evitar abreviações. Elas deixam o código confuso. Se um desenvolvedor novo chegar na equipe e olhar a classe PedMaxMin, ele vai ficar imaginando o que pode ser. ;)

https://github.com/leandro-bsf/Comex/blob/28d7bc339330a58b11df1e3b7c8042d6554ef160/comex/src/br/com/comex/csv/LeitorPedidosCsv.java#L56

— Reply to this email directly, view it on GitHub https://github.com/leandro-bsf/Comex/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV6FKMYZKIQQDAJG2CHCEP3WD22SLANCNFSM6AAAAAARIFTCAM . You are receiving this because you are subscribed to this thread.Message ID: @.***>