代码简洁之道

目的

1. 增加代码易读性，方便后期维护和升级

问题

变量

1. 状态：1，2，3？ 坏:

   // What the heck is 4 for?
   if ($user->access & 4) {
       // ...
   }

   好:

   class User
   {
       const ACCESS_READ = 1;
       const ACCESS_CREATE = 2;
       const ACCESS_UPDATE = 4;
       const ACCESS_DELETE = 8;
   }
   
   if ($user->access & User::ACCESS_UPDATE) {
       // do edit ...
   }

2. true false ? => type: charge nocharge

3. 资源的统一：表的名字，redis资源

4. 变量定义了不使用(冗余代码)

5. 三元表达式

$num = 0;
if ($data)
{
        $num = 1;
}

推荐：

$num = $data ? 1 : 0;

1. 定义了就使用

$data  = [1, 2, 3];
// 此处200行代码 
// 对$data处理

推荐

$data  = [1, 2, 3];
// 对$data处理

函数

1. 函数应该只做一件事

2. if else a. 改写成方法 b. 提前return c. 根据返回结果归类情况 坏:

   function isShopOpen($day): bool
   {
       if ($day) {
           if (is_string($day)) {
               $day = strtolower($day);
               if ($day === 'friday') {
                   return true;
               } elseif ($day === 'saturday') {
                   return true;
               } elseif ($day === 'sunday') {
                   return true;
               } else {
                   return false;
               }
           } else {
               return false;
           }
       } else {
           return false;
       }
   }

   好:

   function isShopOpen(string $day): bool
   {
       if (empty($day)) {
           return false;
       }
   
       $openingDays = [
           'friday', 'saturday', 'sunday'
       ];
   
       return in_array(strtolower($day), $openingDays, true);
   }

3. 代码太长 拆分成方法

4. array函数来精简代码：比如：array_column等

5. 不要用flag作为函数的参数 flag就是在告诉大家，这个方法里处理很多事。前面刚说过，一个函数应当只做一件事。 把不同flag的代码拆分到多个函数里。 坏:

function createFile(string $name, bool $temp = false): void
{
    if ($temp) {
        touch('./temp/'.$name);
    } else {
        touch($name);
    }
}

好:

```php
function createFile(string $name): void
{
    touch($name);
}

function createTempFile(string $name): void
{
    touch('./temp/'.$name);
}
```

类

1. 符合MVC的结构，Controller层不要有数据库操作

冗余代码

image.png | left | 787x538

更多

https://github.com/php-cpm/clean-code-php

背景

重构前：装机盒子和装机APP => 8000+代码 重构后：装机APP => 2000+

目的

1. 增加代码易读性，方便后期维护和升级

问题

变量

1. 状态：1，2，3？ 坏:

   // What the heck is 4 for?
   if ($user->access & 4) {
       // ...
   }

   好:

   class User
   {
       const ACCESS_READ = 1;
       const ACCESS_CREATE = 2;
       const ACCESS_UPDATE = 4;
       const ACCESS_DELETE = 8;
   }
   
   if ($user->access & User::ACCESS_UPDATE) {
       // do edit ...
   }

2. true false ? => type: charge nocharge

3. 资源的统一：表的名字，redis资源

4. 变量定义了不使用(冗余代码)

5. 三元表达式

$num = 0;
if ($data)
{
        $num = 1;
}

推荐：

$num = $data ? 1 : 0;

1. 定义了就使用

$data  = [1, 2, 3];
// 此处200行代码 
// 对$data处理

推荐

$data  = [1, 2, 3];
// 对$data处理

函数

1. 函数应该只做一件事

2. if else a. 改写成方法 b. 提前return c. 根据返回结果归类情况 坏:

   function isShopOpen($day): bool
   {
       if ($day) {
           if (is_string($day)) {
               $day = strtolower($day);
               if ($day === 'friday') {
                   return true;
               } elseif ($day === 'saturday') {
                   return true;
               } elseif ($day === 'sunday') {
                   return true;
               } else {
                   return false;
               }
           } else {
               return false;
           }
       } else {
           return false;
       }
   }

   好:

   function isShopOpen(string $day): bool
   {
       if (empty($day)) {
           return false;
       }
   
       $openingDays = [
           'friday', 'saturday', 'sunday'
       ];
   
       return in_array(strtolower($day), $openingDays, true);
   }

3. 代码太长 拆分成方法

4. array函数来精简代码：比如：array_column等

5. 不要用flag作为函数的参数 flag就是在告诉大家，这个方法里处理很多事。前面刚说过，一个函数应当只做一件事。 把不同flag的代码拆分到多个函数里。 坏:

function createFile(string $name, bool $temp = false): void
{
    if ($temp) {
        touch('./temp/'.$name);
    } else {
        touch($name);
    }
}

好:

```php
function createFile(string $name): void
{
    touch($name);
}

function createTempFile(string $name): void
{
    touch('./temp/'.$name);
}
```

类

1. 符合MVC的结构，Controller层不要有数据库操作

冗余代码

image.png | left | 787x538

更多

https://github.com/php-cpm/clean-code-php