Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Adicionar implementação do algoritmo Ternary Search #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mateushsx merged 2 commits into mateushsx:master from Marllon-Wendel2:master
Oct 8, 2024
Merged

Adicionar implementação do algoritmo Ternary Search #16

mateushsx merged 2 commits into mateushsx:master from Marllon-Wendel2:master
Oct 8, 2024

Conversation

Copy link
Contributor

@Marllon-Wendel2 Marllon-Wendel2 commented Oct 7, 2024 •
edited
Loading

Descrição

Este pull request adiciona a implementação do algoritmo Busca Ternária (Ternary Search) na categoria de algoritmos de busca (search). O algoritmo foi implementado em JavaScript, seguindo as diretrizes de contribuição, e a documentação foi escrita em português, conforme solicitado.

Mudanças principais:

-Implementação da Busca Ternária em index.js.
-Adicionada documentação do algoritmo em README.md com a
explicação de como o algoritmo funciona, sua complexidade
temporal, e exemplos de uso.
-Incluído um teste básico para validar o algoritmo com diferentes
entradas.

Mudanças principais:

-algorithms/search/ternary-search/index.js
-algorithms/search/ternary-search/README.md

Como testar

1 - Execute o algoritmo com os seguintes dados de exemplo:

const array = [1, 2, 3, 4, 5, 6, 7, 8, 9];
const searchValue = 8;
const result = ternarySearch(array,searchValue);
console.log(result !== -1 ? `Elemento encontrado no índice ${result}` : "Elemento não encontrado.");

2 - Verifique se o algoritmo encontra corretamente o valor em diferentes arrays de números.

  • Segui as diretrizes de contribuição descritas em CONTRIBUTING.md.
  • O código foi escrito em inglês.
  • A documentação foi escrita em português.
  • Testei minha implementação localmente para verificar a funcionalidade.

Copy link
Owner

@mateushsx mateushsx left a comment

Choose a reason for hiding this comment

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

Olá @Marllon-Wendel2, tudo tranquilo? 👋🏾

Primeiramente, gostaria de te agradecer pela contribuição, que é de grande ajuda para a comunidade! 🙏🏾

A documentação ficou bem clara e explicativa, e achei muito boa a forma como você dividiu a responsabilidade de cada função, isso deixa o código mais organizado e fácil de entender. Parabéns por essa abordagem! 🎉

Agora, tenho algumas sugestões para pequenos ajustes:

  • Corrigir o nome das funções: searchInFirstArry, searchInSecundtArry, searchIntThirdArry. Acredito que a intenção era usar Array no final, certo?
  • Remover os comentários no código: Como a documentação já explica bem o funcionamento, acho que podemos dispensar os comentários para manter o padrão de limpeza do código.
  • Padronizar o idioma: Seria interessante corrigir as variáveis e funções em português, como resultado e buscaTernaria, para inglês, deixando tudo mais uniforme.

No mais, está excelente! Se precisar de algo, estou à disposição. E mais uma vez, muito obrigado pela contribuição! 🙌🏾

OBS: Não se esqueça de ajustar o exemplo de código na documentação também. 😊

Copy link
Contributor Author

Alteirei conforme comentou, sim eu queria ter escrito array nas funções, e padronizei para inglês.

Copy link
Owner

@mateushsx mateushsx left a comment

Choose a reason for hiding this comment

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

Obrigado pela sua contribuição @Marllon-Wendel2 ! 🎉

Seu PR foi revisado e aceito com sucesso. Agradeço muito o esforço e o tempo que você dedicou para melhorar o projeto.

Contribuições como a sua ajudam a tornar este repositório melhor para todos, e estou ansioso para ver futuras colaborações.

Qualquer coisa estou a disposição😉

@mateushsx mateushsx merged commit d694d3b into mateushsx:master Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mateushsx mateushsx mateushsx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /