前言
轉眼間成為一名PHPer已經快整整兩年了,在這期間也對如何寫出可讀性高,便於擴充套件的程式碼有了一些自己的想法。
使用引用
場景一:遍歷一個陣列獲取新的資料結構
也許你會這樣寫:
// 申明一個新的陣列,組裝成你想要的資料
$tmp = [];
foreach ($arr as $k => $v) {
// 取出你想要的資料
$tmp[$k]['youwant'] = $v['youwant'];
...
// 一系列判斷得到你想要的資料
if (...) {
$tmp[$k]['youwantbyjudge'] = 'TIGERB';
}
...
}
// 最後得要你想要的陣列$tmp
-------------------------------------------------------
// 也許你覺著上面的寫法不是很好,那我們下面換種寫法
foreach ($arr as $k => $v) {
// 一系列判斷得到你想要的資料
if (...) {
// 複寫值為你想要的
$arr[$k]['youwantbyjudge'] = 'TIGERB'
}
...
// 幹掉你不想要的結構
unset($arr[$k]['youwantdel']);
}
// 最後我們得到我們的目標陣列$arr
複製程式碼
接下來我們使用引用值:
foreach ($arr as &$v) {
// 一系列判斷得到你想要的資料
if (...) {
// 複寫值為你想要的
$v['youwantbyjudge'] = 'TIGERB'
}
...
// 幹掉你不想要的結構
unset($v['youwantdel']);
}
unset($v);
// 最後我們得到我們的目標陣列$arr
複製程式碼
使用引用是不是使我們的程式碼更加的簡潔,除此之外相對於第一種寫法,我們節省了記憶體空間,尤其是再操作一個大陣列時效果是及其明顯的。
場景二:傳遞一個值到一個函式中獲取新的值
基本和陣列遍歷一致,我們只需要宣告這個函式的這個引數為引用即可,如下:
function decorate(&$arr = []) {
# code...
}
$arr = [
....
];
// 呼叫函式
decorate($arr);
// 如上即得到新的值$arr,好處還是節省記憶體空間
複製程式碼
使用try...catch...
假如有下面一段邏輯:
class UserModel
{
public function login($username = '', $password = '')
{
code...
if (...) {
// 使用者不存在
return -1;
}
code...
if (...) {
// 密碼錯誤
return -2;
}
code...
}
}
class UserController
{
public function login($username = '', $password = '')
{
$model = new UserModel();
$res = $model->login($username, $password);
if ($res === -1) {
return [
'code' => '404',
'message' => '使用者不存在'
];
}
if ($res === -2) {
return [
'code' => '400',
'message' => '密碼錯誤'
];
}
code...
}
}
複製程式碼
我們用try...catch...改寫後:
class UserModel
{
public function login($username = '', $password = '')
{
code...
if (...) {
// 使用者不存在
throw new Exception('使用者不存在', '404');
}
code...
if (...) {
// 密碼錯誤
throw new Exception('密碼錯誤', '400');
}
code...
}
}
class UserController
{
public function login($username = '', $password = '')
{
try {
$model = new UserModel();
$res = $model->login($username, $password);
// 如果需要的話,我們可以在這裡統一commit資料庫事務
// $db->commit();
} catch (Exception $e) {
// 如果需要的話,我們可以在這裡統一rollback資料庫事務
// $db->rollback();
return [
'code' => $e->getCode(),
'message' => $e->getMessage()
]
}
}
}
複製程式碼
通過使用try...catch...使我們的程式碼邏輯更加清晰,try...裡只需要關注業務正常的情況,異常的處理統一在catch中。所以,我們在寫上游程式碼時異常直接丟擲即可。
使用匿名函式
** 構建函式或方法內部的程式碼塊 **
假如我們有一段邏輯,在一個函式或者方法裡我們需要格式化資料,但是這個格式化資料的程式碼片段出現了多次,如果我們直接寫可能會想下面這樣:
function doSomething(...) {
...
// 格式化程式碼段
...
...
// 格式化程式碼段[重複的程式碼]
...
}
複製程式碼
我相信大多數的人應該不會像上面這麼寫,可能都會像下面這樣:
function doSomething(...) {
...
format(...);
...
format(...);
...
}
// 再宣告一個格式花程式碼的函式或方法
function format() {
// 格式化程式碼段
...
}
複製程式碼
上面這樣的寫法沒有任何的問題,最小單元化我們的程式碼片段,但是如果這個format函式或者方法只是doSomething使用呢?我通常會像下面這麼寫,為什麼?因為我認為在這種上下文的環境中format和doSomething的一個子集。
function doSomething() {
...
$package = function (...) use (...) { // 同樣use後面的引數也可以傳引用
// 格式化程式碼段
...
};
...
package(...);
...
package(...);
...
}
複製程式碼
** 實現類的【懶載入】和實現設計模式的【最少知道原則】 **
假如有下面這段程式碼:
class One
{
private $instance;
// 類One內部依賴了類Two
// 不符合設計模式的最少知道原則
public function __construct()
{
$this->intance = new Two();
}
public function doSomething()
{
if (...) {
// 如果某種情況呼叫類Two的例項方法
$this->instance->do(...);
}
...
}
}
...
$instance = new One();
$instance->doSomething();
...
複製程式碼
上面的寫法有什麼問題?
- 不符合設計模式的最少知道原則,類One內部直接依賴了類Two
- 類Two的例項不是所有的上下文都會用到,所以浪費了資源,有人說搞個單例,但是解決不了例項化了不用的尷尬
所以我們使用匿名函式解決上面的問題,下面我們這麼改寫:
class One
{
private $closure;
public function __construct(Closure $closure)
{
$this->closure = $closure;
}
public function doSomething()
{
if (...) {
// 用的時候再例項化
// 實現懶載入
$instance = $this->closure();
$instance->do(...)
}
...
}
}
...
$instance = new One(function () {
// 類One外部依賴了類Two
return new Two();
});
$instance->doSomething();
...
複製程式碼
減少對if...else...的使用
如果你碰見下面這種型別的程式碼,那一定是個黑洞。
function doSomething() {
if (...) {
if (...) {
...
} esle {
...
}
} else {
if (...) {
...
} esle {
...
}
}
}
複製程式碼
** 提前return異常 **
細心的你可能會發現上面這種情況,可能絕大多數else程式碼裡都是在處理異常情況,更有可能這個異常程式碼特別簡單,通常我會這麼去做:
// 如果是在一個函式裡面我會先處理異常的情況,然後提前return程式碼,最後再執行正常的邏輯
function doSomething() {
if (...) {
// 異常情況
return ...;
}
if (...) {
// 異常情況
return ...;
}
// 正常邏輯
...
}
// 同樣,如果是在一個類裡面我會先處理異常的情況,然後先丟擲異常
class One
{
public function doSomething()
{
if (...) {
// 異常情況
throw new Exception(...);
}
if (...) {
// 異常情況
throw new Exception(...);
}
// 正常邏輯
...
}
}
複製程式碼
** 關聯陣列做map **
如果我們在客戶端做決策,通常我們會判斷不同的上下文在選擇不同策略,通常會像下面一樣使用if或者switch判斷:
class One
{
public function doSomething()
{
if (...) {
$instance = new A();
} elseif (...) {
$instance = new A();
} else {
$instance = new C();
}
$instance->doSomething(...);
...
}
}
複製程式碼
上面的寫法通常會出現大量的if語句或者switch語句,通常我會使用一個map來對映不同的策略,像下面這樣:
class One
{
private $map = [
'a' => 'namespace\A', // 帶上名稱空間,因為變數是動態的
'b' => 'namespace\B',
'c' => 'namespace\C'
];
public function doSomething()
{
...
$instance = new $this->map[$strategy];// $strategy是'a'或'b'或'c'
$instance->doSomething(...);
...
}
}
複製程式碼
使用介面
為什麼要使用介面?極大的便於後期的擴充套件和程式碼的可讀性,例如設計一個優惠系統,不同的商品只是在不同的優惠策略下具備不同的優惠行為,我們定義一個優惠行為的介面,最後對這個介面程式設計即可,虛擬碼如下
Interface Promotion
{
public function promote(...);
}
class OnePromotion implement Promotion
{
public function doSomething(...)
{
...
}
}
class TwoPromotion implement Promotion
{
public function doSomething(...)
{
...
}
}
複製程式碼
控制器拒絕直接的DB操作
最後我想說的是永遠拒絕在你的Controller裡直接操作DB,為什麼?我們的程式絕大多數的操作基本都是增刪改查,可能是查詢的where條件和欄位不同,所以有時候我們可以抽象的把對資料庫增刪改查的方法寫到model中,通過引數暴露我們的where,fields條件。通常這樣可以很大程度的提高效率和程式碼複用。比如像下面這樣:
class DemoModel implement Model
{
public function getMultiDate($where = [], $fields = ['id'], $orderby = 'id asc')
{
$this->where($where)
->field($fields)
->orderby($orderby)
->get();
}
}
複製程式碼
最後
如果有寫的不對的地方,歡迎大家指正,THX~