
Artem
18.03.2017
16:29:23
ну в чём какаха? в том же K&R скобки не обязательны, например

Vadim
18.03.2017
16:29:51
У нас тут не K&R, а psr
Как минимум это

Artem
18.03.2017
16:30:44
ну погоди, говорить что что-то говно только потому, что это не соответствует какому-то там "стандарту" без понимания почему оно так - это говно и дебилизм

Google

Artem
18.03.2017
16:31:11
как бы должны быть адекватные причины

Vadim
18.03.2017
16:31:40
Доводилось работать с кодом, где все так, чуваки экономили символы и строки
Поддерживать это нереально

Artem
18.03.2017
16:33:10
по-моему ты преувеличиваешь, я лет 10 писал без {}, потом начал писать с ними просто ради PSR, особой разницы не заметил. Всё это дело привычки

Vadim
18.03.2017
16:33:19
Сам в своем говне можешь сколько угодно ковыряться, в своих поделках на го у меня тоже то еще гавно, но когда ты делаешь что-то, что в будущем будешь трогать не только ты, то стоит заботиться о хоть какой-то гигиене
Когда пишу одноразовые скрипты тоже не сильно парюсь

Artem
18.03.2017
16:34:28
гигиена тут - плохой пример. Стиль кода может быть везде разный, нет какого-то одного идеального метода оформления
есть просто соглашения, типа "а давайте все писать так, потому что ..."
и вот эти "потому что ..." разные могут быть

Roman ?
18.03.2017
16:35:02
Без фигурных скобок в условное выражение не зайдет дебаггер, например
Тоже недостаточно?

Artem
18.03.2017
16:35:43
например, почему "{" при декларации метода должны быть на следующей строке, а после if - на той же?

Vadim
18.03.2017
16:35:47
О как, про это не знал) будет теперь моим железным аргументом) только потестить надо будет

Google

Artem
18.03.2017
16:36:06

Roman ?
18.03.2017
16:36:06

Vadim
18.03.2017
16:36:29

Roman ?
18.03.2017
16:36:31

Artem
18.03.2017
16:36:57
зачем такая неконсистентность?

Vadim
18.03.2017
16:37:22
Потому же, почему большинство стандартов создается
"так договорились"
Это лучше, чем когда не было пср

Artem
18.03.2017
16:38:04
ну вот о чём и речь. Это не значит что такое оформление - лучшее. Это просто то, на чём смогли сойтись

Roman ?
18.03.2017
16:38:31
А тебе надо не быть как все, или ты просто доебаться?
Унифицированность оформления - плюс в любом случае

Artem
18.03.2017
16:39:26
да я сам по PSR пишу, просто не понимаю этого костерения тех, кто пишет не так. Это не настолько большая проблема - понимать код, в котором О УЖАС фигурная скобка ставится на новой строке

Vadim
18.03.2017
16:40:23
Ну, в некотором смысле о ужас, точнее много маленьких ужасиков

Artem
18.03.2017
16:40:27
я понимаю если отступов нет или переменные назаны чёрти как, но такие мелочи как скобки, по-моему, ничего не значат

Roman ?
18.03.2017
16:40:48

Vadim
18.03.2017
16:41:17
В этом вопросе я в любом случае за фашизм
Тотальное единообразие, все дела

Artem
18.03.2017
16:42:18
ну в одном проекте, понятное дело, единообразие - это хорошо
но если какой-то отдельный проект написан не по PSR - это вовсе не значит что он говно и бяка-бяка

Google

Artem
18.03.2017
16:43:27
тем более что форматирование поменять - это дело запуска одного скрипта

Roman ?
18.03.2017
16:44:11
И пуллреквест отправить ?

Vadim
18.03.2017
16:45:00
Ну я наблюдал, что даже шторм + phpcs не всегда могут автомагически гавно конфеткой сделать

Artem
18.03.2017
16:45:44
ну наверняка что-то длинное отформатируется не так как хотелось бы, а в простых случаях вроде справляется
в простых = в большинстве

Vadim
18.03.2017
16:45:53

Alexander
18.03.2017
16:45:57

Artem
18.03.2017
16:46:20
зачем их разбирать, это должно оформляться отдельным коммитом, если такое всё же кто-то решится сделать

Vadim
18.03.2017
16:46:59
Разбирать этот отдельный пулрекаест
Пр на 20 файлов уже очень больно смотреть

Artem
18.03.2017
16:47:19
а что именно там разбирать, если ты знаешь что там только форматирование поменялось

