Помощь в php новичку ...

[s88s]

Доброго времени суток !

Просьба не удалять тему ... я тут буду задавать вопросы и выкладывать свои скрипты на обзор для советов что подправить что бы что бы код смотрелся хорошо так сказать ...

спасибо за ранее .

[ATL]

Это блог ?

[s88s]

Сообщение от ATL

Это блог ?

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

[Stilet]

Так а какой вопрос по топу? Выложен файл, а вопрос где?

[TranceSmile]

free - свобода.
<form action=<?php echo $PHP_SELF ?> >

[eval]

free - свобода.

freedom вроде как, или что-то поменяли?

[Stilet]

в как написан код норм или что можно подправить для красоты

Если он работает, чего его править? На скорость работы не жалуешься?

[Kostia]

ok, начнем =)

1. Не смешивайте логику с выводом! Обзаведитесь классом View.
2. Роутер, контроллеры и действия.
Т.е. если вы переходите по ссылке httр://example.com/calculator/two, то подключается контроллер calculator, который вызывает действие two, которое прописано для get запроса в роутере(REST) и возвращает конкретный view. И например в действии calc(которое прописано в html форме как action="/calculator/calc") выполняется расчет, потом вызывается действие two которое возвращает view которому вы говорите вычисленные данные и возвращаете его(для реализации ajax несколько иной подход).
3. calc или найти реализацию с обратной польской.
4. Не работать с суперглобальными переменными напрямую. Т.к. в них может прийти что угодно вплоть до кода который по не аккуратности разработчика может выполниться на сервере или у клиента(при использовании eval например), нужно реализовать класс с фильтрами(валидацией).
5. Запрещайте доступ к папкам которые не должны быть видны извне! Во первых если это apache, то в корневом .htaccess дожно быть прописано "Options All -Indexes", во вторых во всех папка куда доступ закрыт должен лежать .htaccess "deny from all", также лучше выносить конфиги и библиотеки за пределы "Document root".

[TranceSmile]

Сообщение от Kostia

ok, начнем =)

1. Не смешивайте логику с выводом! Обзаведитесь классом View.
2. Роутер, контроллеры и действия.
Т.е. если вы переходите по ссылке httр://example.com/calculator/two, то подключается контроллер calculator, который вызывает действие two, которое прописано для get запроса в роутере(REST) и возвращает конкретный view. И например в действии calc(которое прописано в html форме как action="/calculator/calc") выполняется расчет, потом вызывается действие two которое возвращает view которому вы говорите вычисленные данные и возвращаете его(для реализации ajax несколько иной подход).
3. calc или найти реализацию с обратной польской.
4. Не работать с суперглобальными переменными напрямую. Т.к. в них может прийти что угодно вплоть до кода который по не аккуратности разработчика может выполниться на сервере или у клиента(при использовании eval например), нужно реализовать класс с фильтрами(валидацией).
5. Запрещайте доступ к папкам которые не должны быть видны извне! Во первых если это apache, то в корневом .htaccess дожно быть прописано "Options All -Indexes", во вторых во всех папка куда доступ закрыт должен лежать .htaccess "deny from all", также лучше выносить конфиги и библиотеки за пределы "Document root".

Воу-воу. Полегче. Пусть хоть основы подучит. А вы ему сразу паттерн MVC предлагаете использовать.

[Kostia]

А вы ему сразу паттерн MVC предлагаете использовать.

Ну, пока только VC, без M