
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

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 - и дополнительные трудозатраты

Like
26.06.2017
16:52:19

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

dypa
26.06.2017
16:52:59

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

Like
26.06.2017
16:54:45

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

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

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

Like
26.06.2017
17:06:38

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
Чтобы на нее положили хер и сказали " ну йопт, тут 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 для пыха, чтоб не было никаких споров)

Like
26.06.2017
17:15:18

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

Like
26.06.2017
17:15:52

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