@prophp7

Страница 423 из 1387
Like
26.06.2017
16:36:13
Кто как относится к if ($a) return 2; Вместо if ($a) { return 2; } Ну еще можно и так if ($a) { return $a; }

Maks
26.06.2017
16:43:42
плохо The body of each structure MUST be enclosed by braces. This standardizes how the structures look, and reduces the likelihood of introducing errors as new lines get added to the body.

Vladislav
26.06.2017
16:45:09
и вообще глаза сломаешь когда делают foreach ($ar as $k => $v) if($v == ) { ... }

Sergey
26.06.2017
16:47:19
Кто как относится к if ($a) return 2; Вместо if ($a) { return 2; } Ну еще можно и так if ($a) { return $a; }
положительно, будет заставлять большие выражения выносить в приватные методы

Google
Sergey
26.06.2017
16:47:32
ну или если приходится работать с джунами

Like
26.06.2017
16:51:01
Меня просто смущает PSR Он же вроде "заставляет" if ($a) { // }

Maks
26.06.2017
16:51:41
ну хз. есть стандарт, ему либо следуешь (и молодец), либо нет. Но тогда всей команде надо обосновать чем твой вариант лучше. Тут уже и на ревью потом не скинешь ссылку на PSR, а надо кидать ссылку на какой-то локальный style guide, мол вот у нас так принято, а не как в стандарте короче шаг влево от PSR - и дополнительные трудозатраты

Vladislav
26.06.2017
16:52:19
Ошибку получишь
какую интересно

Like
26.06.2017
16:52:32
какую интересно
Посмотри на свой код

foreach ($ar as $k => $v) if($v == ) { ... }

Vladislav
26.06.2017
16:52:47
блин ну понятно что три точки это я просто так)

Like
26.06.2017
16:52:50
Я не о том

Условие сработает что ли?

Google
Like
26.06.2017
16:53:17
Может if($v == ) должно быть внутри foreach 'a?

Ибо это полный зашквар (если сработает)

А стоп понял, пардон)

Vladislav
26.06.2017
16:53:45
php <?php $ar = [1, 1, 1, 1]; foreach ($ar as $k => $v) if($v == 1) { echo $v; } 1111

усе работает

Like
26.06.2017
16:53:50
Скобочки для условия

Vladislav
26.06.2017
16:54:01
просто не надо так

Like
26.06.2017
16:54:05
Ну такого дерьма точно писать не буду)

Фиг поймешь с первого раза даже

Я говорю про такие случаи: if ($a) return;

Maks
26.06.2017
16:54:35
Ну, если у тебя после условие тупо return; или что-то мелкое, почему бы и не писать без {}
потому что есть стандарт) делайте внутри своей команды свои стандарты, минусы выше. главное что б единообразно было и вас (всех членов команды) устраивало

Pavel
26.06.2017
16:55:13
Это не стандарт, а соглашение

Maks
26.06.2017
16:55:41
ну тогда пишите как хотите, в чем вопрос то )

Like
26.06.2017
16:55:58
Maks я дополнил )

Pavel
26.06.2017
16:56:07
Просто зачем писать по-другому

договорились PSR

Like
26.06.2017
16:56:36
Просто зачем писать по-другому
if ($a) { return; } Почему бы не написать if ($a) return;

Maks
26.06.2017
16:56:39
я увидел ) еще потенциальный минус я вижу в автоматических линтерах кода. 99% они настроены по PSR, отойдете от него - надо будет править конфиги линтера. что опять-таки трудозататы

Google
Pavel
26.06.2017
16:57:47
Вариант ПСР более читабельный. Скобки отлично выделяют блок

Maks
26.06.2017
16:58:00
?

Like
26.06.2017
16:58:27
Более читабельный? Хм, странно Я не вижу проблем с читаемостью, для понимания вроде бы самое то Проблема в том, что вы привыкли к этим {} и без них вам плохо) И это факт

Pavel
26.06.2017
16:59:27
» Скобки отлично выделяют блок

Like
26.06.2017
16:59:38
Зачем выделять return ?

Там выражение какое-то? (даже $a + $b будет вполне понятно)

Проблема в том, что ты привык к этим скобочкам и без выделения блока тебе "сложно"

Pavel
26.06.2017
17:02:03
Зачем выделять return ?
неважно что внутри if. Меняется поток выполнения

Like
26.06.2017
17:02:20
Что? Не понял

Pavel
26.06.2017
17:03:24
если а = true программа продолжит идти по другой ветке

Like
26.06.2017
17:04:04
Что он хочет сказать?

Скобки синтакс. Сахар же, не?

dypa
26.06.2017
17:06:05
if ($a) { return; } Почему бы не написать if ($a) return;
следи за руками, меняем return на $return и добавляем строку логики после свежесозданной переменной if() $return; $foo++;

Varyen
26.06.2017
17:08:20
будешь писать все условия со скобками - не нужно додумывать "так, я ща добавлю тут еще строчку, ой надо теперь то что выше в скобки выносить

> понижаешь свой же шанс на ошибку + отбивка пробелами лучше заметна

и скобки тоже

Like
26.06.2017
17:09:22
будешь писать все условия со скобками - не нужно додумывать "так, я ща добавлю тут еще строчку, ой надо теперь то что выше в скобки выносить
Все условия? Если в блоке условия будет более-менее "нормальное" выражение или строчка кода , которая ну просто должна быть выделена с помощью {} - я это сделаю А я говорю про "незначительный" код в блоке условия

Если нужно будет добавить еще кусок кода, а там нету {} - я их добавлю

В чем проблема то?)

Google
Like
26.06.2017
17:09:45
Вы какие-то странные, серьезно

Varyen
26.06.2017
17:09:48
в том что нужно об этом париться

Like
26.06.2017
17:09:55
О чем?

Varyen
26.06.2017
17:10:07
о добавлении, почему не написать их сразу?

Pavel
26.06.2017
17:10:20
В твоем коде легче допустить ошибку

Like
26.06.2017
17:10:36
Один говорит о том, что {} меняют логику Второй говорит про ТВОЮ ошибку (return -> $return) А третий говорит, что сложно добавить {} если это необходимо

Varyen
26.06.2017
17:10:50
меня лично очень раздражало, когда подобные однострочные условия нужно было сделать больше чем на одну строку

лучше сразу писать их

Like
26.06.2017
17:11:16
Зачем тернарки вообще? Вы же без {} не можете

Admin
ERROR: S client not available

Varyen
26.06.2017
17:11:36
с тернарками тоже надо аккуратнее я считаю

dypa
26.06.2017
17:11:42
Ну и к чему это?
не выигрывать тебе в напёрстки ©

Pavel
26.06.2017
17:11:47
Тернарник не меняет поток выполнения, а изменяет только значение

Чаще всего

Varyen
26.06.2017
17:12:11
если ты напишешь тернарку с кучей условий, а то и связку двух сразу в одной строке - чем это круто для читаемости? )

Like
26.06.2017
17:12:15
Как отсутсвие {} может изменить логику? (поток выполнения) Или о чем ты?

Varyen
26.06.2017
17:12:43
а зачем пробелы экономить и {} не писать? )

Like
26.06.2017
17:12:47
Я же не говорю, что вообще забить на {}

Дак зачем они тут: if ($a) return;

Google
Varyen
26.06.2017
17:13:01
короч, по своему личному опыту я пишу {} всегда, авторы PSR пришли к тому же выводу

Pavel
26.06.2017
17:13:10
Не скобки меняют, а if сам по себе. И это СТОИТ выделить скобками

Like
26.06.2017
17:13:40
Не скобки меняют, а if сам по себе. И это СТОИТ выделить скобками
Ок, теперь понял о чем ты Зачем тогда сделали возможным такую конструкцию вообще?)

Чтобы на нее положили хер и сказали " ну йопт, тут return, я буду выделять" ?

Varyen
26.06.2017
17:14:30
я не очень правда понимаю зачем ты этот разговор завел, если единственное что от тебя слышно "хаха ну вы лошпеды чо так не можете писать"

и аргументы особо не слушаешь

Pavel
26.06.2017
17:14:37
И нет проблемы с вводом лишних с символов. В phpstorm например $cond.if -> tab -> if($cond){}

Like
26.06.2017
17:14:59
Почему нету такой проблемы в других языках? В некоторых вообще не пишут {} Что с вами не так?)

Aleh
26.06.2017
17:15:00
нужен prettier для пыха, чтоб не было никаких споров)

Varyen
26.06.2017
17:15:31
отбивка логики пробелами, как в питоне или jade, по мне так ад

Varyen
26.06.2017
17:15:52
или в sass (не scss)

Like
26.06.2017
17:16:02
Вы говорите о том, что лично ВАМ удобно и нравится (ну это правильно наверное)

Varyen
26.06.2017
17:16:03
ну значит ты победил, пиши в строчку )

нет, я говорю о опыте и о PSR

Like
26.06.2017
17:16:25
Я же попросил забить на тот psr )

Varyen
26.06.2017
17:16:38
ну сорян, вот это по-моему наиболее слабый аргумент

Like
26.06.2017
17:16:43
Varyen
26.06.2017
17:16:46
"забить на psr"

Like
26.06.2017
17:17:07
ну сорян, вот это по-моему наиболее слабый аргумент
Кроме Psr'a есть и другие стандарты )

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