Прошу оценить, проверить..

Печать RSS
432

3
Автор
Пришелец
0
Всем привет, занимаюсь движком (мвс), прошу высказать свое мнение, что не так, ваши советы.

htdocs.zip
Изменил: 3KZO (26.10.2014 / 00:49)
J

Оранжевые штаны
0
все не так. запусти хоч на беспл. хосте

Добавлено через 00:22 сек.
+ название какое то....

Добавлено через 02:29 сек.
да и 8 кб движком то и назвать нельзя
3
Автор
Пришелец
0
alexandr.sytnyk (26 Октября 2014 / 01:14)
все не так. запусти хоч на беспл. хосте

Добавлено через 00:22 сек.
+ название какое то....

Добавлено через 02:29 сек.
да и 8 кб движком то и назвать нельзя

-.- а поточнее что тебе там не нравится?
К

Айсберг Визави
0
в индексном файле зачем проверка версии php, если никакого действия не происходит?

.htaccess рекомендую указывать директивы NC и L (строчные и прописные обрабатываются одинаково, после этого преобразования ничего не делать)

rewriteRule ^(.*)$ index.php?router=$1 [NC,L]

о автоматической загрузке классов не слышал? у тебя 6 классов подключаются. а еслы будет 150, и они не нужны будут все при инициализации скрипта?

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

а вообще то молодец, конечно же.
3
Автор
Пришелец
0
Кевин Митник_HHTeam (26 Октября 2014 / 17:47)
в индексном файле зачем проверка версии php, если никакого действия не происходит?

.htaccess рекомендую указывать директивы NC и L (строчные и прописные обрабатываются одинаково, после этого преобразования ничего не делать)

rewriteRule ^(.*)$ index.php?router=$1 [NC,L]

о автоматической загрузке классов не слышал? у тебя 6 классов подключаются. а еслы будет 150, и они не нужны будут все при инициализации скрипта?

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

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

Оранжевые штаны
0
стиль кода не нравиться.
насчет безопасности не слышал походу?
записывать открытый пароль в куки и бд...
3
Автор
Пришелец
0
alexandr.sytnyk (27 Октября 2014 / 03:49)
стиль кода не нравиться.
насчет безопасности не слышал походу?
записывать открытый пароль в куки и бд...

У каждого свой стиль. . .
про безопасность слышал, но зачем шифровать бд если ее не вскроют. . .
М

Малиновые штаны
0
3KZO (27 Октября 2014 / 07:24)
если ее не вскроют. . .
если
J

Оранжевые штаны
0
мда, ты наверное гуру безопасности.
3
Автор
Пришелец
0
alexandr.sytnyk (27 Октября 2014 / 12:49)
мда, ты наверное гуру безопасности.
маг
Стикеры / Теги / Правила / Топ тем / Топ постов / Поиск