2014-09-04 68 views
0

採取例如在下面的代碼是否正在使用&&分配值並執行錯誤操作?

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

我見過的事情是這樣的框架,當是太多

謝謝!

+1

對我來說太多了,但它是基於觀點的問題 – 2014-09-04 07:50:31

+1

我經常做的事情是'if(($ myObject = getSomeObject($ myParam))!== null){//有一個對象}'但我從來沒有做過在if語句之前的字符串連接。理論上你可以把每個語句放在'if'結構中。但是如上所述,它在某些時候會變得太多,導致代碼不可讀。 – TiMESPLiNTER 2014-09-04 07:52:43

回答

1

這是每個人都必須爲自己(或他的項目的編碼標準)做出的決定。

在我看來,像$var == 'test' && test();這樣的簡短的事情都可以。 但我會說你提供的例子是不好的做法。

有足夠的硬盤空間,所以這些東西是沒有必要的。

相關問題