![]() |
Модераторы: LSD, AntonSaburov |
![]() ![]() ![]() |
|
Romper |
|
|||
Новичок Профиль Группа: Участник Сообщений: 35 Регистрация: 7.11.2012 Репутация: нет Всего: 1 |
Привет.
Какое ваше имхо, всегда писать только "is"(пример: isDataCopied) или для множество надо писать "are"(пример: areAllTasksDone) ? Cпасибо. |
|||
|
||||
Arantir |
|
|||
Рыбак без удочки ![]() ![]() Профиль Группа: Участник Сообщений: 960 Регистрация: 18.11.2012 Репутация: 1 Всего: 55 |
Думаю, если бы твой родной язык был английским, то ты бы не спрашивал =)
В целом методы должны называться так, чтобы их можно было запомнить и использовать по памяти, что, несомненно, делает дальнейшую разработку более эффективной. С этой точки зрения при формировании названия метода из памяти, человек, часто и много использующий английский, попытается поставить are для множественного и is для единственного числа. Грамотность — это способ людей правильно понимать друг друга. Думаю и на программирование это распространяется в некоторой степени. -------------------- interface Жопа { // ATTENTION: has to be implemented by every class of the project for proper project work } |
|||
|
||||
Stolzen |
|
|||
![]() Эксперт ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 1041 Регистрация: 17.10.2005 Репутация: 23 Всего: 48 |
Пишите так, как правильно с точки зрения грамматики английского языка
Это сообщение отредактировал(а) Stolzen - 31.8.2013, 18:51 |
|||
|
||||
Samotnik |
|
|||
![]() Super star ! ![]() ![]() ![]() ![]() Профиль Группа: Awaiting Authorisation Сообщений: 7192 Регистрация: 4.11.2006 Где: Минск City Репутация: 8 Всего: 191 |
Romper, кодконвеншен это не регламентирует, значит нужно пользоваться здравым смыслом
![]() |
|||
|
||||
LSD |
|
|||
![]() Leprechaun Software Developer ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 15718 Регистрация: 24.3.2004 Где: Dublin Репутация: 210 Всего: 538 |
Я за следование code conventions. Если мне понадобится узнать boolean свойство я буду искать свойства начинающиеся на isXXX() но не areXXX().
-------------------- Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it. |
|||
|
||||
priam220 |
|
|||
![]() Опытный ![]() ![]() Профиль Группа: Участник Сообщений: 291 Регистрация: 4.6.2010 Репутация: 7 Всего: 8 |
лучше попытаться объединить конвеншн+здравый смысл - попытаться переименовать метод, в Вашем случае, допустим - isAllDone.
А подробности написать в javadoc. |
|||
|
||||
Stolzen |
|
|||
![]() Эксперт ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 1041 Регистрация: 17.10.2005 Репутация: 23 Всего: 48 |
||||
|
||||
Arantir |
|
||||
Рыбак без удочки ![]() ![]() Профиль Группа: Участник Сообщений: 960 Регистрация: 18.11.2012 Репутация: 1 Всего: 55 |
Искать свойство среди методов очень оригинально =) Какая-то инверсия получилась. Я действительно то ли не встречал подобного, то ли это было достаточно редко, то ли все пихали is для множеств. Дело в том, что обговариваемые тут методы класса начинаются с "is" именно потому, что, обычно, относятся к одному конкретному екземпляру этого класса. И характеризуют они один объект. isReady(), isAdmin(), isBanned(), isRed(), isBlue(), isYellowButerfly(), isMagicYellowPurplePony()... Вероятно это и имеется ввиду в code conventions. А areAllTasksDone() — это больше похоже на какую-то независимую функцю. В моем представлении все должно было бы заблаговременно быть как-то так:
Хотя, понятное дело, раз на раз не приходится и это только пример, к котрому не каждый случай сводится. Но тем не менее... -------------------- interface Жопа { // ATTENTION: has to be implemented by every class of the project for proper project work } |
||||
|
|||||
Samotnik |
|
|||
![]() Super star ! ![]() ![]() ![]() ![]() Профиль Группа: Awaiting Authorisation Сообщений: 7192 Регистрация: 4.11.2006 Где: Минск City Репутация: 8 Всего: 191 |
если речь о getter/setter, то согласен. Но метод лучше называть осмысленно. Например: werePhotosUploaded() Кстати, мы один и тот же codeconventions читали? Там нет инфы в данном контексте про то как называть boolean методы |
|||
|
||||
priam220 |
|
|||
![]() Опытный ![]() ![]() Профиль Группа: Участник Сообщений: 291 Регистрация: 4.6.2010 Репутация: 7 Всего: 8 |
Samotnik, я тоже что то не нашел на оф сайте такой инфы. Но вот тут есть.
Кроме того в бинах если не ошибаюсь имена полей должны совпадать с именами геттеров. И вариантов там может просто не быть. Кстати придумал еще одно название для такого поля: hasRunningTasks ![]() |
|||
|
||||
Samotnik |
|
|||
![]() Super star ! ![]() ![]() ![]() ![]() Профиль Группа: Awaiting Authorisation Сообщений: 7192 Регистрация: 4.11.2006 Где: Минск City Репутация: 8 Всего: 191 |
priam220,
![]() Как-то университет Вашингтона слабоватый авторитет в Java для меня. |
|||
|
||||
LSD |
|
|||
![]() Leprechaun Software Developer ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 15718 Регистрация: 24.3.2004 Где: Dublin Репутация: 210 Всего: 538 |
Не знаю кто как, но лично я пользуюсь автокомплитом в IDE. Если метод возвращает некое значение которое харатеризует состояние объекта и при этом само сотояние не меняется, то это вполне можно считать Java Bean property. Понятно что есть особые случаи типа того же итератора, но в данном конкретном случае я не увидил причин писать are, was, did и т.п.
Имена свойств, но не полей, поля приватные и могут называться как угодно. Например List.isEmpty() вполне себе boolean read only property с именем empty, при этом поля там вообще нет. -------------------- Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it. |
|||
|
||||
Arantir |
|
|||
Рыбак без удочки ![]() ![]() Профиль Группа: Участник Сообщений: 960 Регистрация: 18.11.2012 Репутация: 1 Всего: 55 |
А давайте просто считать наличие надобности в префиксах are, was и т.д. плохой архитектурой приложения
![]() -------------------- interface Жопа { // ATTENTION: has to be implemented by every class of the project for proper project work } |
|||
|
||||
Samotnik |
|
|||
![]() Super star ! ![]() ![]() ![]() ![]() Профиль Группа: Awaiting Authorisation Сообщений: 7192 Регистрация: 4.11.2006 Где: Минск City Репутация: 8 Всего: 191 |
LSD, я не про данный случай, а вообще. Если говорить о чистом code convention, то там скорее речь идет про адекватность названий, а не про то что boolean всегда должно быть is.
Вот например как в данном случае поступить?
Методы без is потому что это враппер булевский. Но и называть переменную isReal тоже неправильно по code convention. |
|||
|
||||
Magistrus |
|
|||
![]() Жив ![]() Профиль Группа: Участник Сообщений: 129 Регистрация: 14.6.2006 Где: г. Одесса Репутация: нет Всего: 1 |
частенько для определение наличия множества используется "has" например hasTextured
Это сообщение отредактировал(а) Magistrus - 4.9.2013, 19:10 --------------------
~ вот такая вот загагулина ~ |
|||
|
||||
![]() ![]() ![]() |
Правила форума "Java" | |
|
Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic. |
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей) | |
0 Пользователей: | |
« Предыдущая тема | Java: Общие вопросы | Следующая тема » |
|
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности Powered by Invision Power Board(R) 1.3 © 2003 IPS, Inc. |