pronskiy Jul 31 2017 at 00:45

PHP-Дайджест № 113 – свежие новости, материалы и инструменты (16 – 30 июля 2017)

3 min

19K

Zfort Group corporate blogWebsite development*PHP*

+58

Comments 40

gogolinsky Jul 31 2017 at 09:20

Спасибо!

daggert Jul 31 2017 at 09:21

> RFC: Mixed typehint — Предлагается добавить mixed typehint:

Вот я не понимаю почему нельзя было добавить данный тип с самого начала, при внедрении типизации? Либо я один такой дурак и не мог грамотно реализовать паттерн repository без mixed?

franzose Jul 31 2017 at 09:33

А зачем в репозитории mixed? Репозиторий либо конкретную сущность возвращает, либо набор, т.е. array.

UFO just landed and posted this here

gro Jul 31 2017 at 11:57

Ничего не указано — как вариант просто забыл указать или поленился.
mixed — явно указано, что здесь mixed.

edogs Jul 31 2017 at 13:17

/sarcasm/
предлагаем в следующей версии вместо пустой строки писать «this is empty line, nothing here»
что бы точно быть уверенным в том, что здесь должна быть именно пустая строка, а не что-то важное, что ты просто забыл написать или поленился
/sarcasm/

SamDark Aug 1 2017 at 01:20

Хорошая практика именно так писать в пустых try-catch.

UFO just landed and posted this here

Zhuravljov Aug 1 2017 at 22:52

По разному может быть, и зависит от контекста.

mariczzz Aug 2 2017 at 10:56

О, Сэм )

По поводу очередей на Yii2 — вроде, была инфа, что они пока не особо юзабельны, мол, пользуйтесь другими (сторонними) расширениям. Сейчас ситуация изменилась?

SamDark Aug 2 2017 at 15:30

Да.

SerafimArts Aug 2 2017 at 17:10

offtop

<irony>
Мир перевернулся в тот самый момент, когда SamDark решил поговорить о хороших практиках. Осталось услышать доклад от разрабов битрикса о каких-нибудь SOLID, SRP и прочих страшных штуках, и можно считать, что в жизни видел всё +))))
</irony>

SamDark Aug 2 2017 at 18:08

Я о них всегда говорил и мнение особо не поменялось. Просто раньше я говорил очень неосторожно и меня можно было понять как «не стоит их изучать и применять», хотя на самом деле я имел ввиду то, что думать надо, а не фанатично применять не разобравшись.

samizdam Jul 31 2017 at 10:52

Спорное улучшение. Как и nullable, поскольку ослабляет типизацию.

franzose Jul 31 2017 at 11:24

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

gro Jul 31 2017 at 12:03

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

samizdam Jul 31 2017 at 14:47

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

И другое дело принять правило, но либо с оговоркой: не используем mixed, либо оставить лазейку лентяям.

Я за первый вариант без оговорок и двусмысленностей, как более строгий. И поэтому против mixed.

daggert Jul 31 2017 at 14:59

Категорично против mixed нельзя быть. Ну вот как например быть с реестром, про который я начал беседу? Вот запрашиваете вы registry->get($var) — и вот лично у меня фреймворк сейчас может вернуть bool, string или null. Как тут без mixed обойтись?

gro Jul 31 2017 at 17:37

Ваше мнение по этому поводу, несомненно, имеет право на существование.
Но оно не имеет отношения к тому, что здесь обсуждается.

daggert Jul 31 2017 at 11:05

Ой господа, простите, не repository я имел ввиду а registry. С утра голова не заметила подвоха.

Gemorroj Jul 31 2017 at 10:08

Лучше бы реализовали как в phpdoc сейчас пишут — варианты типов данных: string[]|string, например.
Это позволило бы сильно снизить вообще необходимость такого типа как mixed.

+11

Fr05t1k Jul 31 2017 at 10:16

Попахивает. Зачем в одном методе возвращать массив или строку?

Gemorroj Jul 31 2017 at 10:31

https://github.com/symfony/symfony/search?utf8=%E2%9C%93&q=%22%40param+string[]|string%22&type=

gro Jul 31 2017 at 12:08

Если вводить типизированные массивы, придётся каждый раз в рантайме обходить весь массив и проверять каждый элемент.

franzose Jul 31 2017 at 12:13

А сейчас это и так приходится делать вручную через foreach или какой-нибудь Webmozart\Assert.

gro Jul 31 2017 at 12:53

Но зачем?

franzose Jul 31 2017 at 12:59

Как зачем? Чтобы не прилетало что попало. Целостность данных, все дела.

gro Jul 31 2017 at 13:27

Это вы вообще везде этим занимаетесь?

franzose Jul 31 2017 at 13:46

Только если массив должен быть наполнен объектами пользовательского типа данных.

UFO just landed and posted this here

franzose Aug 7 2017 at 23:35

Как ArrayAccess спасет от доступа к несуществующему индексу и от того, что нужно знать, какие индексы существуют?

UFO just landed and posted this here

Fr05t1k Jul 31 2017 at 10:19

Лучше бы static typehint добавили.

SerafimArts Jul 31 2017 at 10:55

static — это рантайм, а вся типизация резолвится статически, как в константах и аргументах методов (там, например, нет static, но есть self и в т.ч. доступны примитивные операции). Именно по-этому тайпхинтинг не появится никогда =)