Prosba o code review

0

Czesc eksperci Javy!

Czy moge prosic was o krotki code review moich pierwszych dwoch algorytmow napisanych w Javie? :)
Kazdy feedback, ktory pozwoli mi pisac lepszy kod jest bardzo mile widziany jako, ze dopiero raczkuje :)

Instrukcja dot. algorytmow: https://github.com/OskarKaminski/algorithms/blob/627c8a1fbd05340437cbd62b57cfff2f788c43cc/java/divide_and_conquer_problems_20161023.pdf
Rozwiazalem pierwsze dwa i dzialaja poprawnie z zestawem wszystkich testow ale kod to platanina if'ow z for'ami.

Zalozylem pull requesta. Prosze o komentowanie :)
https://github.com/OskarKaminski/algorithms/pull/1/files

0

Decline, No tests were found.

1
(int) Math.floor(l + ((r - l) / 2));

to to samo co

l + ((r - l) / 2)
0

Jest tam mozliwe dodawanie komentarzy przez wszystkich? Bo nie mam jak sprawdzic. W razie czego poprawie zaraz.

1

Wyszukiwanie binarne, czemu bierze dodatkowe dwa elementy, "l" i "r"? Z punktu widzenia designu to nie ma sensu, jak ktoś chce wyszukać to podaje tablicę i element.
Majority jest Θ(nlogn), OK, Θ(nlogn) - sortowanie + Θ(n) pętla, ale nie jest to divide and conquer, o co chodziło autorom zadania. Poza tym nie Masz żadnych komentarzy ani docs strings, chociaż krótkie info by się przydało, jakich typów argumenty biorą funkcje i co zwracają.

1 użytkowników online, w tym zalogowanych: 0, gości: 1