There is one question that has been bugging me...

[metaclass]

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

1)
string auxcode = auxCodeField.getAsString();
if (!Find(auxcode))
Add(auxcode);

и
2) if(!Find(auxCodeField.getAsString())) then
Add(auxCodeField.getAsString());

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

Comments

[bigfrogg.livejournal.com]

Второй код не то нечитабелен, он ещё
1 - Работает дольше (из-за того что auxCodeField.getAsString() вызывается 2 раза)
2 - Потенциально опасен, из-за того что второй вызов auxCodeField.getAsString() может (в зависимости от алгоритма программы) вернуть уже не то, что вернул первый вызов auxCodeField.getAsString()

[metaclass.livejournal.com]

Про первое вышеозначенные авторы писали, что это "ранняя оптимизация" и ее делать не надо :)
А вот про второе у них ничего не написано, хотя это достаточно самоочевидное шыло.