Я не очень внимательно слежу за судом Оракла и Гугля, но, насколько я понимаю, на сегодня Гугль
уличили в копировании куска кода из Оракла, а именно - функции rangeCheck(). Вот как эта функция
выглядит сейчас:
private static void rangeCheck(int arrayLen, int fromIndex, int toIndex
(
Read more... )
Comments 19
Оракл еще бы пожаловался на то, что переменная цикла называется 'i'.
Reply
http://news.ycombinator.com/item?id=3940761
И вроде он действительно его скопипейстил, хотя нет никакого сомнения, что ровно никакого IP этом коде быть не может.
Reply
Reply
Условия местами можно переставить
Reply
Reply
Reply
Раз private - не трожь! :)
Вон судья к public методам претензий не имел.
Reply
Reply
Reply
Может быть вместо if (toIndex > arrayLen) лучше проверять if (toIndex >= arrayLen)
т.к. первый элемент массива это 0, а не 1?
Reply
for (... ; i < toIndex; ...)
При таком раскладе ограничение на допустимые значения toIndex-а такое: toIndex <= arrayLen, и, соответственно ошибка, если toIndex > arrayLen.
Reply
for (... ; i <= toIndex; ...), т.к. лень держать в голове, что индекс from включается в рассмотрение, а toIndex - нет. Не люблю "выколотые точки", но это уже дело вкуса :).
Reply
Reply
Reply
Reply
Reply
Leave a comment