把赋值运算符写在 if 判断里好吗？

[Jat001]

太容易跟比较运算符搞混了，对 php 新手也不友好。

嗯，还是 @explon 一下吧~

[bearcat001]

C里为了避免把判断写成赋值，一般这样做
if (3 == a)

[bearcat001]

没写完就提交了...
编译器会报错的，PHP也一样
所以赋值用 $a = 3 ，判断反过来写 3 == $a

[for4]

这个要看是什么语言。
大部分语言是不建议这样写的。

[for4]

没看到TAG ，
PHP里面 不要这样写

[ashitaka]

第一种方法比较好 意图很明显 判断$a 的值是否为空
第二种意图混乱 你到底是判读$a 的值是否为空 还是$a的值是否为'foo'
虽然此处按 lz 的意思是前者
但是当代码给别的人看时 就要通过判断来推测原作者意图了

[raincious]

看习惯。两种代码没有区别，只是节省了行数。属于小Trick类型的语法。

个人倾向于如果$a接受的是函数返回值，那么无所谓了。但是如果接受的是变量set，那么上面的语法是没有意义的。

$a = 'foo'必然不是false，所以if里面的语法必然会被执行。所以简直不如就是

$a = 'foo';
echo $a;

[Jat001]

@raincious 上面的列子可能太简单了，再举一个：
if ($a = trim($_GET['foo']))
$a 接受的确实是函数返回值。

[romotc]

@Jat001 一般后面需要用到$a 才这样写，即完成判断，又完成赋值。

[yukirock]

既然是給 $a 賦值爲字符串常量，那麼爲什麼又要做一次 if 判斷呢？
這樣的代碼是不是重構可能更好？

[zythum]

@yukirock 她只是举个例子

如果这样的例子是不是更好

if ($param = $_GET['param']) {
//then use $param here.
}

[jsonline]

等你哪天遇到坑了就知道了。

[jsonline]

写代码最好的习惯是，每一行都 donot make me think.

[raincious]

@yukirock

对编译器来说基本山是一回事。但是多了一行，很多人感觉不爽，包括我。

@Jat001

还是看规范的问题。PHP的PSR2没有规定这样的内容，可以理解为它把这个交给了程序员自行组织。

如果你能制定规则的话，那么可以指定一个规则来约束程序员的写法。如果不能的话，可以这样改变自己的思维来适应：

你想要检查$_GET['foo']这个里面的值是不是设定，包括是空白，那么原先的写法事实上是

if (trim($_GET['foo'])) { // 外部变量请自行isset
$a = $_GET['foo'];
// TODO
}

然后你发现，这样浪费了一行，自然要想办法修改，那么就顺理成章的改成了
if ($a = trim($_GET['foo'])) {
// TODO: Use $a
}

个人感觉可读性差别不大，而且个人也没有因为这个出过错。可能只是对待刚刚入门的新人时应该注意下。

[messense]

最近看 APUE 里面其实很多赋值语句写在 if 里，不过一般是这样的：

int fd;
if ((fd = open("/dev/null", O_RDWR)) < 0) {
fprintf(stderr, "open error");
exit(1);
}

[raincious]

@raincious 哦，说错了：

- 你想要检查$_GET['foo']这个里面的值是不是设定，包括是空白，那么原先的写法事实上是
+ 你想要检查$_GET['foo']这个里面的值是不是非空，包括是空白，那么原先的写法事实上是

[nicai000]

极其讨厌楼主和一楼的写法

[hhkbp2]

不是太好，不过go...

[Jat001]

@raincious 我一般这样写
$a = trim($_GET['foo']);
if ($a) {
//do some thing
}
可公司的代码基本是这样写的
if ($a = trim($_GET['foo'])) {
//do some thing
}
因为我有强迫症，所以要发个帖子问问……

另外我也写 shell 脚本，在 shell 中，一个等号与两个等号是一样的，都是比较运算符。

[explon]

你的举例不对，公司大部分逻辑是这样子的，基本赋值的是一个数组，存在下面执行循环

if ($inbox_dialog = $this->model('message')->get_inbox_message($_GET['page'], get_setting('contents_per_page'), $this->user_id))
{

foreach ($inbox_dialog as $key => $val)
{

[explon]

@Jat001 90% 以上 if 赋值都是 model 返回来一个数组，没有你举例的那种情况存在