Ghostlyr
commented
7 years ago По длине значений полей для IPv6 - максимальная длина IPv6 с учетом IPv4-совместимого IPv6 может составлять 45 символов см, что в общем то все равно довольно ощутимо влияет на объем базы.
https://dev.mysql.com/doc/refman/5.7/en/miscellaneous-functions.html
Функции MySQL INET6_NTOA(ip) | INET6_ATON(ip)` могут распознавать так и IPv4 адресы, для эффективного, производительного и многофункционального хранения в бинарном виде, но хз что в этом плане у PHP.
База данных:
uuid,loginиemailUNSIGNEDдля всех колонок, где не требуются отрицательные значения (?)genderтаблицыmcr_usersиспользовать ENUM для удобства разработкиmcr_usersпреобразовать в бинарный формат binary(16)firstname,lastname,birthday(ненадобность, 152-ФЗ РФ)color(ненадобность)commentsдля хранения "кэша" количества комментариев, оставленных пользователем (?)ip_create), не имеет особого смысла. Реализовать хранение изменения IP-адресов пользователей в отдельной таблице, подчищаемой от устарелых данных через CRON (?)text_bb,text_bb_short,text_html_shortв виду грядущих изменений CMStext_htmlна TEXT | MEDIUMTEXT - с головой хватает.dataв отдельную колонкуimgvarchar(255)hiddentinyint для скрытия новостей от пользователей (TODO CMS*)hiddentinyint для скрытых категорий выводе главной страницы модуля (TODO CMS*)CMS:
Модули:
date_lastс текущим временем прямо в момент регистрации (необходимость знать, производился ли вход в аккаунт после регистрации)UNHEX(REPLACE(UUID(), '-', ''))для бинарного вида), вместо использования функцииlogintouuidCOUNT(*), оптимизировав получение данных о количестве просмотров, лайков, дизлайков, храня "кеш" в массивеdata(AltNews) или в собственных колонках таблицыmcr_news(?)tmpбыстрее, чем пользователь проходит по ссылке сбросаБлоки:
Warning: Cannot modify header information - headers already sent by (output started at /modules/admin/blocks.class.php:128) in /engine/core.class.php on line 170Ядро и API:
colorс и без двойных кавычек[color=#ff0000]123[/color]sizeс и без двойных кавычек[size=3]123[/size]send_mail):"[{$this->cfg['main']['s_name']}] {фраза модуля}"msgв одну, данные выводить переменными внутри самих фраз.$core->sp(): проверять существование указанного файла, а при отсутствии выдавать шаблон темы по умолчанию. Текущая система загрузки HTML-шаблонов имеет серьезный недостаток - требует наличие КАЖДОГО шаблона в выбранной теме, включая админ-панель. Для верстальщика это приводит к дублированию файлов и к трудностям с поддержкой - после каждого изменения нетронутого основного шаблона он должен быть обновлен и в кастомной теме. А разработчика заставляет постоянно передавать константу MCR_THEME_PATH.HEADERиDEF_HEADERв новые переменные перед тегом</body>: в<header>должны находиться только критично важные для отображения страницы ресурсы - в данном случае только основной CSS, дабы избежать подергивания страницы при загрузке.uuid), чтобы получить чистую строку, разбирать лучше на стороне PHP и желательно заменой регулярным выражением (производительность на стороне preg_replace против substr: https://wtools.io/php-sandbox/eE)Написание сторонних модулей:
Написание документации:
Front-End:
transform. Костыль - лишние пробелы.short_open_tag = off, но ИМХО рядовому юзеру удобнее использовать сокращенный вид тегов php Объем кода сокращается и синтаксис упрощается.