Прошу заценить мой стиль программирования

[Лисю]

Прошу заценить мой стиль программирования

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

http://air-studia.com/my_php/

Особено интересует, насколько гибко я реализую обработку POST данных, по "схеме" Фаната.

 

[Alexandre]

IMXO почитай Рефакторинг: улучшение существующего кода
и наверно выберешь другой стиль написание кода, хотя те же слова Фаната "ООП - не панацея"

 

[Лисю]

Alexandre
другой это какой?

 

[SiMM]

Что-то мне подсказывает, что у тебя error_reporting(E_ALL ^ E_NOTICE). Рекомендую поставить E_ALL и убрать возникающие NOTICE.
PS: my_extract, возможно, дыра. Вообще это ИМХО довольно грязно - подменивать значения $_POST и иных предопределённых переменных.

 

[confguru]

if(isset($_SERVER["REQUEST_METHOD"]) &&  $_SERVER["REQUEST_METHOD"]=="POST")
{	
$gb_id = isset($_POST["gb_id"]) ? intval($_POST["gb_id"]) : 0;
} else {
$gb_id = isset($_GET["gb_id"]) ? intval($_GET["gb_id"]) : 0;
}

 

[Лисю]

admin
а что, $_SERVER["REQUEST_METHOD"] может не существовать?
зачем isset?

 

[Groove]

Лисю
а подсветку синтаксиса было сложно сделать?

 

[tf]

Лисю ну если сам проверяеш на существование то может быть и может)
Groove к себе скопируй и любуйся
ps. сам в классах пишу подробшейшие комментарии. не для того чтобы другим понятно было. главное самому не забыть

 

[Денч]

Лисю ну если сама проверяеш

Лиси не "Она", Лисю "Он"

 

[tf]

Денч понял. хотя может опять забуду

 

[Лисю]

да называйте как угодно, я оно-м могу побыть...

 

[tf]

:брр не мое это дело

 

[Нечто]

Посмотрел только первый класс (db).
Я бы сказал, что слишком многословно (естественно, не в плане форматирования ) и не всегда по делу. ООП чисто синтаксическое.

 

[Лисю]

Нечто
в смысле многословно?


меня по сути интересует структура модулей. разбирая скрипты других людей, понимаешь, что каждый модуль состоит из кучи каких-то мелких функкций и классов. а у меня только классы как ядро системы и модули, которые предназначены для определённых ситуаций.

 

[Нечто]

Я тоже делал обработку плейсхолдеров и она сводится к 7 строчкам, если убрать специфические для моего класса вещи:

public function query() { // начало как у тебя чисто для оформления
        $args = func_get_args();
        $sql = array_shift($args));
        if (count($args)) {
            $data = array_map('mysql_escape_string', $args);
            $sql = str_replace(array('%', '?'), array('%%', '%s'), $sql);
            $sql = vsprintf($sql, $data);
        }
        // ...
}

-~{}~ 16.12.05 18:17:

Короче, посмотри лучше peardb. Тут дело даже не в многословности, а в том, что ООП нет. В некоторых случаях ты с ним боришься, когда сущестуют естественные пути.