Когда использовать префикс get в методах класса?

Question

[Wentixon]

Недавно поймал себя на мысли, что слишком часто использую префикс get в своих методах по сравнению с крутыми разработчиками opensource.. Проанализировал много популярных фрагментов кода и стало немного непонятно, на основе чего это волшебное слово get включается в название. Даже когда метод представляет из себя банальный геттер, слово get все же очень часто опускается.

Cart, Collection - Вот два примера, где используется такой подход. Вообщем напишите, когда по вашему мнению опускание get является более предпочтительно и почему. Несколько фреймворков в тегах указал специально, так как знаю, что культура наименования у всех разная и хотелось бы услышать мнение каждого.

Лично мое мнение по этому поводу. Если метод обладает минимальной логикой, то есть просто отдает свойство объекта, возможно с каким то незначительным форматированием, то логичнее указать в названии, что это "чистый" геттер, добавив в название get. Если же метод содержит логику какой то фильтрации, выборки или сложные вычисления, то слово get опускается. Также если метод принимает параметры, то он уже не должен считаться геттером.

Comments

[JhaoDa]

Тэгов мало поставил.

[Wentixon]

JhaoDa, Больше не ставится

[JhaoDa]

Wentixon, и ещё детектор сарказма у тебя сломался.

[Wentixon]

JhaoDa, Что такое сарказм? Я про него ничего не слышал

[JhaoDa]

Wentixon, тяжко тебе будет... ну хотя бы правила Тостера почитай, особенно п. 3.1.

[Wentixon]

JhaoDa, Во-первых, не строй из себя умника ладно? Ты попадаешь под пункт 5.16, если что со своими глупыми комментариями. Во-вторых, я описал, почему я поставил эти теги, а ты видимо даже не читал вопроса. Мне интересно мнение спецов из разных фреймворков, так как стандартны именования у разработчиков симфони допустим более строгие и мнение у них будет другое по этому вопросу. В-третьих, пусть модератор решает насколько корректны тут теги, а не ты и в случае чего принимает действия.

тяжко тебе будет...

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

[JhaoDa]

Wentixon, но лишние тэги лучше убрать.

[Wentixon]

JhaoDa, убрал тег "программирование", спасибо за совет

[JhaoDa]

Wentixon, на зло маме уши отморожу, да? :)

[Lander]

JhaoDa, Чем вы опять не довольны? longclaps новый аккаунт завёл?

[JhaoDa]

Lander, ну, вроде бы, из переписки видно — я недоволен лишними тэгами. До тов. longclaps мне ещё далеко.

[Lander]

JhaoDa, Человек удалил лишний тэг и объяснил почему оставил тэги с названиями основных фреймворков. У вас подгорает потому что упомянули ларавель всуе, что ли?

[JhaoDa]

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

[JhaoDa]

Lander, кстати, аккаунт тов. longclaps вполне себе активен.

[Lander]

JhaoDa,

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

И это только ваше мнение.

longclaps вполне себе активен

И это, конечно же, автоматически запрещает ему иметь другой?

Answer 1

[Lander]

Название метода - должно быть понятным! Это единственная рекомендация. Вообще любое название в программировании (переменная, функция, класс, метод...) должно быть понятным! Классически считается что название функции должно представлять глагол. В такой стезе названия вроде make, collapse, check и т.д. вполне оправданы. При условии, что понятно что именно надо make, collapse и check. :)

Comments

[Wentixon]

С понятным названием все понятно :)

Классически считается что название функции должно представлять глагол

Как мы видим, в тех примерах, которые я привел как раз это правило не выполняется. Метод first, all в коллекции это не глагол, tax в корзине тоже не глагол. И это нарушение кажется почему то вполне привлекательным и удобным, так как getFirst, getAll (getItems), getTax кажется слишком исчерпывающим и лишним. Может это глупое правило, которому я тоже привык следовать, приписывая всюду get, чтобы метод превратился в глагол?

[Lander]

Wentixon, Ну в данном случае это хоть и метод, но названия first и all вполне понятны, и выглядят логично. Концептуально это вполне могли быть и свойства типа "Первый элемент" и "Все элементы".

upd. например название функции sum и так понятно что возвращает сумму, не обязательно писать getSum.

[Wentixon]

Lander,

$user->firstName();
$user->email();

Ну это всегда вроде понятно.. Однако геттеры имеют этот префикс. То есть всегда по сути когда имя метода - существительное, то и так ясно, что оно возвращает данные, а не обрабатывает их. И префикс get всегда и добавляется к существительным. getPrice, getTax, getFirstItem..

Интересно просто, когда они пишут код, как они мыслят. Такое ощущение, что они кидают монетку и решают использовать в этом классе или методе префикс или нет.