我正在创建一个大型 PHP 项目,但我对如何进行有一点疑问。
假设我们有一个类books
,在这个类中我有方法ReturnInfo
:
function ReturnInfo($id) {
if( is_numeric($id) ) {
$query = "SELECT * FROM books WHERE id='" . $id . "' LIMIT 1;";
if( $row = $this->DBDrive->ExecuteQuery($query, $FetchResults=TRUE) ) {
return $row;
} else {
return FALSE;
}
} else {
throw new Exception('Books - ReturnInfo - id not valid.');
}
}
然后我有另一种方法PrintInfo
function PrintInfo($id) {
print_r( $this->ReturnInfo($id) );
}
显然,代码示例只是示例,而不是实际的生产代码。
在第二种方法中,我应该(再次)检查 id 是否为数字?或者我可以跳过它,因为在第一种方法中已经注意了,如果不是异常就会抛出?
到目前为止,我总是用冗余检查编写代码(无论是否已经在其他地方检查过,我也会在这里检查)
有最佳实践吗?只是常识吗?
预先感谢您的友好答复。