@laravel_pro

Страница 452 из 2014
Golodnyi
05.05.2017
08:27:49
php_codeSniffer захотел внедрить, прогнал, там 10к ошибок, поставил IDE в psr2, авторефакторингом прошелся, осталось 500 претензий, уже лучше, но вручную один фиг целый день работы -_-

Golodnyi
05.05.2017
08:32:01
да я php_codeSniffer в шторм подключил

Юрий
05.05.2017
08:32:20
На относительно чистом проекте на yii 2к "хотелок" нашлось, на старом легаси находило около 5к, с плагином 28к стало вчера

Google
Golodnyi
05.05.2017
08:32:32
в том и дело что я визуально вижу все проблемы в шторме, + еще инспектор запускаю с проверкой только по сниферу и он выводит только его претензии, игнорируя другии, чтоб не мешались пока

Юрий
05.05.2017
08:32:46
Что нравится - он советует заменять ряд функций на более производительные

Golodnyi
05.05.2017
08:33:19
это как?

Юрий
05.05.2017
08:33:27
https://plugins.jetbrains.com/plugin/7622-php-inspections-ea-extended-

Golodnyi
05.05.2017
08:33:38
встроенные пхпшные? Или твои переписать предлагает?)

вообще конечно производительность в пхп это малоебущий фактор)

кэш сделает все за меня)

Юрий
05.05.2017
08:34:31
Встроенные. Типа is_null замени на простое сравнение с null и тп. Очень много всего. Ну и замена алиасов встроенных функций пхп на оригиналы и тп

вообще конечно производительность в пхп это малоебущий фактор)
Малоебущий до момента как время генерации страницы за 1-2 секунды перевалит даже с кешем

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

Golodnyi
05.05.2017
08:35:31
это эпично, если is_null работает медленнее проверки на null, там же внутри полюбому просто проверка? Из за факта вызова функции что ли медленнее?

Юрий
05.05.2017
08:35:54
Судя по комментарию - да, типа лишний оверхед

Golodnyi
05.05.2017
08:36:01
Малоебущий до момента как время генерации страницы за 1-2 секунды перевалит даже с кешем
с кешем больше 50мс не будет, но и в целом вообще больше 1сек, эт довольно странный код, мб там бд тюнить надо?

Google
Golodnyi
05.05.2017
08:36:54
такие штуки как замена is_null на проверку на null, на столько нано оптимизация, что выигрывать эту одну тысячную долю секунды нахрен не нужно) зря личное время потратишь

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

Юрий
05.05.2017
08:37:35
Да меня просто доставило что он автоматом может пробежаться, потом только бегло взглянуть что тесты не отвалились и код не побился)

Хотя тесты далеко не у всех есть)

Golodnyi
05.05.2017
08:38:32
тесты проблема..да

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

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

Юрий
05.05.2017
08:39:31
А кто нибудь работает уже с указанием что стрикт тайпы (фишка с 7)? Сильно много косяков вылезло?

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

Golodnyi
05.05.2017
08:39:59
ты про типы? У нас php 7.1 в новом проекте

Юрий
05.05.2017
08:40:31
declare(strict_types=1)

Я про это

Golodnyi
05.05.2017
08:41:02
я хз что оно делает... запрещает динамическое преобразование из строки в число?

Юрий
05.05.2017
08:41:15
Да, вроде того)

Golodnyi
05.05.2017
08:42:05
наверно оно в пхп 7.1 по дефолту выключено? Я не помню есть ли у меня попытки что то преобразовать динамически, скорее всего нет, но функции преобразования я точно не юзал

Юрий
05.05.2017
08:42:16
Обещали включить в 8 только

7.0/1/2 на переход

Типа проверьте что строго сравниваете

Golodnyi
05.05.2017
08:42:47
компилятор в си код не хотят сделать?)

Google
Юрий
05.05.2017
08:42:56
Потому что уже (или шторм или плагин) матерятся на ==

Типа или строгое или вали из языка

Aleserche
05.05.2017
08:43:35
штормом правда удобно пользоваться?

Алексей
05.05.2017
08:43:42
да

Golodnyi
05.05.2017
08:43:44
если не выключать, то да

Юрий
05.05.2017
08:43:58
компилятор в си код не хотят сделать?)
Я вчера rfc почитал... чего только не хотят сделать. И да, куча всего с сей.

Golodnyi
05.05.2017
08:44:28
да в нем с рождения много из сей, просто щя интерпретируемый язык, а вдруг в 8-ой версии он станет компилируемым)

