[нубский вопрос] оптимальность кода

> Подскажите, как оптимальнее, с точки зрения компилятора (в т.ч. для байткода JVM), писать:

Premature optimization is the root of all evil.~--- (c) Donald Knuth.

PS:
говножаба не нужна

premature optimization is the root of all evil (c)

млин, обошёл на повороте! ^^

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

> price.getValue().longValue()

> price.getValue().multiply(usdRubExchangeRate);

блин, это же песец какой-то, товарищи, как вы пишете на таком страшноя языке

а вообще плюсую, не в этом месте надо оптимизировать

Строго говоря и по логике это два разных кода. )
Для компиляторов существует порог оптимизации для конструкций else if
- свыше некоторого числа их (на памяти для BSO было как более 11) выполняются попытки построения логики ветвления через табличные переходы.
Оптмизацию байткода я не ковырял.

Тут только я никакой оптимизации не вижу?

В данном конкретном случае разницы никакой.

        if (currency.equals("RUR")) {
           
            if (price.getCurrency() == BasicCurrency.USD) {
                return price.getValue().multiply(usdRubExchangeRate);
            }
           
            return price.getValue();
        }

жеж . не? оптимизация для читабельности

А зачем price.getValue() вычисляешь помногу раз? По-моему было бы достаточно одного.

В общем, корявый код. И там, и там.

> Premature optimization is the root of all evil.~--- (c) Donald Knuth.

Если повторяешь слова за умными дядями, то нужно знать и другой принцип. Он называется "избежание преждевременной пессимизации". :)

В моём видении в первом случае код более оптимален тем, что имеются независимые блоки условных операторов. Но не знаю, даёт ли это какой-то выигрыш помимо читабельности.

Упрёка в разной функциональности не понял, куски кода абсолютно одинаково работают для _данного_ конкретного случая.

Это не желание что-то оптимизировать, а скорее желание определиться, как писать оптимальнее.

Кому-то не понравился код price.getValue().multiply(usdRubExchangeRate) - каким образом в Вашем языке выполняется работа с большими числами ? Это - BigDecimal.

Ну и на счёт общей корявости кода — буду рад примеру исправления, благо код не большой.

P.S. - price.getValue() ничего не вычисляет, это просто геттер.

Слабо запустить и посмотреть что быстрее работает?

Зачем, когда здесь полно гуру? :)

> Ну и на счёт общей корявости кода — буду рад примеру исправления, благо код не большой.

http://www.linux.org.ru/jump-message.jsp?msgid=4075004&cid=4075523

и Ваш вопрос отваливается

        if (currency.equals("RUR")) 
        {          
            if (price.getCurrency() == BasicCurrency.USD) 
            {
                return price.getValue().multiply(usdRubExchangeRate);
            }
            
            return price.getValue();
        }

Ы?

Никак не привыкну к лоркоду :-(

Да, вобщем, товарищ anTaRes уже успел соптимизировать так же :-)

И правда, упустил как-то. Спасибо. Для себя же решил, что лучше лишний if/else не городить.

price * usdRubExchangeRate, очевидно же

нафига вообще по сто раз getValue непонятно, хоть во временную переменную запомнить, хоть не так по-уродски будет выглядеть