metaclass: (Default)
metaclass ([personal profile] metaclass) wrote2009-04-13 03:07 pm

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

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

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

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

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

[identity profile] g-rub.livejournal.com 2009-04-13 12:18 pm (UTC)(link)
А вопросы именования функций для того, чтобы код был чисто на глаз читабельным -- это уже другое дело

[identity profile] familom.livejournal.com 2009-04-13 12:38 pm (UTC)(link)
Ну, каждое имя функи в отдельности может быть читабельным, а композиция уже нет.