chee

class Display
{
    public static function page($page)
    {
         // do display
    }
}

Arh, прокоментировал твои рассуждения по поводу моего варианта, тут http://phpforum.su/index.php?act=Print&cli...limin=9&limit=1

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

volter9, я читат статейку на хабре про условия ступеньками, но потерял статью(жаль). Я именно оттуда взял правило, сокращать уровень вложености условий, а досрочный выход это лишь один из вариантов.


_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

Тут вопрос про именно это место. Досрочный выход - естественно полезна вещь. Но тут он не приносит читабельности, так как
1. увеличивает объем кода
2 удлинняет алгоритм.

А выходить неоткуда, дальше кода нет.

Вот наглядно

function __autoload($fileName)
{
    if (!file_exists($fileName)) {
        return;
    }

    require $fileName;
}

Алгоритм:
Если пойти от обратного, что файл существует, выходим из функции. Если не вышли, подключаем файл. Конец функции.

И вот так:

function __autoload($fileName)
{
    if (file_exists($fileName)) {
        require $fileName;
    }
}

Алгоритм:
Если файл существует, подключаем его. Конец функции.

И что читабельнее? Тут можно только сослаться на собственный стиль построения алгоритмов. Хотя я в репе такого чет не заметил, может плохо смотрел. Не суть. Факт остается фактом. Короткий алгоритм читабелнее длинного, если используются одни и те же инструменты. Можно еще сократить:

function __autoload($fileName)
{
    return file_exists($fileName) ? require $fileName  : '';
}

Но это полный изврат. Читабельность, это золотая середина.

Кстати, полезную тему задели, так можно и алгоритм оценки читабельности придумать. :)

_____________
Если вам недостаточно собственных заблуждений, можно расширить их мнениями экспертов.

Нужно уважать мнение оппонета. Ведь заблуждаться - его святое право.

Настаивал, настаиваю и буду настаивать на своем. На кедровых орешках.

Кстати, если есть сомнения, что я не знаю о досрочном выходе, смотрите сюда. Строка 50, метод deleteCategory(). Там 4 досрочных выхода.

_____________
Если вам недостаточно собственных заблуждений, можно расширить их мнениями экспертов.

Нужно уважать мнение оппонета. Ведь заблуждаться - его святое право.

Настаивал, настаиваю и буду настаивать на своем. На кедровых орешках.

И вообще, самый читабельный и грамотный код вот:

function __autoload($fileName)
{
    @require $fileName;
}

И если бы не паранойя на собак, лучше и придумать нельзя. Опровергните? :D



_____________
Если вам недостаточно собственных заблуждений, можно расширить их мнениями экспертов.

Нужно уважать мнение оппонета. Ведь заблуждаться - его святое право.

Настаивал, настаиваю и буду настаивать на своем. На кедровых орешках.

chee