интеллектуальная собственность

May 16, 2012 00:12

Я не очень внимательно слежу за судом Оракла и Гугля, но, насколько я понимаю, на сегодня Гугль уличили в копировании куска кода из Оракла, а именно - функции rangeCheck(). Вот как эта функция выглядит сейчас:

private static void rangeCheck(int arrayLen, int fromIndex, int toIndex ( Read more... )

США, копирайт, google, профессия

Leave a comment

e_levy May 16 2012, 09:00:19 UTC
Тривиальный, конечно, но и тут можно, имхо, ошибку сделать:
Может быть вместо if (toIndex > arrayLen) лучше проверять if (toIndex >= arrayLen)
т.к. первый элемент массива это 0, а не 1?

Reply

mopexod May 16 2012, 09:23:43 UTC
Обычно всё же toIndex используют как
for (... ; i < toIndex; ...)
При таком раскладе ограничение на допустимые значения toIndex-а такое: toIndex <= arrayLen, и, соответственно ошибка, если toIndex > arrayLen.

Reply

e_levy May 16 2012, 09:29:36 UTC
Ok. Тогда верно. Хотя я часто сильно не задумываясь использую именно
for (... ; i <= toIndex; ...), т.к. лень держать в голове, что индекс from включается в рассмотрение, а toIndex - нет. Не люблю "выколотые точки", но это уже дело вкуса :).

Reply

mopexod May 16 2012, 09:52:32 UTC
Я, пожалуй, даже и не знаю никого, кто бы делал i <= toIndex;

Reply

e_levy May 16 2012, 09:59:57 UTC
Значит это моя личная заморочка :).

Reply


Leave a comment

Up