вторник, 24 февраля 2015 г.

Банальные советы по улучшению читаемости кода

Оформление сложных условий

Плохо:
<?php 
function someFuction($a, $b, $c)
{
    if (!empty($a) && !empty($b) && !empty($c)) {
        // do something
    }
}


Хорошо:
<?php
function someFuction($a, $b, $c)
{
    if (empty($a)) {
        return;
    }
    
    if (empty($b)) {
        return;
    }
    
    if (empty($c)) {
        return;
    }    
    
    // do something
}


Сокрашение количества вложенных условий

Плохо:
<?php
function someFunction($a, $b, $c)
{
    if ($a) {
        if ($b) {
            if ($c) {
                // do something 2
            }
        }
        // do something 1
    }
}


Хорошо:
<?php
function someFunction($a, $b, $c)
{
    if (!$a) {
        return;
    }
    
    // do something 1
    if (!$b) {
        return;
    }
    
    if (!$c) {
        return;
    }
    
    // do something
}

Комментариев нет:

Отправить комментарий