Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

[Proposal] Use unboxed boolean types in ResourceHandlerRequest

Aberta
#341 3 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
35/100
Tipo de issue
Refatoração
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
java
Domínio
tooling

Direção de pesquisa

Comece localizando ResourceHandlerRequest e revisando os dois commits vinculados que introduziram os campos Boolean boxed. Verifique como snapshotRequested e rollback são usados e, em seguida, determine se os booleanos primitivos são apropriados e se os testes existentes cobrem verificações de condições null ou booleanas. A tarefa estará concluída quando a justificativa estiver resolvida e os tipos de flag escolhidos funcionarem com segurança para os callers.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

enhancement good first issue

There are 2 recent changes that landed in ResourceHandlerRequest: https://github.com/aws-cloudformation/cloudformation-cli-java-plugin/commit/ebca1dc12e187b484ed599db65d726a6b23a14c0#diff-adfb207dac8dff5107b68b36280ccad4525f92176184ad2ffff9aa7e591b2e8a and https://github.com/aws-cloudformation/cloudformation-cli-java-plugin/commit/e34f0472d0bc67953f41d4ceb666a175d0a7fb2d#diff-adfb207dac8dff5107b68b36280ccad4525f92176184ad2ffff9aa7e591b2e8a.

Both changes favor boxed Boolean over primitive types.

As a result, the usage of this class implies on importing BooleanUtils or a similar helper in order to avoid constructions like: if (request.getSnapshotRequested() != null && request.getSnapshotRequested() == true) { ... }.

And the code that is performing a "naive" boolean comparison like: if (request.getSnapshotRequested()) { ... } will fail with a null-pointer exception.

The same stands for the rollback flag.

I wonder if var type boxing was done for some specific reasons? If there are no strict reasons, I would vote for unboxing them to simplify condition checking.

Linguagem predominante
Java
Estrelas
30
Forks
48
Nenhum PR com merge em 30d

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de aws-cloudformation/cloudformation-cli-java-plugin

Todas as issues de aws-cloudformation/cloudformation-cli-java-plugin

Issues semelhantes

Mais issues de Java

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.