и по возможностям не уступает, только я доку не запилил

<?php class GeoCoords { /** * @var float */ private $latitude; /** * @var float */ private $longitude; /** * GeoCoords constructor. */ public function __construct() { } public static function createFromCoordinates($latitude, $longitude) { $geoCoords = new static(); $geoCoords->latitude = $latitude; $geoCoords->longitude = $longitude; return $geoCoords; } }

еще код в GeoCoords есть ?

isEqual какой-нибудь и мб distanceFrom

от домена зависит)

здесь только один именнованный конструктор

иии?

http://pastebin.com/ngxBxRQy

чем этот код хуже?

18 строка там не нужна

вот более корректный вариант (в первый раз я просто скопипастил) http://pastebin.com/tvyD0PvR

еще код в GeoCoords есть ?

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

при этом не нарушая инкапусляцию

опять же мое имхо, я стараюсь по возможности делать immutable

блять

при чем тут immutable?)

при том что после создания чтобы нельзя было поменять

по возможности

это мое имхо я не отрицаю что именнованные конструкторы могут пригодится

и могут использоваться

просто по возможности не стоит давать доступ к ним из вне

если они нужны тебе и твоему коду то юзаешь

но если ты делаешь GeoCordinatInterface то их сюда лучше не засовывать

без необходимости

если необходимо добавляй и старайся поддерживать их где возможно

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

потому что если их переделают больно будет твоему коду

но это тоже имхо

например если автор говорит юзай их я буду юзать

например вот http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/configuration.html

но если это внутри кода библиотека и не документированна или внутри кода чужого класса, я подумаю, а смогу ли я подобное сделать через те инструменты что рекомендует автор (в доке или interface) если да то предпочту их наверно

или может я как то не так думаю?

<?php final class GeoCoords { /** * @var float */ private $latitude; /** * @var float */ private $longitude; /** * GeoCoords constructor. * @param $latitude * @param $longitude */ public function __construct($latitude, $longitude) { $this->latitude = $latitude; $this->longitude = $longitude; } /** * @param string $text * @return GeoCoords */ public static function createFromString(string $text): GeoCoords { $arr = explode(":", $text); return new static($arr[0], $arr[1]); } /** * @param AddressInterface $address * @return GeoCoords */ public static function createFromAddress(AddressInterface $address): GeoCoords { return new static($address->getLat(), $address->getLon()); } /** * @param GeoCoords $geoCoords * @return float */ public function distanceTo(GeoCoords $geoCoords) { //impl } /** * @return string */ public function __toString() { return join(":", [$this->latitude, $this->longitude]); } }

вот более реальный пример

это мое имхо я не отрицаю что именнованные конструкторы могут пригодится

конструкторы.... в интерфейсах? Ну или "статические методы в интерфейсах"? Серьезно?

здесь только один именнованный конструктор

какая разница? Именованные конструкторы делают код выразительнее

$this->remember(UserRegisteredEvent::appear());

вместо

$this->remember(new UserRegisteredEvent());

прикольнее пожалуй так

$this->rememberThat(UserRegisteredEvent::appear($this));

$time = Time::fromValues($hours, $minutes); $time = Time::fromString($time); $time = Time::fromMinutesSinceMidnight($minutesSinceMidnight);

Time:toDie()

вместо new Time(Time::TIME_TO_DIE)

User::register

вместо new User

User::createAccount()

попроще пример)

ну зависит

от того че как

Order::createWithUser($user)

Category::new()

)))

new?)

ну а че нет?)

new Category

писать меньше)

а понятность такая же)

Category()

ну да) я просто думал набросить

нельзя ж инвокабл класс запилить?

неа..

?

это должен быть статически инвокабл

нужно больше статики статическому богу

веруешь ты в статического бога или в бога динамического?

нужны мета классы

и тогда можно на статику делать интерфейсы

бугага

хз... функции... модули...

и хватит?

хотя без объектов начнут монадами обмазываться

манки патчинг нужен

без него никак

хы

<?php final class GeoCoords { /** * @var float */ private $latitude; /** * @var float */ private $longitude; /** * GeoCoords constructor. * @param $latitude * @param $longitude */ public function __construct($latitude, $longitude) { $this->latitude = $latitude; $this->longitude = $longitude; } /** * @param string $text * @return GeoCoords */ public static function createFromString(string $text): GeoCoords { $arr = explode(":", $text); return new static($arr[0], $arr[1]); } /** * @param AddressInterface $address * @return GeoCoords */ public static function createFromAddress(AddressInterface $address): GeoCoords { return new static($address->getLat(), $address->getLon()); } /** * @param GeoCoords $geoCoords * @return float */ public function distanceTo(GeoCoords $geoCoords) { //impl } /** * @return string */ public function __toString() { return join(":", [$this->latitude, $this->longitude]); } }

вот createFromString пример того что особо не надо

почему в месте где понадобились координаты не сделать преобразования к получению координат

и не создать координаты через конструктор ?

тоже самое и с createFromAddress

а если таких мест 2..3..10?

это попытка в классе предугадать в классе, как его будут использовать

из за метода createFromAddress у тебя лишняя зависимость от AddressInterface

которой можно было бы вполне избежать

в текущем коде, я понимаю что в реальности завимость от AddressInterface возможно есть или ее использование

но именно в примере это лишняя зависимость

это попытка в классе предугадать в классе, как его будут использовать

вообще то ты всегда должен "предугадывать" или хотя бы "закладывать" способ использования твоих штукенций пряча нюансы

это попытка в классе предугадать в классе, как его будут использовать

что за AddressInterface?

откуда у тебя интерфейс взялся

геокордината это обычный объект же

который хранит координаты

любой! публичный метод или свойство составляют интерфейс объекта

и интерфейс должен быть удобным и простым

определение дистанции между 2 координатами надо стараться сделать в отдельном объекте

явно описывая что и зачем

а не как

определение дистанции между 2 координатами надо стараться сделать в отдельном объекте

не обязательно

ну в примере у тебя у координат определение дистанции

ну в примере у тебя у координат определение дистанции

есть такая штука - информационный эксперт. Данные должны обрабатываться там где они есть.