F01134H
05.05.2017
08:44:34
Или каких модулей надо
все еще хочу бандлы как в симфони

Юрий
05.05.2017
08:45:04
Очень доставило рфц где предлагают переименовать все функции работы со строками и тп в string_*

И сделать типа алиасами

Пропихнут ли в 8 хз, но там список из 200 вроде функций

Golodnyi
05.05.2017
08:45:30
это кому такому яблоко на голову упало?

Golodnyi
05.05.2017
08:45:35
или в каком сне он это увидел?)

Юрий
05.05.2017
08:45:55
это кому такому яблоко на голову упало?
Тому кому надоело видеть stipos, substr и тп

Golodnyi
05.05.2017
08:45:59
стремное предложение на мой вкус

Юрий
05.05.2017
08:46:05
Когда название не всегда то чем кажется

Golodnyi
05.05.2017
08:46:15
ну да, видеть string_substr наверно лучше) кек

а чем есму substr кажется интересно? Это же в большинстве языков именно так

да и расшифровывается как "подстрока"

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

Google
Юрий
05.05.2017
08:49:14
Ничего сексистского, но баба в стадии постоянного пмс видимо отклоняет все рфц которые видит

33 за - одна она против

И так почти по всем пунктам

Aleserche
05.05.2017
08:50:35
а нигде не было предложений унифицировать имена функций и порядок аргументов?

Юрий
05.05.2017
08:50:46
Куча людей которые у себя не то что 12, не то что 2, а даже 1 не соблюдают

а нигде не было предложений унифицировать имена функций и порядок аргументов?
Ну вон я выше отписал. По именам есть рфц переименовать около 200 функций

Чтобы было через андескор все

А не как сейчас слитно

Admin
ERROR: S client not available

Golodnyi
05.05.2017
08:52:08
вообще никогда не испытывал какого то дисананса при использовании пхп-ых функций

что по названиям, что по параметрам

Юрий
05.05.2017
08:52:23
Спасибо шторму и тп

Aleserche
05.05.2017
08:52:26
но без префиксов, указывающих на тип?

Юрий
05.05.2017
08:52:48
но без префиксов, указывающих на тип?
Строковые к строковым, работу с файлами к file_* и тп

Aleserche
05.05.2017
08:53:05
бле, вот это плохо

Юрий
05.05.2017
08:53:07
Сложно сейчас найти нужный, я вчера все смотрел, но там почти по всем блокам

Я крайне сомневаюсь что в 8 вытолкают, но как алиасы - могут

Golodnyi
05.05.2017
08:53:44
ничоси у вас проблемы)

Google
Aleserche
05.05.2017
08:54:15
поделись своими, может общие найдутся

Golodnyi
05.05.2017
08:54:39
да я выше уже сказал, что проблем с названиями никогда не испытывал

Юрий
05.05.2017
08:54:47
Меня больше волнует что пхп катится к стриктам

Golodnyi
05.05.2017
08:54:52
если не знаю какую то функцию, то гуглю по описанию и гугл на php.net сразу выдает

Юрий
05.05.2017
08:55:00
Принятый почти - иммутабл классы и свойства

Golodnyi
05.05.2017
08:55:04
не, стрикты этож хорошо, меньше ошибок, все дела

Юрий
05.05.2017
08:55:20
Компейребл тоже почти принят

Golodnyi
05.05.2017
08:55:27
иммутабл классы...не видел такого в пхп

Юрий
05.05.2017
08:55:28
Типа в дополнение к итеребл

Golodnyi
05.05.2017
08:55:41
этож функциональная тема ?

Юрий
05.05.2017
08:56:08
Ну это с нового не включённого еще

F01134H
05.05.2017
08:56:22
кто-нибудь laravel boilerplate юзал?

Golodnyi
05.05.2017
08:56:25
дак пхп не очень подойдет же под иммутбл принцип

на сколько я знаю, иммутбл это не изменение объекта, если надо изменить, склонируй объект и измени его, не?

Golodnyi
05.05.2017
08:57:28
это не иммутбл...

это защита так сказать

ниразу не практиковал ее

Aleserche
05.05.2017
08:58:03
а как же final class ...?
закрывает класс для наследования

Юрий
05.05.2017
08:58:29
https://wiki.php.net/rfc/immutability

Вот рфц эта

F01134H
05.05.2017
08:58:35
ага

в 7.2 будет

Страница 452 из 2014