Skip to content

Branch - Eduardo Yucho Yamaguti [Programador Plêno] - #45

Open
EduardoYucho wants to merge 2 commits into
bempaggo:mainfrom
EduardoYucho:branch_1
Open

Branch - Eduardo Yucho Yamaguti [Programador Plêno]#45
EduardoYucho wants to merge 2 commits into
bempaggo:mainfrom
EduardoYucho:branch_1

Conversation

@EduardoYucho

Copy link
Copy Markdown

Melhorias na Abstração e POO Pura:
Refatorei as classes para seguir princípios de POO de forma mais pura.
Renomeei classes para representar os elementos de forma mais clara.

Redução de Código:
Simplifiquei a lógica de busca de adjacências e remoção de células vazias.
Limpei o código de partes desnecessárias e redundantes.

Experiência do Usuário:
Corrigi erros de lógica na exibição de mensagens.
Simplifiquei a lógica de atualização do tabuleiro e verificação de jogo completo.
Corrigi um erro que ocorria quando o espaço vazio era clicado.

Espero que essas mudanças tornem o código mais robusto, legível e alinhado com os princípios de programação orientada a objetos.

Alteração na formatação do código para deixa-lo padronizado com igualmente.
Código refatorado para aprimorar as orientações a objetos. Classes e métodos alterados para seguirem a mesma linha de programação, e, deixa-lo mais coerente com o solicitado.

@EduardoYucho EduardoYucho left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

this.emptyCell = cell;
});
Optional<GameVertex> minCell = this.cells.stream()
.min(Comparator.comparing(GameVertex::getValue));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa Tarde,
Objetos devem trocar mensagens.
O objetivo é uso de POO, o uso de programação funcional atrapalha na coesão.

Att.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants