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

1. 3KZO (26.10.2014 / 00:48)
Всем привет, занимаюсь движком (мвс), прошу высказать свое мнение, что не так, ваши советы.

htdocs.zip

2. JustZero (26.10.2014 / 01:14)
все не так. запусти хоч на беспл. хосте

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

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

3. 3KZO (26.10.2014 / 03:30)

alexandr.sytnyk (26 Октября 2014 / 01:14)
все не так. запусти хоч на беспл. хосте

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

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

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

4. Кевин Митник (26.10.2014 / 18:47)

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

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

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

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

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

а вообще то молодец, конечно же.

5. 3KZO (26.10.2014 / 19:57)

Кевин Митник_HHTeam (26 Октября 2014 / 17:47)

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

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

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

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

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

а вообще то молодец, конечно же.

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

6. JustZero (27.10.2014 / 04:49)
стиль кода не нравиться.
насчет безопасности не слышал походу?
записывать открытый пароль в куки и бд...

7. 3KZO (27.10.2014 / 07:24)

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

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

8. Михаил (27.10.2014 / 08:46)

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

если

9. JustZero (27.10.2014 / 13:49)
мда, ты наверное гуру безопасности.

10. 3KZO (27.10.2014 / 15:26)

alexandr.sytnyk (27 Октября 2014 / 12:49)
мда, ты наверное гуру безопасности.

маг

11. Кевин Митник (28.10.2014 / 00:40)
и подготовленные выражения - не панацея от инъекций. нужно обрабатывать данные.

12. 3KZO (28.10.2014 / 05:08)

Кевин Митник_HHTeam (27 Октября 2014 / 23:40)
и подготовленные выражения - не панацея от инъекций. нужно обрабатывать данные.

с этим проблем не возникнет.

URL: https://visavi.net/topics/41490