Skip to content

Desenvolvedor Estágio. - #30

Closed
leooresende01 wants to merge 13 commits into
bempaggo:mainfrom
leooresende01:main
Closed

Desenvolvedor Estágio.#30
leooresende01 wants to merge 13 commits into
bempaggo:mainfrom
leooresende01:main

Conversation

@leooresende01

Copy link
Copy Markdown

No description provided.

&& pMovimento.getDeslocamentoHorizontal() == semMovimento.getDeslocamentoHorizontal();
}

public static Boolean posicaoEhInvalida(PosicaoPeca pMovimento) {

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.

Oi,
Obrigado por tentar resolver o desafio.

Já recebemos muitos pull request de candidatos com falta de disciplina em relação à POO.

Vc pode conferir um aqui #21

Atenciosamente

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.

Olá, muito obrigado por promover esse desafio, foi bem interressante ter tentado resolver ele, acho que valeu bastante a pena o tempo gasto.

Sobre a forma da comparação de objetos usando "==", a principio eu estava usando tipos primitivos, por conta de no android, ocorrer varios problemas quando se trata um wrapper como um tipo primitivo, ai pelo costume de programar pra essa plataforma eu acabei descuidando e usando alguns tipos primitivos.

Porém quando eu percebi eu já tinha feito o pull request, ai já era tarde, mas no ultimo commit eu ainda fiz a mudança pros wrappers, só que em alguns locais (como esse), eu deixei escapar.

Mas uma vez, muito obrigado pelo desafio e pelo feedback.

@leooresende01 leooresende01 closed this by deleting the head repository Jul 5, 2023
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