2013-03-06 15 views
11

Netbeans似乎認爲太多的任務是非常糟糕的,應該改變。在PHP中,是否有太多的任務被認爲是不好的?如果是這樣,爲什

$foo = ' bar '; 
$foo = trim($foo); 

應更好地編碼爲

$foo = ' bar '; 
$trimmed_foo = trim($foo); 

這是可以接受的?如果是這樣,爲什我知道我可以在設置中關閉這個特定的提示類型,但只是檢查是否有人發現任何東西。

+0

這是一個有趣的問題。我認爲它確實無關緊要,但PHP引擎讓我感到驚訝。 – 2013-03-06 02:59:22

+0

感謝您接受不回答的問題。 – 2017-10-19 03:32:25

回答

5

理想情況下,變量名稱應描述內容。如果您重複分配給相同的變量,則表明變量內容沒有很好地定義。

另外,如果您的代碼是這樣的:

$foo = ' bar '; 
// some code, mybe 100 lines of it 
// now do something with $foo 

如果你更新的代碼在上面添加$foo = trim($foo);,會發生什麼?你打破下面的代碼。

5

這些變體沒有區別。如果第一個更好,因爲它可以避免使用變量混淆範圍。

我想警告真正的意思是你應該嘗試做

$foo = trim(' bar '); 

直接(或任何$ foo的是真正被設定爲第1名),而不是將其存儲在一個臨時的。當然,這並不總是可能的。

0

恕我直言,第一種方式可以比第二種更好的性能。

另一方面,如果您需要更多的描述性變量,第二種方法會更有幫助。如果多個人在同一個項目上工作,這也可能會有所幫助。

7

THS是一個新的警告,大約在2012年年

由於您使用的裝飾例子,我猜你一定讀過這https://blogs.oracle.com/netbeansphp/entry/several_new_hints,其試圖解釋的警告。我接受這個警告是它試圖警告你意外重用一個變量。因此它非常方便。在修剪情況下,你看起來有點沉重。如果重新分配在他後面,那可能不是偶然的。

+1

正確彼得,我讀過你引用的頁面,但它並不完全告訴你爲什麼它不好。避免意外發生,並保持開機狀態。 – xelber 2013-03-06 03:41:33

2

如果你不喜歡這個提示,你可以簡單地關閉它在工具 - >選項 - >編輯 - >提示 - > PHP - >不可變的變量

相關問題