Войти или зарегистрироваться
  • Этот сайт использует файлы cookie. Продолжая пользоваться данным сайтом, Вы соглашаетесь на использование нами Ваших файлов cookie. Узнать больше.
  • Вы находитесь в сообществе Rubukkit. Мы - администраторы серверов Minecraft, разрабатываем собственные плагины и переводим на различные языки плагины наших коллег из других стран.

  • Данный раздел создан исключительно для релизов! Вопросы по лаунчеру или обвязке задавайте ТОЛЬКО в соответсвующей теме автора. Любые другие темы будут удалены, а авторы понесут наказание.

Веб ZephCMS [0.1][Free][Alpha]

Статус
Закрыто для дальнейших ответов.
ZephyR сказал(а):
Спасибо,прочитал,лайкнул,очень полезная информация,возьму себе на заметку!
Нажмите, чтобы раскрыть...
Рад, что адекватно отреагировал) Я уже, если честно, ожидал гнева и ярости :D Удачи в разработке)
 
@gomulinsergey72, нету тут Зетрикса :confused: зачем оффтопишь?
 
Вот сидят и обсерают пацана, что в CMS дыры и т.д. А ему всего 14 лет как он выше указал.... Вот лучше вощьмите и укажите ему на эти дыры или не пишите всякую чушню если не хотите помогать...
 
mr.xp сказал(а):
Вот сидят и обсерают пацана, что в CMS дыры и т.д. А ему всего 14 лет как он выше указал.... Вот лучше вощьмите и укажите ему на эти дыры или не пишите всякую чушню если не хотите помогать...
Нажмите, чтобы раскрыть...
Да правда, что вы его поносите?! Он молодец, вы сами то на что способны? Дыры есть, но это всего лишь Beta, естественно они будут, имхо не все доработано!
 
@Alimentum, я бы даже сказа это всего-лишь недо-альфа, так как и самим разработчиком тестировалась наверное мало (ну хотели получить что-то вот и получили)
 
А ему всего 14 лет как он выше указал.... Вот лучше вощьмите и укажите ему на эти дыры или не пишите всякую чушню если не хотите помогать
Нажмите, чтобы раскрыть...
Возможно всем будет абсолютно пофиг, но решил немножко помочь своими наблюдениями.

Если поменять номер версии с 1.0 на 0.1, то будет больше соответствовать действительности.

По коду вот то, что сразу бросилось в глаза:

1. Под каждую функцию или процедуру выделен новый файл (!!!).
Попробуй объединить и реорганизовать действия например через функции (в коде не встретил не одной пользовательской функции) и конструкции языка.
Но это если следовать процедурной логики программирования. Если же применять ООП с MVC, то это можно просто все удалить и начать сначала, предварительно окунувшись в дебри мануалов.

2. Большое количество дублируемого кода. Так например на текущий момент в каждом файле приходится восстанавливать сессию и права пользователя (на данный момент процедура копируется из файла в файл и занимает ~10 строк кода),
как правило лишь для того чтобы выполнить 1 строку в самом файле.
Код авторизации администратора так же дублируется многократно и повсеместно.
Думаю не надо объяснять, что если ты захочешь, доработать проверку авторизации, тебе придется переписать все файлы где она используется.
А это будет необходимо и не раз т.к. проверки на данный момент не безопасны и отсутствует возможность запомнить пользователя ( восстановление авторизационных данных идет только по php сессии),
а тебе возможно в будущем захочется реализовать эту функцию.

Да и в общем случае количество дублируемого кода в каждом файле так и идет в соотношении 9 скопипащеных строк из 10 ( файлы avatar.php vs avatar_com_fullnews.php \ group_delete.php vs block_delete.php и т.д.)

3. В архиве есть библиотеки для работы с шаблонами, но в коде невозможно что-то понять неподготовленному человеку (и тебе, если забудешь про проект и вернешься к нему скажем через месяц)
Ввиду просто зашкаливающего вывода через echo вперемешку с вставками <? ?> которые кстати по умолчанию не на всех хостингах работают.
Как следствие HTML, PHP, кое где CSS и даже AJAX - все вперемешку.

4. Авторская обфускация кода - форматирование отсутствует, сплошной текст, только изредка перенос строки (опять же сложно будет перезапустить работу над проектом).

5. В SQL запросах конструкции вида ...'$_POST[title]'... для входных данных, если попадаются другие, то в них данные не всегда правильно проверяются.
Нужно все входные данные в момент построения SQL запроса преобразовывать через функцию mysql_real_escape_string. На данный момент безопасное формирование SQL запроса не предусмотрено.

6. Вывод скина скопирован у кого-то, эти конструкции я видел не раз.
Будут баги с отображением ног \ рук у скинов в которых важна симметрия ( например левый и правый карман на брюках, будут скопированы в одну сторону).
Нужно написать \ скопипастить функцию отзеркаливания части изображения.

7. Загрузка скина поспешна. Пользователь может загрузить что угодно, ну и возможно иногда скин.

8. Ни в одной CMS не встречал, чтобы дату и время в БД хранили по отдельности да еще и с типом данных varchar (255) для каждого поля.
в MySQL есть встроенная ф-ция NOW() и соответствующий тип данных, либо некоторые предпочитают UNIX time

9. Файл comment_post.php даже для тупого хакера подойдет, или мусорные файлы вроде RconTest.php

10. Кодировка нигде не задается. В итоге браузер пытается определить её автоматически. Кодировка обычно задается в заголовках ( header) либо в htaccess

