j-alves-c / trabalho-gcs

Repositório do Trabalho para a Disciplina de Gerência de Configuração de Software 2022-1
1 stars 0 forks source link

Código não nomeado corretamente #2

Open j-alves-c opened 2 years ago

j-alves-c commented 2 years ago

O código fonte do Sistema de Loja de Sapatos não segue inteiramente um padrão de nomeação de variáveis e métodos. Um exemplo, dentre vários, é a nomeação de variáveis na classe EfetuarVenda.java: Trabalho APSOO – ControlaEV java 10_06_2022 01_45_45 Há variáveis seguindo o lowerCamelCase (como campoVendedor), o snake_case (como codigo_sapato) e que não seguem nenhum padrão (como campovaloruni). Em outros pontos do código, há ocorrência de notação húngara e não há diferenciação entre variáveis e constantes. Vamos acordar em um padrão de nomenclatura para termos um código mais limpo?

melonathalia commented 2 years ago

Eu acho que utilizar o snake_case vai deixar o código mais agradável de se ver.

guilhermectprado commented 2 years ago

O snake_case eu acharia mais útil para variáveis da parte de interface gráfica, e um outro padrão para demais variáveis. No código daria para perceber onde tem parte da interface gráfica inserida e poder utilizar sem criar confusão.

j-alves-c commented 2 years ago

Considerando que nosso código está em segue a arquitetura MVC, somada aos padrões de projeto GRASP e Dao, temos grande separação entre partes da interface gráfica com outras partes do código: apenas nossa controladora interage, de forma relativamente eficiente, com a View. Não vejo como a interface gráfica criaria confusão e precisaria estar nomeada de forma diferente.

j-alves-c commented 2 years ago

Além disso, penso que deveríamos adotar as convenções da linguagem Java ( a que usamos para codificar o nosso Sistema). Seguem links que discorrem sobre os padrões de nomenclatura convencionais de Java:

https://www.devmedia.com.br/convencoes-de-codigo-java/23871 https://micreiros.com/padroes-de-codificacao-no-desenvolvimento-de-sistemas/

melonathalia commented 2 years ago

Vendo o exemplo do devmedia o modelo a se seguir deveria ser um lowerCamelCase, sempre fico confusa quando utilizam ele, mas se forem escolher ele por mim tudo bem.

guilhermectprado commented 2 years ago

Realmente, erro meu ao confundir as coisas. Após ser auxiliado na review do código é visto que a controladora usa as variáveis pelo método get, ela não sabe os nomes das variavéis. No mais então vou concordar no padrão camelCase. Gostei do DevMedia que tratou dos comentários e da forma das condições além de mostrar padrões de variáveis e melhores formas de escrever com uma linha muito comprida, e do Micreiros ele fornece uma padronização boa também que já foi mostrada também no DevMedia.

giovanavalli commented 2 years ago

A observação de padronizar os nomes das variáveis e métodos foi muito boa e concordo! Confesso que não sabia que cada linguagem tem o seu padrão de nomeação.

leticiayurie commented 2 years ago

Muito válido a padronização de códigos! Além de ficar mais organizado, ele também facilita no entendimento do código. Gosto da ideia do snake_case, pra ficar evidente a separação das palavras. No entanto, o site que a @j-alves-c mostrou sobre "Padronizações de Codificação no Desenvolvimento de Sistemas" mostrou que o estilo camelCase é bem universal para quem programa em Java. Podemos adotar as convenções de nomes, classes, interfaces, métodos, etc.