Давайте тут собирать правила хорошего кода.

Например:
1. Не писать код работающий только с register_globals on. Вместо этого использовать $_GET[''] и $_POST['']
2. Не называть функции непонятными для посторонних именами
3. Делать комментарии в коде, чтобы можно было легко разобраться в нем.

4. Отступы.
5. Минимум переменных необходимых для решения задачи.
6. У функций return
7. Вывод гипертекста - один раз.

8. Отделение ввода данных в базу от вывода гиперетекста

Цитата(HalkaR @ 3.4.2005, 00:02)

8. Отделение ввода данных в базу от вывода гиперетекста

Я бы сказал: " отделение всего кода от вывода ьгипертекста". По сути шаблоны. Что скажешь?

Кстати говоря насчет вывода текста один раз не могу согласиться - это неправильно.

Цитата(yUAC @ 3.4.2005, 03:14)

Кстати говоря насчет вывода текста один раз не могу согласиться - это неправильно.

Почечму?
Во-первых отпадает (практически) трабла с заголовками и установкой куков.
Во-вторых существенно увеличиваетс производительность скрипта.
В третьих это более грамотно с точки зрения проектирования, т.к. в случае чего можно пройтись по всему гипертексту.

9. Краткость сестра таланта.
Пример:

Код

<?php // Есть у нас такой кусок кода: if($a == 1) {    $b = $a; } else {    $b = 2; } // Можно сократить до: $b = ($a == 1) ? $a : 2; ?>

10. Оптимизация строковых функций.
Пример:

Код

<?php // Замена переменных в шаблоне $text = ereg_replace("<!--a-->", $a, $text); $text = ereg_replace("<!--b-->", $b, $text); $text = ereg_replace("<!--c-->", $c, $text); $text = ereg_replace("<!--d-->", $d, $text); // Это по меньшей мере ужасно, но встречается очень часто.  // Оптимизируем и получаем одну строку: $text = str_replace(array('<!--a-->', '<!--b-->', '<!--c-->', '<!--d-->'), array($a, $b, $c, $d), $text); // Разбивка текста всегда осуществляем с помощью explode, // а не split'a как это очень часто делают некоторые неопытные люди. Т.е. такой код: $arr = split("|", $line); // Заменяем на  $arr = explode("|", $line); // Разумеется если вам необходимо разбить строку при помощи регулярных выражений, // лучше всего сначала подумать: а может быть есть другой выход из ситуации? ?>

PS Пока все.

Цитата(Mal @ 2.4.2005, 23:12)

Цитата(HalkaR @ 3.4.2005, 00:02) 8. Отделение ввода данных в базу от вывода гиперетекста Я бы сказал: " отделение всего кода от вывода ьгипертекста". По сути шаблоны. Что скажешь?

Полностью отделить код от html очень тяжело, но естественно желательно. Я просто более мягко сформулировал.

Слишком четкое отделение РНР кода от HTML приведет скорее всего к тому что появится еще один интерпретируемый язык, что приведет к большому снижению производительности.

http://pear.php.net/manual/en/standards.php

Irokez
это правила составления PEAR.

Mal Hack ну да я знаю ... а он на чем пишется? на пхп ...
по-моему стандарты написания пеар пакетов стоит взять за стандарты написания всех пхп скриптов ...

Irokez
я к тому, как бы что там описывается схема структура файла PEAR

Mal Hack
так пеар файл и есть ни что иное как грамотно написанный пхп скрипт,
и по ссылке описываются как такой скрипт грамотно написать (в плане синтаксиса)

Цитата(Irokez @ 7.4.2005, 21:20)

так пеар файл и есть ни что иное как грамотно написанный пхп скрипт,

не факт. Многие считаю его тяжелым

Цитата(Irokez @ 7.4.2005, 21:20)

и по ссылке описываются как такой скрипт грамотно написать (в плане синтаксиса)

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