Использует && для присвоения значений и выполнения функций плохой практики?

Возьмите в пример следующий кодИспользует && для присвоения значений и выполнения функций плохой практики?

preg_match(
    $curlyBracesCapturer, 
    $directory, 
    $match 
) && $directory != '' && isset($values[$match[1]]) && $directoryStack .= str_replace(
    $match[0], 
    $placeholdersReplacements[$match[1]] . '/', 
    $directory 
);

Я видел, что это, как это на обрамления, , когда это слишком много?

Спасибо!

источник

2014-09-04 Joel Hernandez

Для меня это слишком много, но вопрос основан на мнениях –

Что я делаю чаще всего: 'if (($ myObject = getSomeObject ($ myParam))! == null) {// получил объект}', но я никогда не делал этого string concatination ранее в 'if' -запросе. Теоретически вы можете поместить каждое утверждение в конструкцию 'if'. Но, как указано выше, в какой-то момент он становится слишком сильным и приводит к нечитаемому коду. – TiMESPLiNTER

Это то, что каждый должен решить для себя (или для стандартов кодирования своего проекта).

На мой взгляд, короткие вещи вроде $var == 'test' && test(); в порядке. Но я бы сказал, что приведенный вами пример - плохая практика.

Существует достаточно места на жестком диске, поэтому такие вещи не нужны.

источник

2014-09-04 07:54:51 Johni

Использует && для присвоения значений и выполнения функций плохой практики?

ответ

Смежные вопросы