Пример кода


Правильно ли так делать циклы в жаве? Смущает то, что если MyClass::getSize() вернёт 100000, то вызов функции culcSize() будет выполнен 100000 раз. Если тело функции из одной строчки и она не инлайновая, 

то как минимум будет выполнено 100000 входов в функцию getSize() и 100000 выходов. А если гэтСайз - это целый код по вычислению размера, то ещё и этот код будет выполнятся 100000 раз.
Было бы правельнее сделать так

Я тонкастями жавы не владею..... Может компилятор жавы сам сделать из 1-го варианта 2-ой и не будет 100000 вызовов?

Я тоже,

Зачем возлагать на компилятор то, что должен делать программист?  
Второй образец кода неверный, примеры правильного кода можно посмотреть в папке с jdk из  файла src.zip

 А что не верно? Может синтаксис жавы неверен.... я писал c++_like. Если не сложно, дайте в форум правильную версию кода? 

ps хотя на этом же форуме есть пример кода

Чем мой второй пример кода отличается от этого? 

Это сообщение отредактировал(а) juvf - 6.11.2013, 09:29

Так не сделаешь. Как правильно смотри в пособиях, учебниках и т.п.

Все зависит от того, что может быть в цикле.
Например, если размер MyClass может изменяться в цикле, то корректным будет только вариант

А вот если Вы ТОЧНО ЗНАЕТЕ, что размер меняться не будет, то можно использовать такой вариант

 Компилятор не способен решить какой из вариантов выбрать, так что решать придется Вам   

2Mirkes
конечно если getSize() в течении цыкла меняет возвращяемое значение - тут вероятнее ошибка программирования. речь не об этом.

допустим вы знаете что возвращяемое значение ТОЧНО не изменится. getSize - это обычный геттер



size равен 100000. на первый взгляд здесь, по крайней мере в с++, видно сто тысяч входов в функцию getSize, и 100 тысяч выходов. Это же на лицо перегрузка процессора ненужными пустыми операциями.  Допустим вход в функцию 16 тактов, выход - ещё 16. как минимум 3 200 000 тактов теряется в таком коде.  А  getSize возвращяет 1 000 000 000 и потоков 100,  И каждый поток выполняет вычисления....
А если getSize не геттер... а он подсчитывает размер таблицы/людей/машин/молекул/чеготоЕщё, или запрашивает кол-во у какойнить базы данных.... да ещё по TCP.... 

Поспрашивал "живых" жава-программистов - автор ответил что так все пишут, другие толком ни чего не могут сказать. Даже если это простой геттер, не знают - будет ли 100000 входов в функцию getSize(). Меня удивило, что они об этом даже не задумываются. Пишут, потому что так все пишут. Интересно, как жава работает.

Первый и второй вариант не эквивалентны. Поэтому компилятор этого делать не будет.

Добавлено через 1 минуту и 16 секунд

Если "вы" точно знаете, то и проведите соответсвующую оптимизацию. Компилятору неоткуда взять эту информацию.

juvf,
цикл в форме 
в общем случае эффективней чем

как раз вынесением вычисления size() из цикла
но вообще java компилятор оптимизирует. как именно, зависит от компилятора.
то, чем вы озабочены, называется преждевременной оптимизацией. делайте код более простым и понятным, оставьте остальное компилятору до того времени, пока производительность достаточна

Почему же ошибка программирования, бывают разные задачи, иногда в ходе перебора увеличивается размер перебираемой коллекции. Это нормально. Именно поэтому компилятор не может знать меняется значение или нет.

baldina 
Первый работает через итератор, а итератор это те же самые 2 вызова метода на каждую итерацию. Вместо size()/get() будет hasNext()/next().

уфф... Если логика метода getSize не очень сложная, то скорее пофигу с точки зрения производительности. В жабе просадку большУю даёт создание объектов и последующая сборка мусора(особенно внезапная, когда не надо), к примеру...
Если логика getSize хитрая(особенно если объекты порождаются), то таки да, будет просадка.

Канонично класс унаследовать от какой-либо реализации(?) Collection и итерировать по ней (или просто одно из полей сделать коллекцией объектов и дать доступ к ней через геттер - это тоже самое, только через композицию)...

Понятнее было бы из контекста задачи, возможно тебе совсем не стоит самому свой контейнер с 0 избретать... Обычно используют готовые коллекции, под конкретные задачи.

А оптимизатор жабы достаточно хитрожоп) http://www.ibm.com/developerworks/ru/library/j-jtp12214/
Как точно будет код работать, нельзя угадать...

Это сообщение отредактировал(а) Farmazon - 12.11.2013, 05:07