如何整理多个 if 条件以提高可读性？

Question

如何整理多个 if 条件以提高可读性？

例如，

if( $key != 'pg_id' && $key != 'pg_tag' && $key != 'pg_user'  )

当处于这种情况的物品变长时，这会让我感到困惑。

它在下面的这种情况下使用 - foreach()，

$editable_fields = array(
    'pg_id',
    'pg_url',
    'pg_title',
    'pg_subtitle',
    'pg_description',
    'pg_introduction',
    'pg_content_1',
    'pg_content_2',
    'pg_content_3',
    'pg_content_4',
    'pg_backdate',
    'pg_highlight',
    'pg_hide',
    'pg_cat_id',
    'ps_cat_id',
    'parent_id',
    'tmp_id',
    'usr_id'
);

$sql_pattern = array();

foreach( $editable_fields as $key )
{
    if( $key != 'pg_id' && $key != 'pg_tag' && $key != 'pg_user'  ) $sql_pattern[] = "$key = ?";
}

我正在考虑使用 switch 但我想我错了！

Answer 1

您可以使用 in_array：

if (!in_array($key, array('pg_id', 'pg_tag', 'pg_user'))

Answer 2

一种可能的解决方案是将每个条件放在自己的行上：

foreach( $editable_fields as $key )
{
    if( $key != 'pg_id' && 
        $key != 'pg_tag' && 
        $key != 'pg_user'  )
    { 
        $sql_pattern[] = "$key = ?";
    }
}

我发现这为我澄清了这些类型的复合条件。

Answer 3

因此，使用换行符：

if( $key != 'pg_id' &&
    $key != 'pg_tag' &&
    $key != 'pg_user'
 ){
    $sql_pattern[] = "$key = ?";
}

使其更具可读性。

switch 语句仅用于测试一种条件。就像您要检查号码一样。

$number = 12;
switch($number){
  case 1:
    // Do stuff...
  case [...]
}

Answer 4

您可以将其展开：

if(
    $key != 'pg_id' && 
    $key != 'pg_tag' && 
    $key != 'pg_user'
) {
    /* code here */
    $sql_pattern[] = "$key = ?"
}

...或使用 switch 语句

switch ($key) {
    default:
        /* code here */
        $sql_pattern[] = "$key = ?"
        break;
    case 'pg_id':
    case 'pg_tag':
    case 'pg_user':
        break;
}

Answer 5

重构为具有描述其用途的名称的函数：

function KeyIsNotMatch($key) {
    if( $key != 'pg_id' && $key != 'pg_tag' && $key != 'pg_user'  )
        return true;
    else
        return false;
}

foreach( $editable_fields as $key )
{
    if (KeyIsNotMatch($key)) $sql_pattern[] = "$key = ?";
}