Это 15 минут чтения кода. Запускать не стал... -_-
 
Warning: include(config/database_config.php) [function.include]: failed to open stream: No such file or directory in Z:\home\localhost\www\def.php on line 4

Warning: include() [function.include]: Failed opening 'config/database_config.php' for inclusion (include_path='.;C:\php\pear') in Z:\home\localhost\www\def.php on line 4

Warning: include(config/database_config.php) [function.include]: failed to open stream: No such file or directory in Z:\home\localhost\www\classes.php on line 3

Warning: include() [function.include]: Failed opening 'config/database_config.php' for inclusion (include_path='.;C:\php\pear') in Z:\home\localhost\www\classes.php on line 3

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\classes.php on line 5

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\classes.php on line 12

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\classes.php on line 15

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\classes.php on line 18

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\classes.php on line 22

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\classes.php on line 24

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\index.php on line 8

Fatal error: Uncaught exception 'SmartyException' with message 'Unable to load template file 'main.tpl'' in Z:\home\localhost\www\libs\sysplugins\smarty_internal_templatebase.php:127 Stack trace: #0 Z:\home\localhost\www\libs\sysplugins\smarty_internal_templatebase.php(374): Smarty_Internal_TemplateBase->fetch('main.tpl', NULL, NULL, NULL, true) #1 Z:\home\localhost\www\index.php(11): Smarty_Internal_TemplateBase->display('main.tpl') #2 {main} thrown in Z:\home\localhost\www\libs\sysplugins\smarty_internal_templatebase.php on line 127
[/spoiler]
Запускал в денвере.
 
NC22 сказал(а):
Возможно всем будет абсолютно пофиг, но решил немножко помочь своими наблюдениями.

Если поменять номер версии с 1.0 на 0.1, то будет больше соответствовать действительности.

По коду вот то, что сразу бросилось в глаза:

1. Под каждую функцию или процедуру выделен новый файл (!!!).
Попробуй объединить и реорганизовать действия например через функции (в коде не встретил не одной пользовательской функции) и конструкции языка.
Но это если следовать процедурной логики программирования. Если же применять ООП с MVC, то это можно просто все удалить и начать сначала, предварительно окунувшись в дебри мануалов.

2. Большое количество дублируемого кода. Так например на текущий момент в каждом файле приходится восстанавливать сессию и права пользователя (на данный момент процедура копируется из файла в файл и занимает ~10 строк кода),
как правило лишь для того чтобы выполнить 1 строку в самом файле.
Код авторизации администратора так же дублируется многократно и повсеместно.
Думаю не надо объяснять, что если ты захочешь, доработать проверку авторизации, тебе придется переписать все файлы где она используется.
А это будет необходимо и не раз т.к. проверки на данный момент не безопасны и отсутствует возможность запомнить пользователя ( восстановление авторизационных данных идет только по php сессии),
а тебе возможно в будущем захочется реализовать эту функцию.

Да и в общем случае количество дублируемого кода в каждом файле так и идет в соотношении 9 скопипащеных строк из 10 ( файлы avatar.php vs avatar_com_fullnews.php \ group_delete.php vs block_delete.php и т.д.)

3. В архиве есть библиотеки для работы с шаблонами, но в коде невозможно что-то понять неподготовленному человеку (и тебе, если забудешь про проект и вернешься к нему скажем через месяц)
Ввиду просто зашкаливающего вывода через echo вперемешку с вставками <? ?> которые кстати по умолчанию не на всех хостингах работают.
Как следствие HTML, PHP, кое где CSS и даже AJAX - все вперемешку.

4. Авторская обфускация кода - форматирование отсутствует, сплошной текст, только изредка перенос строки (опять же сложно будет перезапустить работу над проектом).

5. В SQL запросах конструкции вида ...'$_POST[title]'... для входных данных, если попадаются другие, то в них данные не всегда правильно проверяются.
Нужно все входные данные в момент построения SQL запроса преобразовывать через функцию mysql_real_escape_string. На данный момент безопасное формирование SQL запроса не предусмотрено.

6. Вывод скина скопирован у кого-то, эти конструкции я видел не раз.
Будут баги с отображением ног \ рук у скинов в которых важна симметрия ( например левый и правый карман на брюках, будут скопированы в одну сторону).
Нужно написать \ скопипастить функцию отзеркаливания части изображения.

7. Загрузка скина поспешна. Пользователь может загрузить что угодно, ну и возможно иногда скин.

8. Ни в одной CMS не встречал, чтобы дату и время в БД хранили по отдельности да еще и с типом данных varchar (255) для каждого поля.
в MySQL есть встроенная ф-ция NOW() и соответствующий тип данных, либо некоторые предпочитают UNIX time

9. Файл comment_post.php даже для тупого хакера подойдет, или мусорные файлы вроде RconTest.php

10. Кодировка нигде не задается. В итоге браузер пытается определить её автоматически. Кодировка обычно задается в заголовках ( header) либо в htaccess

Это 15 минут чтения кода. Запускать не стал... -_-
Нажмите, чтобы раскрыть...
Спасибо за адекватный комментарий,я уже думал,что будешь так же гавном поливать.Версию переименовал !
 
Ладно,я ухожу на часа 2 ,прийду , буду делать файлы функций,в которых будут хранится и подключение к базе,проверки администратора,и старт сессии,все,я ушел.
 
Статус
Закрыто для дальнейших ответов.

Поделиться этой страницей

Назад
Верх