Programming Forums - View Single Post

DaWei · Dec 10th, 2006, 11:55 PM

What is this? (rearranged for logical clarity)

if (!$dblink) 
{
   // Empty blocks are acceptable, but why?
}
elseif (!mysql_select_db($this->dbname, $dblink)) 
{
   $dblink = false;
}

That's like saying,

if ($dblink && !mysql_select_db ($this->dbname, $dblink)) $dblink = false;

Which would you rather come across, cold?

Dec 10th, 2006, 11:55 PM	#2
DaWei Resident Grouch Join Date: Jun 2005 Posts: 6,453 Rep Power: 10	What is this? (rearranged for logical clarity) (Toggle Plain Text) if (!$dblink) { // Empty blocks are acceptable, but why? } elseif (!mysql_select_db($this->dbname, $dblink)) { $dblink = false; } if (!$dblink) { // Empty blocks are acceptable, but why? } elseif (!mysql_select_db($this->dbname, $dblink)) { $dblink = false; } That's like saying, (Toggle Plain Text) if ($dblink && !mysql_select_db ($this->dbname, $dblink)) $dblink = false; if ($dblink && !mysql_select_db ($this->dbname, $dblink)) $dblink = false; Which would you rather come across, cold? __________________ Abstraction doesn't make it impossible to write bad code; it makes it possible to write superior code. Contributor's Corner: Grumpy on C++ Exceptions DaWei on Pointers