Vadim
18.03.2017
16:47:48
Убедиться, что действительно только оно поменялось

Artem
18.03.2017
16:48:53
а с чего там поменяется что-то ещё, если скрипт переформатирования проверен и работает как надо
но со стороны принимать такие пулл-реквесты я бы принимать не стал, это да

Roman ?
18.03.2017
16:49:08
Омг
Ну вот

Vadim
18.03.2017
16:49:23
Openssl тоже проверен и работает нормально
Да, но нет
Ну, давай теперь я буду говорить, что тесты нужны, а ты, что не нужны)

Artem
18.03.2017
16:50:51
ну тогда и на интерпретатор php нечего надеяться, всё тлен - перестём программировать

Google

Salavat
18.03.2017
16:51:05
Народ, кто-нибудь invisible reCaptcha кто-нибудь использовал?

Vadim
18.03.2017
16:51:15
Во, тут да, тут мы плавно подошли к тестам)

Artem
18.03.2017
16:51:20

Vadim
18.03.2017
16:52:13
Ну ок, тогда так же, только про мутационное тестирование и 100% коверейдж

Artem
18.03.2017
16:52:57
ты применяешь мутационное тестирование?

Vadim
18.03.2017
16:54:10
Так, жопа в опасности, надо срочно менять тему с мутационного тестирования ?

Artem
18.03.2017
16:55:14
тренируй жопу :)
огнезащитные пропитки, вот это всё

Admin
ERROR: S client not available

Vadim
18.03.2017
16:57:11
Короче иф без скобок плохо

Artem
18.03.2017
16:57:41
так а что в итоге, мутируешь код? Есть какие-то стабильные инструменты для этого?
я бы применил, на самом деле, если что-то хорошее для этого есть
нагуглил humbug, но норм ли он?

Vadim
18.03.2017
16:59:35
Мутировать не, все пытаюсь натравить на что-нибудь
Смотрел есть около 2х популярных тулз, humbug среди них
Мне это сейчас больше для го актуально, а там с этим как-то очень плохо

Artem
18.03.2017
17:02:32
а там вообще как с тулингом? Например, покрытие посмотреть, кто дёргает

Vadim
18.03.2017
17:04:12
"go test" в покрытие умеет, насчет кто дергает хз, гогланд умеет

Artem
18.03.2017
17:23:49
чёт humbug я даже запустить не могу :( Всё по ману, но phpunit внутри него не работает - на опции ругается. Версия чтоли не та
запустил.

Google

Artem
18.03.2017
17:38:26
в общем-то даже неплохо сработал. Поймал возвращаемые значения, которые никто не проверяет и ещё по мелочи
ни одного ложно-положительного срабатывания
но проектик мелкий - 1к строк всего, не считая тестов

Vadim
18.03.2017
17:57:04
ну а так а чего там, поменять плюсы на минусы и еще раз запустить тесты грубо говоря
но было что-то поновее, 8 месяцев назад последний коммит
и коммитов много

Artem
18.03.2017
17:59:23
ну вот тут как раз намного сложнее, чем форматирование поменять :) Проблема даже не столько в парсинге, сколько в интеграции с пхпюнитом, например. Чтобы понять что чем покрыто, что запускать, что мутировать. Короче там грабли есть, как мне кажется. Что, собственно, подтверждается тем, что у меня по ману сразу запустить не удалось

Vadim
18.03.2017
17:59:41
:)

Alexander
19.03.2017
07:42:41

Sergey
19.03.2017
11:02:15
А в скобки обернуть?
есть мнение что лучше никогда не оборачивать в скобки дабы не возникало желания больше одного выражения по условию пихать. Ну мол если надо несколько варажений - лучше в приватный метод.

Aleh
19.03.2017
12:13:37

Artem
19.03.2017
12:16:04
в рамках одного проекта - один стиль, это понятно. Кто пишет не в том, который принят на проекте, тот редиска
но вообще расстановка или не расстановка скобок для однострочных блоков кода и в пределах одного проекта особых проблем не создаёт. Плохо становится только если кто-то хочет пользоваться автоформатированием целиком файла
есть такое же мнение, что if вообще не нужен и надо всё полиморфизмом делать

Sergey
19.03.2017
12:49:15
а что до "больших проектов" - это крайне плохая идея для подражаний
ну и есть кучи других способов полностью избавиться от if

Artem
19.03.2017
12:50:08
большие проекты показывают применимость идеи на практике