@prophp7

Страница 294 из 1387
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
Roman ?
18.03.2017
16:36:31
что за дебаггер такой?
Ну xdebug + idea. Не заходит.

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
я понимаю если отступов нет или переменные назаны чёрти как, но такие мелочи как скобки, по-моему, ничего не значат

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
Во, тут да, тут мы плавно подошли к тестам)

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
:)

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

Aleh
19.03.2017
12:13:37
да я сам по PSR пишу, просто не понимаю этого костерения тех, кто пишет не так. Это не настолько большая проблема - понимать код, в котором О УЖАС фигурная скобка ставится на новой строке
при внесении изменений и автоформате будут лишние изменения, писать в рамках проекта в одном стиле это добро, сам стиль должен быть наименьшим злом из возможных

Artem
19.03.2017
12:16:04
в рамках одного проекта - один стиль, это понятно. Кто пишет не в том, который принят на проекте, тот редиска

но вообще расстановка или не расстановка скобок для однострочных блоков кода и в пределах одного проекта особых проблем не создаёт. Плохо становится только если кто-то хочет пользоваться автоформатированием целиком файла

есть такое же мнение, что if вообще не нужен и надо всё полиморфизмом делать

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

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

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

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