Skip to content
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

Adiciona implementação para o RadixSort #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

winiciusallan
Copy link

Código com a implementação do RadixSort.

}
}

private static int getDigit(int N, int i) {
Copy link
Owner

Choose a reason for hiding this comment

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

n

}

private static int getDigit(int N, int i) {
return (int) (N / Math.pow(10, i - 1)) % 10;
Copy link
Owner

Choose a reason for hiding this comment

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

n. usar padrão de java para nomear variáveis.


public class RadixSort {

public static void radixSort(int[] v) {
Copy link
Owner

Choose a reason for hiding this comment

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

vamos fazer esses métodos pertencer à abstração RadixSort. Tira os static de todos eles.


private static int getMax(int[] v) {
// Considerando que não vamos receber número inteiros < 0
int max = -1;
Copy link
Owner

Choose a reason for hiding this comment

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

melhor começar considerando o maior o primeiro elemento do array e fazer o for de 1 até o final.

Copy link
Owner

@joaoarthurbm joaoarthurbm left a comment

Choose a reason for hiding this comment

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

Comentários no código. Cadê os testes? :)

@joaoarthurbm
Copy link
Owner

-> In god we probably trust. All others must bring tests.

@winiciusallan
Copy link
Author

Comentários no código. Cadê os testes? :)

Devo fazer os testes em um main mesmo usando asserts ou crio uma classe de testes com Junit? Caso seja usando o Junit, devo criar a estrutura de diretório reservadas pros testes, certo?

@joaoarthurbm
Copy link
Owner

Comentários no código. Cadê os testes? :)

Devo fazer os testes em um main mesmo usando asserts ou crio uma classe de testes com Junit? Caso seja usando o Junit, devo criar a estrutura de diretório reservadas pros testes, certo?

Isso!

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