@prophp7

Страница 189 из 1387
Timur
13.01.2017
19:03:24
Я ещё @Symfony добавляю, но тут у всех разные фреймворки.

У Symfony там свои особенности есть.

Виталий
13.01.2017
19:05:55
Он везде юзается. И в Symfony, и в Laravel.
эм.. а чего код на пхп? Я с консольки хотел его дернуть для папки с проектом и все.

Расскажи плиз как именно ты его используешь?

Google
Timur
13.01.2017
19:09:32
https://github.com/FriendsOfPHP/PHP-CS-Fixer Этот конфиг кладёшь в корень проекта под именем ".php_cs.dist". Потом ставишь Composer'ом и пускаешь так: $ ./vendor/bin/php-cs-fixer fix Глобально ставить его не люблю. У себя ставлю локально.

В мануалах там описано всё.

В .gitignore добавляешь .php_cs и .php_cs.cache файлы.

Можешь глобально поставить. Тут каждый сам решает.

F01134H
13.01.2017
19:10:42
получается он и на проде крутится чоле?

Timur
13.01.2017
19:10:58
Зачем? Нет. Локально запустил, он всё и пофиксил.

В require-dev его надо.

Виталий
13.01.2017
19:11:45
так а для чего есть в проект подключать? Немного не понимаю. Это ведь больше инструмент разработчика и его к шторму надо подключать.

Timur
13.01.2017
19:12:05
У каждого проекта свой стиль кодирования.

У меня так по-крайней мере в команде.

У Symfony 3.* проекта свой, у Yii 2 проекта свой, у древнего Yii 1.1 проекта свой.

Поэтому у нас per project.

Виталий
13.01.2017
19:13:15
угу, понял. И у вас есть пхп файлик в корне проекта запуская который вы все фиксите, так?

Google
Timur
13.01.2017
19:15:49
Ну, да, так. PHP файл этот просто настройки для PHP CS Fixer хранит.

https://github.com/symfony/symfony/blob/master/.php_cs.dist

https://github.com/laravel/framework/blob/5.3/.php_cs

В Laravel набор правил вообще свой.

Виталий
13.01.2017
19:17:55
да, крутяк. Спасибо! Очень помог:)

чет пропустил я эти фишки с phpcs

Коллеги, забыл как в PHP Storm хоткей для интерактивного переимнования переменной везде где она юзается. Ставишь курсор на переменную, жмакаешь хоткей и она выделяется рамочкой иможно печатать и когда меняешь название она автоматом меняется везде по коду, интрактивно.

Перерыл все доки, не могу найти.

F01134H
13.01.2017
20:25:40
вроде бы через ctrl+f решается

dypa
13.01.2017
20:25:48
code -> refactor

F01134H
13.01.2017
20:25:59
ну лан :D

dypa
13.01.2017
20:26:07
у всех хоткеи разные

Виталий
13.01.2017
20:26:30
ну вот как так! Только написал вам и сразу налось решение. Два вечера искал хоткей.

Спасибо!

Виталий
13.01.2017
20:27:42
shift + f6

а я f7 жмакал

dypa
13.01.2017
20:27:51
pdf'ку с хоткеями можно сделать через help -> keymap reference

F01134H
13.01.2017
20:28:07
Google
dypa
13.01.2017
20:28:08
ctrl+shift+r переиновать в папке
у всех хоткеи разные!!!

Максим
13.01.2017
20:28:22
shift + f6
нет такого. ось другая, наверное

Виталий
13.01.2017
20:28:35
пальцы привыкли. и в голове комбинация не запомнилась, а потом дого не кодил и пальзы забыли и кароче трабла ппц (

mac os

dypa
13.01.2017
20:28:57
Виталий
13.01.2017
20:29:00
я через ПКМ на переменную > Refactor

а я блин наугад два вечера подбирал https://yadi.sk/d/7BxSSBt838rCJR

разве по пср так правильно? <?php foreach ($authClientCollection as $client) : ?> а так нет? <?php foreach ($authClientCollection as $client): ?>

Виталий
13.01.2017
20:42:48
в пробеле между ")" и двоеточнием

Fayozjon [CybernatiC]
13.01.2017
20:42:55
?

Виталий
13.01.2017
20:43:31
ну вот мне phpcs подсвечивает, грит мол ожидается 1 пробел после скобки

Timur
13.01.2017
20:53:17
PHP CS Fixer ещё можно встроить в Continious Integration процесс.

При сборке и тестировании пускай выявляет косяки в стиле кодирования.

https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/472#issuecomment-53514112

Вот тут они обсуждали это.

И ниже.

Короче просто забили. Там походу все юзают обычный синтаксис или Twig.

У себя в Yii2 проекте я просто исключил шаблоны из анализа и правки. ?

Google
Виталий
13.01.2017
21:01:05
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/472#issuecomment-53514112
да, спс. Надо найти как исключить и мне

Коллеги, а как использовать файл .php_cs, не могу нагуглить примеров. Кому его скармливать? или он просто запускается как php .php_cs

или даже он просто должен лежать в папке проекта, а потом php-cs-fixer сам его подхватит

F01134H
13.01.2017
21:56:41
Народ, а вы юзаете public при создании метода? Это в PSR описано где-нибудь? Ну т.е. метод по умолчанию public, но можно жеж дописать при желании. Интересует - нужно ли оно

F01134H
13.01.2017
21:58:52
а по русски

Admin
ERROR: S client not available

F01134H
13.01.2017
21:58:56
не все ж в США живут

da horsie
13.01.2017
21:59:10
по-русски это к 1С-никам

F01134H
13.01.2017
21:59:17
((

da horsie
13.01.2017
21:59:28
область видимости должна быть прописана

у методов и у свойств

обязательно

F01134H
13.01.2017
21:59:43
оо

спасибо

da horsie
13.01.2017
22:00:30
и именно в таком порядке (abstract|final) (public|protected|private) (static)

Роман
13.01.2017
22:01:55
/yazaban

?

Прочитал ЭТО...

Я тоже пишу скобочку где надо (на той же строке). Будем вместе гореть.

Google
da horsie
13.01.2017
22:03:01
/yazaban

че это такое

F01134H
13.01.2017
22:06:40
ого себе

Роман
13.01.2017
22:08:19
Команда должна была быть может. Но нету. ?

http://pastebin.ru/gwjpSY1J

С самого начала писал так, все остальные стили не понимаю. ? Особенно когда скобка на следующей строке.

А ещё отвратительный стиль когда между арифметическими знаками не ставят пробелы. Вот за это я бы наверное в яму на сутки сажал. ?

Вокруг точнее, а не между.

Dmitriy
13.01.2017
22:13:44
http://pastebin.ru/gwjpSY1J
этот бы код у нас зареджектил CI

Роман
13.01.2017
22:13:59
?

Dmitriy
13.01.2017
22:14:21
CI проверяет на соотвествие стандарту

Роман
13.01.2017
22:14:23
Да я сам по себе. Мелкие заказы на фрилансе. В команде не работал никогда.

Виталий
13.01.2017
22:14:35
зачем называть файл .php_cs.dist ?

Роман
13.01.2017
22:15:01
CI проверяет на соотвествие стандарту
А что там не так? Что исправить пришлось бы?

Dmitriy
13.01.2017
22:15:32
зачем называть файл .php_cs.dist ?
что бы правила от туда взялись

Виталий
13.01.2017
22:15:43
dist = distribution (распространяемый)

Страница 189 из 1387