Не всегда же называть методы глаголами?

Question

[Максим Федоров]

Пишу тест на PHP и в нем придумываю, как мне нужно строить объект, пример:

<?php
$myObj = new MyClass;
$my = $myObj->build(SomeClass $obj, AnotherClass $obj2)
            ->event(new OtherEvent)
            ->property(new Property)
            ->date(date());

Понял, что тут некоторые методы будут называться существительным по названию свойства, то есть существительным
верна такая практика?

UPD: Именно вопрос и состоит в том, что тут же логично убрать set или в названии вспомогательных методов, так как по сути они олицетворяют свойство, которое мы задаем

Пример из фреймворка Yii2 на языке PHP:


тут функции all() и alias()

существительные

Comments

[Алексей]

верна такая практика?

Почему бы и нет. Если быть перфекционистом, то надо бы было типа getProperty и setProperty, но по-моему это не всегда удобно. Во многих проектах (библиотеках) имена методов совпадают с именами свойств и это считается нормой.

[Максим Федоров]

Алексей, я вот тоже часто натыкаюсь на это и мне очень логичным синтаксисически кажется это

[amorphine]

Посмотрите на любую ORM, там практически все существительные. Потому что настолько глубокие абстракции.

Answer 1

[Максим Федоров]

судя по всему у Вас методы устанавливают значения соответствующих свойств.
Для таких методов принято формировать названия с префиксом set, например: setEvent (установить событие), setDate(установить дату)

Что касается приведенного фрагмента: это базовый класс Yii фреймворка, а сам по себе код этого фреймворка желает лучшего, с точки зрения соответствия различным практикам программирования. Давайте забудем что это за фреймворк, забудем все что описано в документации по нему, и посмотрим на этот код просто с точки зрения ООП, глазами человека не привязанного к какому-то конкретному инструменту. Например, возьмем метод alias(). Что мы можем сказать о этом методе? Мы можем сказать что в этом методе что-то делается с псевдонимом. Но что конкретно делает этот метод? за что он отвечает? Исходя из его названия - мы ничего о этом сказать не можем, т.к. такое название неочевидно.

Comments

[Anton Mashletov]

ну надо же. я уж подумал, что ты сам себе отвечаешь :D

[Максим Федоров]

Anton Mashletov, такие мысли не у тебя одного возникают)))

[Максим Федоров]

Максим Федоров,
обновил вопрос и дал пример из Yii2, мне кажется вполне логичным не приписывать к вспомогательным методам get

[Максим Федоров]

Максим Федоров, обновил ответ

Answer 2

[GavriKos]

Называйте их SetDate и GetDate в зависимости от того, что они делают.

Answer 3

[Станислав Макаров]

Когда вы пользуетесь fluent interface - вы всегда создаёте маленький язык внутри другого языка. Поэтому нет ничего странного в том, что вам хочется вспомогательные конструкции называть по иным правилам.

Answer 4

[Дмитрий Башинский]

Лучше первый метод назвать что-то типо Create - создать, а в конце уже место Build - построить.
Методам лучше сделать приставку Set - потому как они устанавливают значения

Answer 5

[Anton Mashletov]

Глагол всегда должен быть, я считаю.
1) глагол + существительное: getDate, setName
2) просто глагол — update, build, create, make, get

Просто существительные можно использовать в атрибутах, а-ля ActiveRecord.
$x->name = 'asd'; Но никак не в виде метода.

Answer 6

[Александр]

Мой вариант.

Либо так:

interface Entity {

    /**
     * Property setter
     *
     * @param string $name Set property
     *
     * @return Entity (return $this;)
     */
    public function setName(string $name): Entity;

    /**
     * Property getter
     *
     * @param null|string $default Default value of property
     *
     * @return string Property value
     */
    public function getName(?string $default = null): string;
}

Либо так:

interface Entity {

    /**
     * Property setter and getter in one method
     *
     * @param null|string $name Set property value or get on $name = null
     *
     * @return Entity|string
     */
    public function name(?string $name = null);
}

Причем для 2-ого варианта желательно реализовать еще и магические методы __set и __get