Question

Netbeans似乎暗示太多的任务非常糟糕，应该改变。

即

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

应该更好地编码为

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

这可以接受吗？如果是这样的话？我知道我可以在设置中关闭这个特定的提示类型，但只是检查是否有人发现了这个。

Answer 1

这是一个新警告，大约在2012年中期。

由于你使用了修剪示例，我猜你已经读过这个https://blogs.oracle.com/netbeansphp/entry/several_new_hints，试图解释警告。我对此警告的看法是，它试图警告您意外重用变量。因此它非常方便。在装饰的情况下你给它看起来有点沉重。如果重新分配是他跟随线，那可能不是偶然的。

Answer 2

理想情况下，变量名称应描述内容。如果重复分配给同一个变量，则表明变量内容没有很好地定义。

此外，如果您的代码具有以下内容：

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

如果更新代码以便在上面添加$foo = trim($foo);，会发生什么？你打破了下面的代码。

Answer 3

这些变体没有区别。如果有的话，第一个更好，因为它避免了变量使范围混乱。

我认为警告的真正含义是你应该尝试

$foo = trim(' bar ');

直接（或者首先真正设置$ foo），而不是将其存储在临时中。当然，这并不总是可行的。

Answer 4

如果你不喜欢这个提示，你可以在工具 - ＆gt;中简单地停用它。选项 - ＆gt;编辑 - ＆gt;提示 - ＆gt; PHP - ＆gt;不可变变量

Answer 5

恕我直言，第一种方式可以比较第二种方式获得更多性能。

另一方面，如果您需要更多描述性变量，第二种方法会更有帮助。如果多个人在同一个项目上工作，这可能会有所帮助。

在PHP中，太多的任务被认为是不好的？如果是这样的话？

5 个答案: