Prosba o code review

Prosba o code review
Oskar Kamiński
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 2
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

panryz
  • Rejestracja: dni
  • Ostatnio: dni
0

Decline, No tests were found.

Jaca777
  • Rejestracja: dni
  • Ostatnio: dni
1
Kopiuj
(int) Math.floor(l + ((r - l) / 2));

to to samo co

Kopiuj
l + ((r - l) / 2)
Oskar Kamiński
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 2
0

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

lion137
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 5025
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ą.

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.