这些初学者风格代码有什么问题？

Question

我是一个初学者，正在阅读https://stackoverflow.com/questions/237241/what-c​​oding-mistakes-are-a-telltale-giveaway-of-an-experienced-programmer 。 我什么都没得到。

1.为什么不喜欢写这样的东西？

if (IsAManMan == true) { 
   Console.WriteLine("It's a MAN, man!");
}

2.这是什么？

if (test) {
  return true;
}
else {
  return false;
}

我不写上面的代码。 应该这样写吗： return test ？ 还是出于可读性，有时我会像return test?true:false或return (test==true)? true:false那样写它return (test==true)? true:false return (test==true)? true:false

Answer 1

在此示例中：

if (IsAManMan == true) { 
   Console.WriteLine("It's a MAN, man!");
}

在大多数语言中， if条件始终评估布尔表达式，则表示IsAManMan只能为true或false。 因此，将其与true或false进行比较是不必要的，因为这是隐含的 。 所以我们只写这个：

if (IsAManMan) {
   Console.WriteLine("It's a MAN, man!");
}

在这个例子中：

if (test) {
  return true;
}
else {
  return false;
}

按照上面的示例，这意味着test只能为true或false，这就像说：

if (true) {
  return true;
}
else { // If it's not true, it's false
  return false;
}

这又是不必要的冗长代码，所以我们只是

return test;

代替。

最后，这仍然是首选项和可读性的问题。 例如，如果您认为if (IsAManMan == true)比if (IsAManMan)更容易解释，则仍然可以用这种方式编写它，而编译器不会介意。

Answer 2

这只是一个重言式。 如果下雨了 ， 如果确实是下雨了，那么，您可以（或应该）省去多余的比较，检查并返回。

Answer 3

在这两种情况下，您都在测试布尔变量的值。 因为您的值已经是布尔值，所以不需要显式测试true因为它包含在if语句中（以及包含布尔测试的类似构造，例如循环）

例如

if(isAManMan == true){ 
    //...
}

可以评估为

if(true == true){ //answer: true
     //...
}

要么

if(false == true){ //answer: false
     //...
}

这和写作一样

if(isAManMan){ 
    //...
}

在第二个示例中，在确定要返回的布尔值之前，您正在检查另一个布尔值的值。

再次，

if (test) {
  return true;
}else {
  return false;
}

可以评估为

if (true == true) { //returns true
  return true;
}else {
  return false;
}

要么

if (false == true) { //returns false
  return true;
}else {
  return false;
}

if语句是多余的，因为变量已经保存了您要返回的值，因此这是等效的，并且是首选的：

return test;

需要特别注意的是，您的示例是完全有效的，但是良好的编译器会将它们优化为我提供的改进版本，因此最终归结为可读性和约定。 我相信我给出的版本更具可读性，并且更接近于公认的惯例。

Answer 4

这些代码执行相同的操作。 我认为，这只是可读性。

Answer 5

由于此标签与语言无关：

在某些语言中，关键字可以用作变量名。 在FORTRAN中，这是一个有效的语句：IF IF THEN然后ELSE ELSE。

如果将true定义为值为false的布尔变量，则建议的速记解决方案与更冗长的解决方案不同。 test，IsAMan，true和false都不是布尔值。

Python 2.0中的真实示例

>>>True = False
>>>True
False

舌头在脸颊上。

Answer 6

假设您要返回布尔值，而不是第二种情况的test的实际值。

你可以做

return !!test 

在大多数语言或类似语言中

return bool(test)

这是更明确和可读的。