какой-то говнокод получается

[Духовность™]

какой-то говнокод получается

я пишу-пишу, а мне мой код все не нравится и не нравится. я уже устал от этого, правда.

вот код: http://adverts.ru/codes/

работает вся эта байда пока так: http://adverts.ru/admin/ (test, test)

покритикуйте архитектуру, что можно изменить пока не поздно?

 

[Adelf]

JS: use jQuery!

PHP: $maps тебе обязательно нужны в Application обьекте. Следовательно их надо принимать параметром в конструкторе.

Установка Headers в конструкторе Application очень удивила.
Дальше смотреть пока не стал.

 

[Доктор]

А что это за
MD5(CONCAT(`user_login`, `user_password`, "4357435felwfew455km")) = "758d7ff21bb45d084915c71c8d50787a"
?

 

[Splurov]

Зачем тебе закомментированный код? Репозиторий не используешь?

 

[Single]

kernel/Base/Access.php

public function getGroupRulesByIdWithControllerNames($id_group)
{
  ...
  WHERE `group`.`id` = ?i', $id_group);
}

какая то путаница в названии методов, так и не понял зачем в названии приписка "WithControllerNames" и почему нельзя было сделать пару методов.

 

[HraKK]

а в zip выложить можешь? а то я неумею такое качать

 

[fixxxer]

Это хорошо, что тебе не нравится. Более того, это совершенно нормально, когда открывая код, написанный полгода назад, ты смотришь на него, как на говно. Значит, развиваешься и учишься. Как только тебя начнет все устраивать, значит, ты остановился в развитии.

Советы с форума помогут мало. Читай больше умных книжек, качественного кода хороших фреймворков. И главное, мысли самостоятельно - ну ты вроде так и делаешь =). Осознание приходит только на практике. Code smells не когда он лежит на полочке, а когда активно используется.

 

[grigori]

мне сходу бросилась в глаза куча глобальных констант, недокументированных функций со странными названиями и определение системных параметров в kernel/configuration.php
у меня все параметры приложения в полях класса, системные настройки - в системных файлах (.htaccess, например)
определение ф-й должно идти отдельной библиотекой
дальше читать не хочется

 

[Fortop]

triumvirat
Hint: если файликам дать расширение .phps то они раскрасятся на апаче автоматически

HraKK
а в zip выложить можешь? а то я неумею такое качать

wget?

-~{}~ 29.04.10 17:22:

Base_View

$this->data->request_uri = $_SERVER['REQUEST_URI'];

А View не плевать ли какой там request_uri?
А если я через консоль скрипт хочу запустить?

Нет, я могу, конечно, сделать

system('wget http://localhost/myscript.php');

но...

    /**
     * Возвращает сгенерированый HTML-код текущего шаблона.
     */
    public function run()
    //...
    /**
     * Возвращает сгенерированный html-код.
    */
    public function getOut()

А в чем разница?

    /**
     * Принимает объект редиректа, получает необходимые значения 
     * и транслирует их во внутреннее представление view.
     * 
     * @access public
     * @param object Redirect_Redirect
     * @return void
     */
    public function setRedirect(Redirect_Redirect $redirect)

что делает - понятно. А можно узнать зачем оно это делает?

 

[HraKK]

Fortop
у меня виндовоз.

 

[Fortop]

HraKK
Teleport и иже с ним

Для извращенцев-линуксоидов есть cygwin

 

[HraKK]

Fortop
Только мне кажется что проще и логичнее triumvirat'у выложить в zip?

 

[Wicked]

HraKK
http://www.mingw.org/

 

[флоппик]

Для извращенцев-линуксоидов есть cygwin

httrack

 

[HraKK]

Ну все, прощай тема. Теперь наш пхпклаб будет с толком, чувством и расстановкой рассуждать как лучше через жопу выкачать исходник.

 

[A1x]

да wget есть под винду

 

[Духовность™]

А что это за
MD5(CONCAT(`user_login`, `user_password`, "4357435felwfew455km")) = "758d7ff21bb45d084915c71c8d50787a"
?

стандартная проверка пользователя на основании md5 хэша из куков.
4357435felwfew455km - соль. 758d7ff21bb45d084915c71c8d50787a - хэш из куки.

мне сходу бросилась в глаза куча глобальных констант

остались со старой версии, они все закомментированы и почти не используются, ща удалю.

определение ф-й должно идти отдельной библиотекой

dj всем приложении 4 функции, больше функций в приложении нет.

kernel/Base/Access.php

это очень сырой код, написан буквально вчера. я ещё не знаю что там должно быть, в этом Access.

А в чем разница?

опечатка в описании

что делает - понятно. А можно узнать зачем оно это делает?

что бы отобразить информацию, полученную с предыдущей страницы.


а что вы на меня вообще все накинулись, я не фреймворк для публичного пользования пишу, я пишу adverts.ru!! =)

 

[HraKK]

похоже меня игнорируют

 

[Духовность™]

HraKK
http://adverts.ru/codes/codes.rar

 

[Fortop]

опечатка в описании

Там и без описания не все однозначно ясно.

что бы отобразить информацию, полученную с предыдущей страницы.

? непонял. Можно чуть подробнее схему?
Или это попытка реализовать layout?

что вы на меня вообще все накинулись, я не фреймворк для публичного пользования пишу, я пишу adverts.ru!! =)

Эм? Критики хотелось? Или всем так показалось?

HraKK
Дольше всего живут люди, на которых не обращают внимания